From: "Ludovic Courtès" <ludo@gnu.org>
To: Ricardo Wurmus <rekado@elephly.net>
Cc: 37123@debbugs.gnu.org
Subject: bug#37123: gnome-shell: LD_LIBRARY_PATH setting propagates to entire session
Date: Wed, 13 Nov 2019 23:02:52 +0100 [thread overview]
Message-ID: <87pnhvmo37.fsf@gnu.org> (raw)
In-Reply-To: <87sgpvlj8t.fsf@elephly.net> (Ricardo Wurmus's message of "Tue, 20 Aug 2019 21:38:58 +0200")
Hello,
Ricardo Wurmus <rekado@elephly.net> skribis:
>> Since commit 2b0c755d195c79bfc95cdbe802e1e2dea1adb7a2 in August 2018,
>> our 'gnome-shell' executable has been wrapped by a script that sets
>> LD_LIBRARY_PATH.
>>
>> One consequence of this, which I just noticed, is that if 'gnome-shell'
>> is based on 'core-updates' (or in my case, 'core-updates-next'), many
>> programs based on 'master' will fail to run within the resulting GNOME
>> session.
> […]
>> I was unable to easily find an existing bug report tracking this issue,
>> so I created this one.
>
> There was no bug report about this, so thanks for reporting it. I once
> brought this issue up on the mailing list here:
>
> https://lists.gnu.org/archive/html/guix-devel/2019-05/msg00372.html
Looking at this bit in the ‘gnome-shell’ definition:
--8<---------------cut here---------------start------------->8---
(wrap-program (string-append out "/bin/gnome-shell")
`("GI_TYPELIB_PATH" ":" prefix (,gi-typelib-path))
;; FIXME: gnome-shell loads these libraries with unqualified
;; names only, so they need to be on LD_LIBRARY_PATH. The
;; alternative might be to patch gnome-shell.
`("LD_LIBRARY_PATH" ":" prefix
,(map (lambda (pkg)
(string-append (assoc-ref inputs pkg) "/lib"))
'("gdk-pixbuf"
"gnome-bluetooth" "librsvg" "libgweather"))))
--8<---------------cut here---------------end--------------->8---
I checked in Gjs etc. how those imports (e.g., “imports.gi.Rsvg” in
Javascript) are turned into a dlopen. AIUI, this is done by
gobject-introspection based on info found in .gir files.
In Guix, .gir files contain absolute file names of share libraries. At
run-time, ‘gobject-introspection-absolute-shlib-path.patch’ ensures that
dlopen is passed absolute file names.
So, IIUC, “imports.gi.Rsvg” should lead to dlopen by absolute file name,
in which case setting LD_LIBRARY_PATH is useless.
However, does anyone know about we can test whether removing the
LD_LIBRARY_PATH wrapping above breaks something?
Thanks,
Ludo’.
next prev parent reply other threads:[~2019-11-13 22:04 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-20 19:21 bug#37123: gnome-shell: LD_LIBRARY_PATH setting propagates to entire session Mark H Weaver
2019-08-20 19:38 ` Ricardo Wurmus
2019-11-13 22:02 ` Ludovic Courtès [this message]
2021-12-02 9:54 ` Ricardo Wurmus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87pnhvmo37.fsf@gnu.org \
--to=ludo@gnu.org \
--cc=37123@debbugs.gnu.org \
--cc=rekado@elephly.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).