Bootstrap broken

Phil Edwards pedwards@disaster.jaj.com
Thu Dec 6 11:29:00 GMT 2001


On Wed, Dec 05, 2001 at 06:29:01PM -0500, Richard Kenner wrote:
>     /cvs/gcc/gcc/optabs.c: In function `init_optabs':
> 
> Sory.  I left optabs.c off of my commit line.

These sorts of errors are really annoying to those of us who are told to
test and commit only the exact same patch that was posted and approved
(and no less).

You might consider automating more of this with a script.  For example, the
output of 'cvs diff' or 'cvs -n update' can be used to assemble the list
of filenames to pass to 'commit'.  Even smarter is to build the ChangeLog
entry, then extract the filenames to commit out of that (this may not be
feasible with multiple ChangeLogs being written to).

Also useful are multiple trees, say:

    1)  Do work in tree X.
    2)  Generate a diff out of tree X.
    3)  Post diff, get approval.
    4)  Apply patch to otherwise-clean tree Y.
    5)  Commit out of tree Y.

Right now I have 8 trees, mostly for separate projects and different
branches.  (It helps that all but one are checked out from a local mirror
of the CVS repo; this makes things like "cvs diff" work in reasonable
amounts of time rather than chunking across the Internet.)

Just some friendly suggestions.


Phil

-- 
If ye love wealth greater than liberty, the tranquility of servitude greater
than the animating contest for freedom, go home and leave us in peace.  We seek
not your counsel, nor your arms.  Crouch down and lick the hand that feeds you;
and may posterity forget that ye were our countrymen.            - Samuel Adams



More information about the Gcc-patches mailing list