Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 412159 - waf-utils.eclass is missing -j on jobs variable
Summary: waf-utils.eclass is missing -j on jobs variable
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords:
: 412167 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-04-16 03:45 UTC by Ben Kohler
Modified: 2012-04-16 14:49 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Kohler gentoo-dev 2012-04-16 03:45:29 UTC
This change breaks ebuilds that inherit waf-utils:
http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/eclass/waf-utils.eclass?r1=1.7&r2=1.8

For my system w/ MAKEOPTS="-j5" as an example, midori-0.4.5 errors thusly:

"/var/tmp/portage/www-client/midori-0.4.4/work/midori-0.4.4/waf" build 5
 error: No such command '5'


I believe it should be local jobs="-j$(makeopts_jobs)" or similar.

Reproducible: Always
Comment 1 Samuli Suominen (RETIRED) gentoo-dev 2012-04-16 07:53:44 UTC
*** Bug 412167 has been marked as a duplicate of this bug. ***
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2012-04-16 07:55:50 UTC
Assigning to gnome@ because it's listed as maintainer in the eclass. 

CCing vapier@ as he committed the change that broke this (just to notify him):

http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/eclass/waf-utils.eclass?r1=1.7&r2=1.8
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2012-04-16 07:58:29 UTC
And should be fixed with this (at least www-client/midori works like it did before vapier's change):

http://sources.gentoo.org/viewvc.cgi/gentoo-x86/eclass/waf-utils.eclass?r1=1.8&r2=1.9

Changed:

local jobs=$(makeopts_jobs)

To:

local jobs="--jobs=$(makeopts_jobs)"

As the old method also passed same:

$ export MAKEOPTS="-j9"
$ jobs=$(echo -j1 ${MAKEOPTS} | sed -r "s/.*(-j\s*|--jobs=)([0-9]+).*/--jobs=\2/" )
$ echo $jobs
--jobs=9

  16 Apr 2012; Samuli Suominen <ssuominen@gentoo.org> waf-utils.eclass:
  Unbreak waf-utils.eclass by restoring --jobs= argument wrt #412159

  15 Apr 2012; Mike Frysinger <vapier@gentoo.org> db.eclass perl-module.eclass
  waf-utils.eclass:
  Use new makeopts_jobs helper from eutils.eclass.
Comment 4 Aidan Taniane 2012-04-16 08:37:40 UTC
This has broken pycairo for me.  Even if I unset -j in MAKEOPTS it still implies -j1 so it tries to call a '1' function, which doesn't exist, of course.
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2012-04-16 09:21:46 UTC
(In reply to comment #4)
> This has broken pycairo for me.  Even if I unset -j in MAKEOPTS it still
> implies -j1 so it tries to call a '1' function, which doesn't exist, of
> course.

What is this "This" you are talking about? The problem is already fixed in tree as mentioned in Comment #3. No point in leaving comments from outdated tree.
Comment 6 Aidan Taniane 2012-04-16 09:27:55 UTC
(In reply to comment #5)
> What is this "This" you are talking about? The problem is already fixed in
> tree as mentioned in Comment #3. No point in leaving comments from outdated
> tree.

I synced before I wrote my comment and still didn't work at that point.  Don't be so obnoxious.  How about "Try syncing and emerging pycairo now"?  You might elicit more enthusiasm from users instead of disdain.
Comment 7 SpanKY gentoo-dev 2012-04-16 14:49:05 UTC
(In reply to comment #3)

sorry about that ... a bunch of files had different sed's and it was hard to pick out the subtleties of them.  i tried a few, but i guess not the waf-utils one.