[Bug 174883] Review Request: distcc -- A free distributed C/C++ compiler system

bugzilla at redhat.com bugzilla at redhat.com
Sun Mar 18 09:54:05 UTC 2007


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: distcc -- A free distributed C/C++ compiler system


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=174883


seg at haxxed.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|nobody at fedoraproject.org    |seg at haxxed.com
               Flag|                            |fedora-review?




------- Additional Comments From seg at haxxed.com  2007-03-18 05:54 EST -------
1) Is it really necessary to split off the init scripts and xinetd stuff into
sub-packages? I see no reason for it.

2) distcc.sh is incredibly unreadable. And broken, it will fail on x86_64
because ccache is in /usr/lib64/ccache, which is also broken. Perhaps libexec is
the place both ccache and distcc should be playing their gcc hijacking tricks.

How about something more like this:


3) What does release_func accomplish? It obfusticates the fact that the release
tag violates policy. It should be a single integer followed by the release tag,
unless you're bumping an old tree in which case you can add a number after the
release tag.



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list