On Mon, 27 Jan 2014, Bruce Korb wrote: > On Sun, Jan 26, 2014 at 9:38 PM, Hans-Peter Nilsson <h...@bitrange.com> wrote: > > On Sun, 8 Dec 2013, Bruce Korb wrote: > >> On 12/08/13 13:06, Gerald Pfeifer wrote: > >> > Lovely. Thank you very much! > > > > (Looks like nobody replied to this and it isn't committed.) > > > No, flex isn't "required to build GCC" as a hard and fast rule. > > It's not required for releases, just when building from a svn > > checkout (or really: when the flex-generated files are not in > > the source directory). > > > > You'd need some additional conditions. There might be the > > additional issue that any "lex" is expected to work too, not > > just "flex". > > It isn't committed 'cuz nobody said, "Okay." > I do wish either someone would say, "Okay." or come up with something > that works. > I went to the effort to figure out where things got off the rails and > did something that > worked for me. Just saying, "That won't work" without a workable alternative > is a bit irritating.
There are surely plenty of opportunities around for irritatation! :) Dropped patches surely; I thought I was helpful there. Ungraceful errors from easily identifiable common gotchas definitely, so what you're trying to achieve is IMHO desirable. Patches hacking in something that just happened to work for someone too; I tried to stop that from happening. > I do not know the difference between a checkout build and a "normal" > configured build. There's a difference between builds from released tarballs and builds from svn co + contrib/gcc_update, yes. > My understanding was that generated files were to > be part of the repo and that there was *not* a difference. Nope, fewer generated files in the repo. (Not all of them, not none of them.) > If there is, > then someone who understands the difference could maybe add some > configure infrastructure to test the environment and decide if flex (or lex) > was needed, rather than just say, "what you did won't work." See is_release in that same configure.ac, that might be the only additional condition that's needed. Happy Hacking. brgds, H-P
