Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 588216 - app-admin/needrestart-9999: QA Notice: make jobserver unavailable: using -j1.
Summary: app-admin/needrestart-9999: QA Notice: make jobserver unavailable: using -j1.
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal QA (vote)
Assignee: Craig Andrews
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-07 11:32 UTC by Coacher
Modified: 2017-01-01 13:06 UTC (History)
1 user (show)

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


Attachments
emerge --info needrestart (info,7.16 KB, text/plain)
2016-07-07 11:32 UTC, Coacher
Details
build.log (build.log,17.31 KB, text/plain)
2016-07-07 11:33 UTC, Coacher
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Coacher 2016-07-07 11:32:13 UTC
Created attachment 439942 [details]
emerge --info needrestart

Hello.

QA notice from portage:

* QA Notice: make jobserver unavailable:
*
* make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.

Please fix.
Comment 1 Coacher 2016-07-07 11:33:06 UTC
Created attachment 439944 [details]
build.log
Comment 2 Craig Andrews gentoo-dev 2016-07-07 18:38:18 UTC
Coacher - the build log says:
---
>>> Install needrestart-9999 into /var/tmp/portage/app-admin/needrestart-9999/image/ category app-admin
make --jobs=5 DESTDIR=/var/tmp/portage/app-admin/needrestart-9999/image/ install 
make -C po/needrestart
make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
---

But there is no po directory or po files in upstream's git repository. Also, when I run the build, I do not get this error, and I also do not end up with a po directory in my work directory.

Do you know you got these po files so I can reproduce the problem and ultimately trace down the fix for this QA notice?

Thanks!
Comment 3 Craig Andrews gentoo-dev 2016-07-07 19:05:50 UTC
nevermind, I was looking at the wrong source code. Let's pretend that embarrassing confusion didn't occur.

I've submitted a PR upstream: https://github.com/liske/needrestart/pull/34

Also, submitted a PR to Gentoo: https://github.com/gentoo/gentoo/pull/1843

Note that since I'm hoping upstream merges the PR to fix the issue soon, I'm not using the patch in the -9999 ebuild. So this issue will only be fixed for version 2.8.
Comment 4 Patrice Clement gentoo-dev 2016-07-08 20:55:45 UTC
commit 4557b08753a97b359688bb5231ed80753dbbbc32 (HEAD -> master)
Author:     Craig Andrews <candrews@integralblue.com>
AuthorDate: Thu Jul 7 15:51:52 2016 -0400
Commit:     Patrice Clement <monsieurp@gentoo.org>
CommitDate: Fri Jul 8 18:52:04 2016 +0000

app-admin/needrestart-2.8: Fix QA Notice: make jobserver unavailable: using -j1.

Gentoo-Bug: https://bugs.gentoo.org/588216
Closes: https://github.com/gentoo/gentoo/pull/1843

Signed-off-by: Patrice Clement <monsieurp@gentoo.org>

.../needrestart/files/needrestart-parallelmake.patch    | 17 +++++++++++++++++
app-admin/needrestart/needrestart-2.8.ebuild            |  4 ++++
2 files changed, 21 insertions(+)
create mode 100644 app-admin/needrestart/files/needrestart-parallelmake.patch