FS#24892 - [makepkg] - add support for pump (part of distcc)

Attached to Project: Pacman
Opened by John (graysky) - Sunday, 26 June 2011, 08:56 GMT
Last edited by Dan McGee (toofishes) - Tuesday, 30 August 2011, 19:05 GMT
Task Type Feature Request
Category makepkg
Status Closed
Assigned To No-one
Architecture All
Severity Low
Priority Normal
Reported Version 3.5.3
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Add support for distcc's pump script into makepkg. Since version 3, pump sends header files to the remote machines, so that preprocessing is distributed too. In many cases, this leads to significant speed increases.

Some benchmarks (quite amazing): http://distcc.googlecode.com/svn/trunk/doc/web/benchmark.html
This task depends upon

Closed by  Dan McGee (toofishes)
Tuesday, 30 August 2011, 19:05 GMT
Reason for closing:  Won't implement
Additional comments about closing:  See comments, KISS and do outside of makepkg.
Comment by Dan McGee (toofishes) - Sunday, 26 June 2011, 16:26 GMT
Patches welcome.
Comment by John (graysky) - Sunday, 26 June 2011, 17:45 GMT
I would if I could... /usr/bin/makepkg is a pretty sophisticated piece of bash code to me :)
Comment by Jakob Matthes (jakobm) - Tuesday, 30 August 2011, 18:46 GMT
It actually comes down to some simple changes:

Run eval $(/usr/bin/pump --startup) before the build function
Run /usr/bin/pump --shutdown after the build function

I tested this and have not encountered any problems so far.

Needs clarification:
- Assume pump mode is standard or introduce DISTCC_PUMP=1 variable (my preference) or even add an additional buildenv option?
(Pump mode has some extensive requirements, builds can fail or produce broken results if not met)
- Should it be implemented at all, given that one can simply set the distcc hosts list and run eval $(pump --startup) before invoking makepkg?

Please share your thoughts.
Comment by Dan McGee (toofishes) - Tuesday, 30 August 2011, 18:51 GMT
Seems like it makes more sense for someone that wants pump mode to just start it up on their own (probably as a daemon anyway?).

Is /usr/bin/pump even in the arch package? Not seeing it, looks like that needs to get enabled.
Comment by Jakob Matthes (jakobm) - Tuesday, 30 August 2011, 19:03 GMT
I filed a bug earlier today regarding the missing pump script in the distcc package.

One can even run "pump makepkg [...]". The only obstacle would be exporting DISTCC_HOSTS which is normally done by makepkg (pump only starts with DISTCC_HOSTS set), although this could be easily achieved with sourcing makepkg.conf.
Given these circumstances I tend towards not implementing.

Loading...