1 |
On Thu, Dec 17, 2020 at 10:58:23PM +0100, Michał Górny wrote: |
2 |
> On Thu, 2020-12-17 at 16:50 -0500, Mike Gilbert wrote: |
3 |
> > On Thu, Dec 17, 2020 at 4:44 PM Michał Górny <mgorny@g.o> |
4 |
> > wrote: |
5 |
> > > |
6 |
> > > On Thu, 2020-12-17 at 16:30 -0500, Mike Gilbert wrote: |
7 |
> > > > Closes: https://bugs.gentoo.org/759433 |
8 |
> > > > Signed-off-by: Mike Gilbert <floppym@g.o> |
9 |
> > > > --- |
10 |
> > > > eclass/meson.eclass | 3 ++- |
11 |
> > > > 1 file changed, 2 insertions(+), 1 deletion(-) |
12 |
> > > > |
13 |
> > > > diff --git a/eclass/meson.eclass b/eclass/meson.eclass |
14 |
> > > > index 21338280df33..6296f1dd26e5 100644 |
15 |
> > > > --- a/eclass/meson.eclass |
16 |
> > > > +++ b/eclass/meson.eclass |
17 |
> > > > @@ -126,7 +126,8 @@ EOF |
18 |
> > > > # '--unicode-16=𐐷', '--unicode-32=𐤅'] |
19 |
> > > > # |
20 |
> > > > _meson_env_array() { |
21 |
> > > > - python -c "${__MESON_ARRAY_PARSER}" "$@" |
22 |
> > > > + local python="$(eselect python show)" |
23 |
> > > > + ${python} -c "${__MESON_ARRAY_PARSER}" "$@" |
24 |
> > > > } |
25 |
> > > > |
26 |
> > > > # @FUNCTION: _meson_get_machine_info |
27 |
> > > |
28 |
> > > You're missing a BDEPEND on app-eselect/eselect-python. |
29 |
> > > |
30 |
> > > Also, I really don't like these workarounds. It takes a lot of |
31 |
> > > effort |
32 |
> > > to figure out how to break stuff, so people stop doing awful |
33 |
> > > things. |
34 |
> > > It's disrespectful to my time when you invent new hacks. Now I'll |
35 |
> > > have |
36 |
> > > to figure out how to change eselect-python to break it. |
37 |
> > |
38 |
> > Why is this such an awful thing to do? |
39 |
> > |
40 |
> > The code should be able to execute with any version of python |
41 |
> > currently supported by Gentoo. |
42 |
> > |
43 |
> > Please don't assume that I'm trying to avoid a proper solution here. |
44 |
> > Please suggest a better alternative if you have one. |
45 |
> |
46 |
> I actually liked installing the script to the system. |
47 |
|
48 |
If we are going to install it to the system, that should not be done by |
49 |
dev-util/meson; it is a gentoo-specific script, so we should probably |
50 |
create a separate package for it. Also, it is debatable whether that |
51 |
script should be installed in a directory that is on the path. |
52 |
|
53 |
I have major concerns about the native-symlinks use flag for python-exec. |
54 |
It looks like turning this flag off would result in /usr/bin/python not being |
55 |
installed which will cause massive breakage. This is similar to removing |
56 |
/bin/sh., so I am strongly against the idea of this use flag unless |
57 |
upstream python is recommending it. If they are not installing |
58 |
/usr/bin/python in their native builds any longer, I'll be quiet. |
59 |
Otherwise, imo this is a really bad idea for a use flag. |
60 |
|
61 |
William |