From patchwork Wed Jul 3 22:12:48 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Richard Sent X-Patchwork-Id: 65895 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id C257B27BBE9; Wed, 3 Jul 2024 23:14:19 +0100 (BST) X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-2.7 required=5.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,MAILING_LIST_MULTI,SPF_HELO_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.6 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTPS id 60D9227BBE2 for ; Wed, 3 Jul 2024 23:14:19 +0100 (BST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1sP8Ek-0007vE-FD; Wed, 03 Jul 2024 18:14:02 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1sP8Ei-0007tL-Uv for guix-patches@gnu.org; Wed, 03 Jul 2024 18:14:01 -0400 Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1sP8Ei-0001UA-Mu for guix-patches@gnu.org; Wed, 03 Jul 2024 18:14:00 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1sP8Ek-0007G6-LK for guix-patches@gnu.org; Wed, 03 Jul 2024 18:14:02 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#71928] [PATCH] gnu: Throw error when extending services that do not use extensions. Resent-From: Richard Sent Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Wed, 03 Jul 2024 22:14:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: report 71928 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 71928@debbugs.gnu.org Cc: Richard Sent X-Debbugs-Original-To: guix-patches@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.172004482927870 (code B ref -1); Wed, 03 Jul 2024 22:14:02 +0000 Received: (at submit) by debbugs.gnu.org; 3 Jul 2024 22:13:49 +0000 Received: from localhost ([127.0.0.1]:40933 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1sP8EW-0007FS-LE for submit@debbugs.gnu.org; Wed, 03 Jul 2024 18:13:49 -0400 Received: from lists.gnu.org ([209.51.188.17]:47650) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1sP8EV-0007FK-4T for submit@debbugs.gnu.org; Wed, 03 Jul 2024 18:13:47 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1sP8ES-0007kd-SG for guix-patches@gnu.org; Wed, 03 Jul 2024 18:13:44 -0400 Received: from mail-108-mta139.mxroute.com ([136.175.108.139]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1sP8ER-0001S1-5P for guix-patches@gnu.org; Wed, 03 Jul 2024 18:13:44 -0400 Received: from filter006.mxroute.com ([136.175.111.3] filter006.mxroute.com) (Authenticated sender: mN4UYu2MZsgR) by mail-108-mta139.mxroute.com (ZoneMTA) with ESMTPSA id 1907aa90b5d00017a3.001 for (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384); Wed, 03 Jul 2024 22:13:37 +0000 X-Zone-Loop: 557bade68aaa11c4221e3a536d21c674027bc9e8c38a X-Originating-IP: [136.175.111.3] DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=freakingpenguin.com; s=x; h=Content-Transfer-Encoding:MIME-Version: Message-ID:Date:Subject:Cc:To:From:Sender:Reply-To:Content-Type:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:In-Reply-To:References:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=rNBm5jTy66oDdv/NCuz1/DMBQPaQo0xwiy9uyeOQ1aM=; b=GrK2qG8/xu0D2SYVfwF3ihvPbz O6vZKWu038BcB+oODrrDSSkyviYTOxDcP11PjkDpbtBIZRckXrUnxwAyW2XM9tXiMZAhsqLbrpEDh LYInQeq8qzr3VK9Ej/k35VE1D6Lg1FqKHk200JRYoWbl3c+4wS0FSXIGqQ7CQ7PG7v4Asn1PaVSPA QfqGfgljWGpOl5Ru8FxX3QWW4D8C+wF1TiqIGxRg123pVJH7IZYDGk4AkhVs6NGs08IOsJ8HyYB9C a7Lm9T4y6aQpaG7nfw66ZLceXuP2cYFcUtW3+jL8OOink9SYwMo61NoJ/FmghDOksuANyHOh9X2Zi 4WYSyZvg==; From: Richard Sent Date: Wed, 3 Jul 2024 18:12:48 -0400 Message-ID: MIME-Version: 1.0 X-Authenticated-Id: richard@freakingpenguin.com Received-SPF: pass client-ip=136.175.108.139; envelope-from=richard@freakingpenguin.com; helo=mail-108-mta139.mxroute.com X-Spam_score_int: -16 X-Spam_score: -1.7 X-Spam_bar: - X-Spam_report: (-1.7 / 5.0 requ) BAYES_00=-1.9, DKIM_INVALID=0.1, DKIM_SIGNED=0.1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org X-getmail-retrieved-from-mailbox: Patches * gnu/services.scm (fold-services): Add error handling when using service extensions. Fixes: https://issues.guix.gnu.org/71887 Change-Id: Ic8d631674bfddde495c93952d9e6cd5649bb287d --- gnu/services.scm | 22 ++++++++++++++++------ 1 file changed, 16 insertions(+), 6 deletions(-) base-commit: 85012e64819b39fd6112038134548b415fd5daff diff --git a/gnu/services.scm b/gnu/services.scm index 88593e8091..a7a9e00478 100644 --- a/gnu/services.scm +++ b/gnu/services.scm @@ -1223,12 +1223,22 @@ (define* (fold-services services (params -> (service-value sink)) (service -> - ;; Distinguish COMPOSE and EXTEND because PARAMS typically - ;; has a different type than the elements of EXTENSIONS. - (if extend - (service (service-kind sink) - (extend params (compose extensions))) - sink))) + (begin + (unless (or (null? extensions) + ;; A value of #t is a convention for "make + ;; sure the service is present." + (every (cut eq? #t <>) extensions) + (and extend compose)) + (error (format #f "Extensions are not supported in \ +~a yet the following service types extend it:~{~% ~a~}" + (service-kind sink) + (map service-kind dependents)))) + ;; Distinguish COMPOSE and EXTEND because PARAMS typically + ;; has a different type than the elements of EXTENSIONS. + (if extend + (service (service-kind sink) + (extend params (compose extensions))) + sink)))) (mbegin %state-monad (set-current-state (vhash-consq sink service visited)) (return service))))