From: Maxim Cournoyer <maxim.cournoyer@gmail.com>
To: John Kehayias <john.kehayias@protonmail.com>
Cc: "Wojtek Kosior" <koszko@koszko.org>, 宋文武 <iyzsong@envs.net>,
edk@beaver-labs.com, guix-devel@gnu.org
Subject: Re: Guix's python has pip's user dir in its loadpath
Date: Thu, 06 Jul 2023 11:35:15 -0400 [thread overview]
Message-ID: <87o7kpyrws.fsf@gmail.com> (raw)
In-Reply-To: <87cz16kspd.fsf@protonmail.com> (John Kehayias's message of "Wed, 05 Jul 2023 20:29:50 +0000")
Hi John,
John Kehayias <john.kehayias@protonmail.com> writes:
> Hi,
>
> On Sat, Jul 01, 2023 at 11:57 AM, Maxim Cournoyer wrote:
>
>> Hi,
>>
>> Wojtek Kosior <koszko@koszko.org> writes:
>>
>>> The precedence of local, pip-installed Python libraries over Guix ones
>>> has already been a source of bugs. And these can be hard to diagnose.
>>
>>> I imagine an optimal solution would be to configure this behavior on
>>> per-package basis. The vast majority of applications does not need to
>>> load local libraries. There are just a few exceptions like
>>> `python-virtualenv`.
>>>
>>> Once I did write a package definition that deliberately disabled user
>>> site dir package loading. I used code similar to what's below.
>>>
>>>> (modify-phases %standard-phases
>>>> (add-after 'wrap 'prevent-local-package-interference
>>>> (lambda* (#:key outputs #:allow-other-keys)
>>>> (substitute* (string-append (assoc-ref outputs "out")
>>>> "/bin/<program-name>")
>>>> (("^#!/.*$" shabang)
>>>> (string-append shabang
>>>> "export PYTHONNOUSERSITE=1\n"))))))
>>
>> That is indeed a simple thing we could do to harden Python binaries from
>> picking up user pip-installed dependencies potentially causing
>> problems. I would welcome such a patch.
>>
>
> Perhaps, but if this is expected (and known) upstream behavior, I'm
> wary of deviating from these expectations. This general area does seem
> tricky and no simple best answer I guess.
While it's true that it's an intended upstream behavior, I think in the
context of Guix users packages to be self-contained or in some case be
able to load Guix-installed plugins or extensions, but here it seems
reasonable that a Guix-packaged Python binary prefers loading Python
libraries from Guix rather that from the Python user site.
>>> Of course, it makes no sense to add such snippet to all definitions.
>>> Instead, we could modify python-build-system to allow doing a similar
>>> thing based on a flag passed in package's `(arguments)`.
>>
>> I think it need not be made configurable but just applied
>> indiscriminately to the wrap phase used in the python-build-system.
>
> And this is part of the same question then, we should try to be
> consistent, yes. I don't see a clear right path, but I haven't thought
> much about this area. I think it comes down to a current
> issue/limitation/quirk of Python from upstream and packaging for our
> distro puts us in between what comes from them and how to take care of
> our users.
>
> But we'll be rebuilding the Python world anyway, so now is a chance to
> try out some changes like that, though maybe it is a bit much with
> what we are trying already. See <https://issues.guix.gnu.org/63139>
It's a simple change, I guess we could try it at the same time, if
someone volunteers to do it!
--
Thanks,
Maxim
next prev parent reply other threads:[~2023-07-06 15:36 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-14 6:42 Guix's python has pip's user dir in its loadpath edk
2023-06-29 10:22 ` 宋文武
2023-07-01 3:13 ` Maxim Cournoyer
2023-07-01 11:32 ` Wojtek Kosior via Development of GNU Guix and the GNU System distribution.
2023-07-01 15:57 ` Maxim Cournoyer
2023-07-05 20:29 ` John Kehayias
2023-07-06 15:35 ` Maxim Cournoyer [this message]
2023-07-06 21:28 ` Wojtek Kosior via Development of GNU Guix and the GNU System distribution.
2023-07-07 13:32 ` Maxim Cournoyer
2023-07-07 14:44 ` Wojtek Kosior via Development of GNU Guix and the GNU System distribution.
2023-07-08 5:08 ` Maxim Cournoyer
2023-07-11 18:21 ` Wojtek Kosior via Development of GNU Guix and the GNU System distribution.
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=87o7kpyrws.fsf@gmail.com \
--to=maxim.cournoyer@gmail.com \
--cc=edk@beaver-labs.com \
--cc=guix-devel@gnu.org \
--cc=iyzsong@envs.net \
--cc=john.kehayias@protonmail.com \
--cc=koszko@koszko.org \
/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).