Re: PATCH: Remove generated YACC stuff

[Mark Mitchell <mark at codesourcery dot com>]

>>>>> "Jeffrey" == Jeffrey A Law <law@cygnus.com> writes:

    Jeffrey> Mark -- you shouldn't have done this without first
    Jeffrey> coordinating with me to make sure the snapshot script was
    Jeffrey> changed beforehand and with Geoff Keating to ensure his
    Jeffrey> tinderbox stuff was ready.

There are lots of systems out there depending on lots of aspects of
GCC; sometimes they change, and sometimes things break. :-(

I lost hours yesterday tracking down a skew problem here -- even
knowing that was what I was looking for.  I checked that the checked
out files were built with the same version of bison -- but the
Makefiles randomly rebuilt one or the other of the files depending on
totally random NFS/tar/cvs interactions.  If we prevent that
happenning a few times in the next week, that will outweigh the minor
costs of a little hiccup in snapshots (which will work fine if you
have bison installed) and the tinderbox (which will work fine as soon
as CVS is put on that box).

Still, co-ordination would not have been a bad thing, I agree.

    Jeffrey> I'll try to get a chance to fix the snapshot script
    Jeffrey> before next week, but I can't guarantee that.  jeff

It would be great if you could get the snapshot script checked in.
Someone else could then fix it.  Even if the snapshot script is not
the finest, most beautiful code you've ever crafted, we'll all promise
to remember the beauty of GCSE instead. :-) Unless the script actually
presents a security risk in some way, I suggest you check it in under
contrib/make_snapshot or some such.
 
Thanks,

--
Mark Mitchell                   mark@codesourcery.com
CodeSourcery, LLC               http://www.codesourcery.com