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@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@xxxxxxxxxxxxxxxxx |seg@xxxxxxxxxx Flag| |fedora-review? ------- Additional Comments From seg@xxxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review