From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp1 ([2001:41d0:8:6d80::]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) by ms0.migadu.com with LMTPS id 0EI4N8MTY2C4eQEAgWs5BA (envelope-from ) for ; Tue, 30 Mar 2021 14:04:19 +0200 Received: from aspmx1.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp1 with LMTPS id QMxJMcMTY2DecQAAbx9fmQ (envelope-from ) for ; Tue, 30 Mar 2021 12:04:19 +0000 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id DACBD264AC for ; Tue, 30 Mar 2021 14:04:18 +0200 (CEST) Received: from localhost ([::1]:42308 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1lRD6W-0003Ze-RL for larch@yhetil.org; Tue, 30 Mar 2021 08:04:16 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:49662) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1lRD6K-0003ZX-Gy for bug-guix@gnu.org; Tue, 30 Mar 2021 08:04:04 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:38987) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1lRD6I-0000Go-8c for bug-guix@gnu.org; Tue, 30 Mar 2021 08:04:04 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1lRD6I-0005bq-4s for bug-guix@gnu.org; Tue, 30 Mar 2021 08:04:02 -0400 X-Loop: help-debbugs@gnu.org Subject: bug#47479: inkscape retains a reference to imagemagick, even though it is in native-inputs Resent-From: Efraim Flashner Original-Sender: "Debbugs-submit" Resent-CC: bug-guix@gnu.org Resent-Date: Tue, 30 Mar 2021 12:04:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 47479 X-GNU-PR-Package: guix X-GNU-PR-Keywords: To: Mark H Weaver Received: via spool by 47479-submit@debbugs.gnu.org id=B47479.161710581921528 (code B ref 47479); Tue, 30 Mar 2021 12:04:02 +0000 Received: (at 47479) by debbugs.gnu.org; 30 Mar 2021 12:03:39 +0000 Received: from localhost ([127.0.0.1]:50533 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lRD5v-0005bA-3s for submit@debbugs.gnu.org; Tue, 30 Mar 2021 08:03:39 -0400 Received: from flashner.co.il ([178.62.234.194]:38524) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1lRD5q-0005at-3U for 47479@debbugs.gnu.org; Tue, 30 Mar 2021 08:03:38 -0400 Received: from localhost (unknown [31.210.177.71]) by flashner.co.il (Postfix) with ESMTPSA id 6870E40088; Tue, 30 Mar 2021 12:03:27 +0000 (UTC) Date: Tue, 30 Mar 2021 15:02:44 +0300 From: Efraim Flashner Message-ID: References: <9beb8d6af78a517d53aaaa43179272b8953da78f.camel@telenet.be> <87lfa53bkj.fsf@netris.org> <8735wd2f77.fsf@netris.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="AnAk6LbvaDZ56IY8" Content-Disposition: inline In-Reply-To: <8735wd2f77.fsf@netris.org> X-PGP-Key-ID: 0x41AAE7DCCA3D8351 X-PGP-Key: https://flashner.co.il/~efraim/efraim_flashner.asc X-PGP-Fingerprint: A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-guix@gnu.org List-Id: Bug reports for GNU Guix List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: 47479@debbugs.gnu.org Errors-To: bug-guix-bounces+larch=yhetil.org@gnu.org Sender: "bug-Guix" X-Migadu-Flow: FLOW_IN ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1617105859; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:resent-cc:resent-from:resent-sender: resent-message-id:in-reply-to:in-reply-to:references:references: list-id:list-help:list-unsubscribe:list-subscribe:list-post; bh=DH4E907nXQJ3QBrhOT+6VTPoxIsdJzbuykwUF6bah+U=; b=iajvE26+mLZ507ekGCq6dzAlb6TKgbYyNYEzfD6LsmK6qGscF8lFmfOGlRV1tefRuCPpKW RzUkzgNZ3qicJThf5vgEbwRMjKnLmryBEZHWc3mbjQBJs5IDXBxANiKI18h+ayjL2mV8Mp JZ2C95jsIQAAXCkj7fBLEknxuibbv+Mhi1Vma5W9Xv452gQu/buWMM3y0/ZjRd8j1yFdO2 PGeyDDHwQJICApKolOj0TPACOXpy+U/LjZDVk75s6RK/2n5XqbE5El03RUaurJvuXHm7RH aCcflXQWQ7/0d6DBopM/pBze5/NDuijovEgp+vvGhi1RcjLemBCFAaXvk1g6Uw== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1617105859; a=rsa-sha256; cv=none; b=ThmHMuVwccluUW3pdV5vgcf0EAn6bvD+Nkt8sMGH8BHX348lsx90GbZXtWM4Qfs6S2rckl Bptq6moky2b3y3DbTsBxM9OXMCnBw7tneLj4Ziw2MrQLJL4/2H0M3ixjUsO+ba/cwLJuEV CFFmhU6t0R3KDWva6vRuesOm8UuA3LoU2Ah13/hF8yQZqVoOS3Exaxf2HhGC0t/fBxW8Mb obh8Pcaktfavzd0gZacG+hXFOefqXKMsFb+DgqVXU4xhrErANFp7VfQ/0VJ6pi16/MQFCr BB+pc+OPETzZCdiMGOED+yTuSIQMiz2OToF7Em/2r1zB/dbMEauEj67+vWRbjw== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx1.migadu.com: domain of bug-guix-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=bug-guix-bounces@gnu.org X-Migadu-Spam-Score: -4.58 Authentication-Results: aspmx1.migadu.com; dkim=none; dmarc=none; spf=pass (aspmx1.migadu.com: domain of bug-guix-bounces@gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=bug-guix-bounces@gnu.org X-Migadu-Queue-Id: DACBD264AC X-Spam-Score: -4.58 X-Migadu-Scanner: scn0.migadu.com X-TUID: EEzCfG+r0HKc --AnAk6LbvaDZ56IY8 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Mar 30, 2021 at 04:55:13AM -0400, Mark H Weaver wrote: > Hi Efraim, >=20 > Efraim Flashner writes: > > It is the case for inkscape@1.0.2 >=20 > I see now that I'm using an older version, although I would have > preferred the newer one. I refer to the variable name 'inkscape' from > my manifest file, and I expected that to point to the latest stable > version. However, it seems that one must use the 'inkscape-1.0' > variable to get the latest stable version. That's seems suboptimal. >=20 > I wonder if the 'inkscape' variable should be renamed 'inkscape/stable' > (for use in packages such as 'dblatex/stable'), and then 'inkscape' > could be repurposed to point to the latest stable version. Thoughts? In the past we've kept the most up-to-date version as the 'main version' and given the version suffix to the older version(s). Except for when they all get version suffixes and then a separate (define rust rust-1.45) package added. >=20 > > (ins)efraim@3900XT ~$ guix package -A inkscape > > inkscape 1.0.2 out gnu/packages/inkscape.scm:121:2 > > inkscape 0.92.4 out gnu/packages/inkscape.scm:56:2 > > (ins)efraim@3900XT ~$ guix gc --references $(guix build inkscape@1) | g= rep -i imagemagick > > /gnu/store/y4rym48vihmh3jk9qnll0zqfnxzi9n6v-imagemagick-6.9.12-4 > > > > I believe it comes from the glib-or-gtk-wrap phase where it wraps the > > XDG_DATA_DIRS and likely grabs more than it needs. >=20 > Good catch! >=20 > So, for now, we shouldn't use 'imagemagick/stable' in 'inkscape', even > though it's in 'native-inputs'. This shows that we'll have to be very > careful about this, at least until we have better tooling to catch these > problems automatically. >=20 > I think the fundamental problem here is that the build-side code cannot > distinguish between 'inputs' and 'native-inputs' unless you are > cross-compiling. When compiling natively, the 'inputs' keyword argument > passed to the build phases includes the packages listed in the > 'native-inputs' package field, and the 'native-inputs' keyword argument > is not passed at all. I ran into this also with the cargo-build-system. I only wanted to propagate regular inputs, not native inputs. It's probably worth figuring out how to fix some of it on core-updates. > This is why we must write (or native-inputs inputs) in so many places: > because to find the location of a package listed in the 'native-inputs' > package field from within build-side code, you must look in one of two > places depending on whether you're cross-compiling. If you're > cross-compiling you must look in 'native-inputs'. When compiling > natively, that argument doesn't even exist, and you must look in the > 'inputs' keyword argument instead. >=20 > I don't know why it was done this way. It seems to me an error-prone > design, but at this point it would be hard to change it. >=20 > Now we see another disadvantage. The 'glib-or-gtk-wrap' phase should be > iterating over the 'inputs' but not the 'native-inputs'. It's not > obvious how to fix this given the current design. >=20 > What do you think? We can always try to make it better. In the mean time perhaps we can try changing the way some of the wrappers work to only accept regular inputs, or possibly to specify exactly which inputs to iterate over and to use in the wrap phases. > Regards, > Mark --=20 Efraim Flashner =D7=90=D7=A4=D7=A8=D7=99=D7=9D = =D7=A4=D7=9C=D7=A9=D7=A0=D7=A8 GPG key =3D A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351 Confidentiality cannot be guaranteed on emails sent or received unencrypted --AnAk6LbvaDZ56IY8 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEoov0DD5VE3JmLRT3Qarn3Mo9g1EFAmBjE2EACgkQQarn3Mo9 g1FREQ/+OyADors/rDNPeO1H71+JIg325BJh6HC1CS/WBUP/nHBZDALy6mgXrgeq IShrIzyWt4hOsWHwj1imUQECvT9+ctYsnBXZE5OySROFArfXRMZF0OpPrBAagAcB XVY+5/i5SLUJaiLj50GaOIP80/mGGsHiLdGD3JiUC4NDMQMGRh2vRztLYZdXBzpM et99xNXfkRRiwS7v2vwUS+ttcU7W8LPGcscvI9A4w/Nl4gLR9ONYXElzEp9adfMB xeoPx0fiMecBxreoJAl/6ZYipbcnsaddnAIAqjLIwLR/r5/bzWGyV7t3XEQprGw8 6RL58ms58HQQJvQUkAmuKDnmGu8F8nP180P+5J0V7yR+i0WtowBLM4Hes0t+4j4L IZpk9WTLfa9RoV5QfK68wdSIEDibn1kCbfssdReyrecC5PDNsSLU3L3GBSZc80V2 rzCkOt/CJ2uX6R6JKVm2uHvaP2u4HH7zG8Di03KmGwvM66oUSfrQF5hcDaVQi/ft Vq3JzL8FFQpxfs+qGUYEfeEtSsL7lFYuk4MATOqs+7G1EwSufRFXd9HrYEonk4SP DHO5j3Ypk5rK1W6LZ9O7JElsparvbXDVxMHcemQe4FqFuRWMsD2dIvnYethK4o7M pvE20IGjasgCL8QWyaz+CjdYKQKwMZ6YUXa/yMuwNMoG54DEVgY= =DAN4 -----END PGP SIGNATURE----- --AnAk6LbvaDZ56IY8--