Gaetan Nadon <[email protected]> writes: > On Tue, 2010-12-21 at 14:10 -0700, tom fogal wrote: > > > In the meantime, please consider the attached build.sh patch. I > > used it to configure my X build --without-fop, on all packages. > > That answers a question I had. Had you noticed the CONFFLAGS variable > which is meant to hold such configuration option like --without-fop? > Probably not as it the name and the description was obscure.
Oh, no, I hadn't noticed it. I guess it's just a personal preference of mine to (for the most part) ignore environment variables. They're nice when you have a deep pipeline with a complex frontend and thus it's poke at the guts, but when I'm scripting or just typing in commands, I like the option I am setting to be tightly coupled with the command; it's more obvious when I need to look back at what I did. > I was wondering if I should add an option on the build.sh command > line and your patch tells me I should. I like the option name you > have chosen. Should I keep the CONFFLAGS as well? Yes; IMHO it's always nice for an option to be backed by an env var. I do think that they should be named the same though; i.e. CONFFLAGS and --confflags or AUTOCONF and --autoconf. I think AUTOCONF means something to `autoreconf' though, so I'd actually lean towards your original CONFFLAGS/--confflags, now that I think about it. > I have a dozen patches in the pipeline, so I'll integrate yours. Feel > free to review the patches when posted. Okay, and will do! Thanks, -tom _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
