From debbugs-submit-bounces@debbugs.gnu.org Mon Jul 08 12:30:12 2019 Received: (at 36535) by debbugs.gnu.org; 8 Jul 2019 16:30:12 +0000 Received: from localhost ([127.0.0.1]:59382 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hkWWl-0002ee-0U for submit@debbugs.gnu.org; Mon, 08 Jul 2019 12:30:12 -0400 Received: from out3-smtp.messagingengine.com ([66.111.4.27]:33217) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hkWWj-0002db-1E for 36535@debbugs.gnu.org; Mon, 08 Jul 2019 12:30:05 -0400 Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.nyi.internal (Postfix) with ESMTP id 1552D21ADD; Mon, 8 Jul 2019 12:29:59 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute5.internal (MEProxy); Mon, 08 Jul 2019 12:29:59 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastmail.com; h= from:to:cc:subject:in-reply-to:references:date:message-id :mime-version:content-type; s=fm3; bh=qo6rLz2asrAEuXtZDyB0SvnNNj strWGsBQd0RJ8hZO0=; b=aNz7t7Ux1/I7vbGRpFWFv0gIOYE7P6xA1BfXoPPEyc HYTpf7iSRZrtdUMSjDdV+auYctzeDeAZf3XtUoQa3QYdVrLpWeAeWhPQFvnIjnp5 Y5tU8tHC0Hwx0VuzXKo5egO7pG0woFeJiTmOtMvI3gjG4wwO4RETo0INCflkJ+iD ZkwgoJ2OTpoujLJeqYxovQ9GIz5+BtsEGYYXJ8Rh2+siHKeTh+irWxZsfi5NW1+L uHcjwpDo4b1ihaEsrAbj+QI8q9y7Cy0kv1/L0KOJJ6jkIh4JU36CooH8zHoMWWmp bRtSSRUl1kn357d96GFsJgl9MAV37QTLnNiUHuNMFl6Q== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=qo6rLz 2asrAEuXtZDyB0SvnNNjstrWGsBQd0RJ8hZO0=; b=gl4lVf0foV55kQKiKchSg1 8SzOJF25XeHNr2oFuZxawt6cBw9luhqDTJ76suslz6wu8At3FLyCCsocxdhcPm8w zqt34rQEkvtC/7D/VVpEM4/gej20sja4O+j0s/DfHRRMGee1D0OPmcjTA1nhS1E0 DuSvPx0iiSoUu3RkGi+7RfVXsx0EOzyvngJDiCeAh7E1FHD0N0UJMhb8g1PoZAjR lPeUNkS9zmjBETU6NpWKk8dmhM5Ao5GZgKL0xwmfXKgRsfdBhbqcfKUX6x10hZ2a NtW1Ac9HTu+g9Bu0X94Y78PxHccUtX/Fwt0WmFbsgw0FQplIwEBjZexS34Tv7ksw == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduvddrgedtgddutdegucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucenucfjughrpefhvffujghffgffkfggtgesghdtre ertdertdenucfhrhhomhepofgrrhhiuhhsuceurghkkhgvuceomhgsrghkkhgvsehfrghs thhmrghilhdrtghomheqnecukfhppeeivddrudeirddvvdeirddugedtnecurfgrrhgrmh epmhgrihhlfhhrohhmpehmsggrkhhkvgesfhgrshhtmhgrihhlrdgtohhmnecuvehluhhs thgvrhfuihiivgeptd X-ME-Proxy: Received: from localhost (140.226.16.62.customer.cdi.no [62.16.226.140]) by mail.messagingengine.com (Postfix) with ESMTPA id 4E24280065; Mon, 8 Jul 2019 12:29:58 -0400 (EDT) From: Marius Bakke To: Christopher Baines Subject: Re: bug#36535: [PATCH] gnu: gobject-introspection: Update absolute-shlib-path.patch. In-Reply-To: <87bly4a566.fsf@cbaines.net> References: <20190707104803.23662-1-mail@cbaines.net> <87d0il9deh.fsf@cbaines.net> <875zocr4hl.fsf@devup.no> <87bly4a566.fsf@cbaines.net> User-Agent: Notmuch/0.29.1 (https://notmuchmail.org) Emacs/26.2 (x86_64-pc-linux-gnu) Date: Mon, 08 Jul 2019 18:29:56 +0200 Message-ID: <87zhlopjzv.fsf@devup.no> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" X-Spam-Score: -0.7 (/) X-Debbugs-Envelope-To: 36535 Cc: 36535@debbugs.gnu.org X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -1.0 (-) --=-=-= Content-Type: text/plain Christopher Baines writes: > Marius Bakke writes: > >> Hi Chris, >> >> Christopher Baines writes: >> >>> Christopher Baines writes: >>> >>>> Incorporate some changes from nixpkgs to the gobject-introspection package >>>> patches. This is motivated by looking at issues with libsoup and lollypop. >>>> This changes means that the share/gir-1.0/Soup-2.4.gir file within libsoup >>>> references libsoup-2.4.so.1 with an absolute filename, whereas previously, the >>>> filename wasn't absolute. >>>> >>>> * gnu/packages/patches/gobject-introspection-absolute-shlib-path.patch: >>>> Incorporate changes from nixpkgs. >>>> --- >>>> ...ct-introspection-absolute-shlib-path.patch | 141 +++++++++++++++++- >>>> 1 file changed, 137 insertions(+), 4 deletions(-) >>>> >>> >>> I've pushed this as [1] to core-updates now, as I wanted to get it in >>> before the freeze. >> >> Thank you for addressing this. IIUC previously lollypop failed to >> retain a reference to libsoup-2.4.so.1, whereas with this patch it does? > > Not quite... I think lollypop was reading the typelib in libsoup, but > the shared library was just referenced by filename, not the absolute > filename, and I think this was causing issues when trying to use libsoup > from lollypop. I see, thanks for explaining. In Guix, we usually resolve these situations by native-search-paths, do you know if gobject-introspection supports looking up the 'share/gir-1.0' directory from an environment variable (similar to how GI_TYPELIB_PATH works today)? However... > > On master: > > grep shared-library /gnu/store/bafaiiblr2vmmf1zvidkw1137ndqnqg2-libsoup-2.66.2/share/gir-1.0/Soup-2.4.gir > shared-library="libsoup-2.4.so.1" > > On core-updates: > > grep shared-library /gnu/store/b1ykh6xj11v7zav4r68v8qflk31cnddm-libsoup-2.66.2/share/gir-1.0/Soup-2.4.gir > shared-library="/gnu/store/b1ykh6xj11v7zav4r68v8qflk31cnddm-libsoup-2.66.2/lib/libsoup-2.4.so.1" ...this is even better, so I am mostly just curious :-) >> A few comments about the patch: >> >>> diff --git a/gnu/packages/patches/gobject-introspection-absolute-shlib-path.patch b/gnu/packages/patches/gobject-introspection-absolute-shlib-path.patch >>> index d00cc5a420..3c0bb1c6cf 100644 >>> --- a/gnu/packages/patches/gobject-introspection-absolute-shlib-path.patch >>> +++ b/gnu/packages/patches/gobject-introspection-absolute-shlib-path.patch >>> @@ -2,10 +2,131 @@ >>> # add the full path. >>> # >>> # This patch was provided by Luca Bruno for >>> -# 'gobject-introspection' 1.40.0 in Nix. >>> ---- ./giscanner/utils.py.orig 2014-08-14 22:05:05.055334080 +0200 >>> -+++ ./giscanner/utils.py 2014-08-14 22:05:24.687497334 +0200 >>> -@@ -110,17 +110,11 @@ >>> +# 'gobject-introspection' 1.40.0 in Nix. >>> +# >>> +# It has since been updated to work with newer versions of >>> +# gobject-introspection. >>> +--- a/giscanner/scannermain.py >>> ++++ b/giscanner/scannermain.py >>> +@@ -95,6 +95,39 @@ def get_windows_option_group(parser): >>> + return group >>> + >>> + >>> ++def _get_default_fallback_libpath(): >>> ++ # Newer multiple-output-optimized stdenv has an environment variable >>> ++ # $outputLib which in turn specifies another variable which then is used as >>> ++ # the destination for the library contents (${!outputLib}/lib). >>> ++ store_path = os.environ.get(os.environ.get("outputLib")) if "outputLib" in os.environ else None >>> ++ if store_path is None: >>> ++ outputs = os.environ.get("outputs", "out").split() >> >> gnu-build-system does not currently export an "outputs" variable. >> Perhaps it should? > > Ah, I didn't realise this part of the patch was as Nix specific as it > is... > > At least for the change I was trying to affect, this seems to be > probably redundant, or somehow doing the job. Maybe this part of the > patch relating to the fallback_libpath should be removed. I'd keep the "$outputs" logic, it sounds like a useful and easy change to do in gnu-build-system, although maybe not for this 'core-updates' round. We can use it in package recipes for fun and profit meanwhile. However I doubt we'll ever use "outputLib", so it would be good to remove that. If you are updating the patch, could you also add a link to the upstream patch, as well as one to this discussion? Thank you! --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCgAdFiEEu7At3yzq9qgNHeZDoqBt8qM6VPoFAl0jb4QACgkQoqBt8qM6 VPqcDAgAzh5OBm920W7x3miCb3EamzZSzWij+OMp0q+3vl0UU5bYaOQeUNlOJiZO REkwR9cVwVobqiUAgBi5YOOsNa5tk6g/brC/1Pr4Z8QBtewmy6BR0hw5q40ywCaw pf2Y4h2JR5rWBbMmoltN5ltJQv4kbbBAGZe0DQUDFNNhDFctTS/A9DXJB72bLPot ag7xy6wYow2xUnYz7SAmfU1g835Mm9VhsaMHrvBL5/+nejHmt6dTarr+pszRqIRu oEFznl+8MoK3z1W6R6GiYLezyR6RkMwOp1+H8gEVwJN1AWgiWgobKAbH4BjcWFxk MFEZ8Mai3Pv7nr/BfxlzKWFOq+U3GQ== =N53A -----END PGP SIGNATURE----- --=-=-=--