Re: [gentoo-pms] [PATCH] pkg-mgr-commands: Replace MAKEOPTS conf var with explicit description

["Michał Górny" <mgorny@gentoo.org>]

[-- Attachment #1: Type: text/plain, Size: 1548 bytes --]

On Sat, 14 May 2016 00:33:20 +0200
Ulrich Mueller <ulm@gentoo.org> wrote:

> >>>>> On Sat, 14 May 2016, Michał Górny wrote:  
> 
> > Replace the mentioning of MAKEOPTS configuration variable with
> > generic description of 'make options'. This is in line with other
> > parts of the specification where we avoid referencing package
> > manager configuration explicitly. Furthermore, MAKEOPTS was only
> > mentioned this one time with no explanatory description and no
> > guarantee of being exported to the ebuild environment, so it is
> > better to avoid making developers think it is reliable.  
> 
> The council has rejected the EJOBS feature (see bug 273101) twice,
> namely for EAPI 5 and EAPI 6. In the discussion about EAPI 6 the
> main argument was that similar functionality already exists with
> the makeopts_jobs() and makeopts_loadavg() functions provided by
> multiprocessing.eclass (see 20140617 council log). Both functions
> rely on the MAKEOPTS variable.
> 
> So, IMHO removal of this variable cannot be done in a simple editorial
> change but needs to be discussed in the -dev mailing list. We would
> also need a replacement for the two above-mentioned functions (which
> presumably will require an EAPI bump).

Well, the alternative would be to list it properly in ebuild-exported
variables section. Considering what you pointed out, this may be
a better idea indeed. After all, we re-use it for other build systems
as well.

-- 
Best regards,
Michał Górny
<http://dev.gentoo.org/~mgorny/>

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 949 bytes --]