emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#51948: closed ([PATCH] [core-updates-frozen] Wrap meson with python


From: GNU bug Tracking System
Subject: bug#51948: closed ([PATCH] [core-updates-frozen] Wrap meson with python code)
Date: Mon, 22 Nov 2021 21:45:02 +0000

Your message dated Mon, 22 Nov 2021 22:44:21 +0100
with message-id <87czmrn9vu.fsf_-_@gnu.org>
and subject line Re: bug#51948: [PATCH] [core-updates-frozen] Wrap meson with 
python code
has caused the debbugs.gnu.org bug report #51948,
regarding [PATCH] [core-updates-frozen] Wrap meson with python code
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
51948: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=51948
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- Subject: [PATCH] Meson needs proper wrapping Date: Thu, 18 Nov 2021 15:23:48 +0100
Dear guix,

Meson needs to be wrapped, but the wrapping must not execute a new
process, if I understand correctly. I could use wrap-script, but I think
it’s more pythonic to set the path through sys.path rather than setting
an environment variable.

Now, this will create a world rebuild… What do you think?

Vivien

Attachment: 0001-gnu-meson-Extend-the-python-path-in-the-installed-pr.patch
Description: Properly wrap meson

Attachment: signature.asc
Description: PGP signature


--- End Message ---
--- Begin Message --- Subject: Re: bug#51948: [PATCH] [core-updates-frozen] Wrap meson with python code Date: Mon, 22 Nov 2021 22:44:21 +0100 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)
Hi,

Vivien Kraus <vivien@planete-kraus.eu> skribis:

> From 3f77d8d5c652725411a21434443cbf2a2d135575 Mon Sep 17 00:00:00 2001
> From: Vivien Kraus <vivien@planete-kraus.eu>
> Date: Thu, 18 Nov 2021 15:17:52 +0100
> Subject: [PATCH] gnu: meson: Extend the python path in the installed program.
>
> * gnu/packages/build-tools.scm (meson-wrapped): New variable.

Applied, thanks!

As discussed on IRC, I followed up with a patch that moves Python from
‘propagated-inputs’ (!) to ‘inputs’ in ‘meson-wrapped’.  Most likely
Python was propagated just for the sake of setting GUIX_PYTHONPATH, but
it’s bad style as it forces it into user profiles.

Ludo’.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]