Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 702646 - sys-devel/distcc - add IUSE=pump, off by default
Summary: sys-devel/distcc - add IUSE=pump, off by default
Status: UNCONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Cluster Team
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2019-12-13 06:53 UTC by Enne Eziarc
Modified: 2022-05-31 07:02 UTC (History)
1 user (show)

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


Attachments
Add USE=pump (0001-distcc-make-pump-optional.patch,2.82 KB, patch)
2019-12-13 06:53 UTC, Enne Eziarc
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Enne Eziarc 2019-12-13 06:53:03 UTC
Created attachment 599320 [details, diff]
Add USE=pump

With distcc-pump support being removed from portage itself (bug #702146) it would be good to not have to install that part at all, as it's currently the only thing on all my systems pulling in python 3.6.

Attached is an ebuild patch that implements this. The configure script has a --disable-pump-mode flag but unfortunately it doesn't do anything.
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2019-12-13 10:59:47 UTC
(In reply to Anthony Parsons from comment #0)
> Created attachment 599320 [details, diff] [details, diff]
> Add USE=pump
> 
> With distcc-pump support being removed from portage itself (bug #702146) it
> would be good to not have to install that part at all, as it's currently the
> only thing on all my systems pulling in python 3.6.
> 
> Attached is an ebuild patch that implements this. The configure script has a
> --disable-pump-mode flag but unfortunately it doesn't do anything.

Thanks.  I'm going to review and merge it later today.

While at it, could you report it upstream that the option doesn't do anything?  ;-)
Comment 2 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2019-12-13 20:50:24 UTC
I'm not sure if it's really worth.  We still need Python because of other Python scripts.  If we need Python anyway, then having a flag not to install a few Python modules doesn't seem very useful.
Comment 3 Enne Eziarc 2019-12-13 23:08:37 UTC
(In reply to Michał Górny from comment #2)
> I'm not sure if it's really worth.  We still need Python because of other
> Python scripts.  If we need Python anyway, then having a flag not to install
> a few Python modules doesn't seem very useful.

Yeah, I don't have strong feelings about it either way and the work required feels kind of hacky. Bumping PYTHON_TARGETS up to 3.7 would be an easier way to achieve the same end result - upstream already seems to support that (but not 3.8). If that sounds better, go ahead and reject this.
Comment 4 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2019-12-14 09:00:29 UTC
Heh.  Actually, the main reason I didn't add new impls yet is because I don't use pump and I haven't figured out yet how to sanely test it.