RFR: JDK-8034199 Add 'reconfigure' target for re-creating a configuration
Erik Joelsson
erik.joelsson at oracle.com
Thu Feb 13 10:30:53 UTC 2014
Looks even better to me.
/Erik
On 2014-02-12 23:31, Magnus Ihse Bursie wrote:
> On 2014-02-12 19:19, Martin Buchholz wrote:
>
>> I'm not sure exactly what the problem is, but do you need a contribution
>> like this:
> Yeah, something like that. :-) Thanks! I knew something like that was
> needed, but I did not feel like writing it.
>
> I took your snipped and adapted it slightly, to improve readability
> for common cases. E.g. if you call --with-extra-cflags="-DFOO -DBAR"
> it will store it as --with-extra-cflags='-DFOO -DBAR' rather than
> '--with-extra-cflags=-DFOO -DBAR'. They both evaluate to the same
> result, but the former is -- to me, at least, more easy to read and
> more likely to match what the user initially wrote.
>
> This interception needs to happen early, in the configure wrapper.
>
> I've tried to make this break with various evil combinations, but
> failed, so I think it's fairly robust. (And once again I really miss
> that I can't turn such adhoc testing into proper tests. :-( )
>
> Here's a new webrev:
> http://cr.openjdk.java.net/~ihse/JDK-8034199-add-reconfigure-target/webrev.02
>
> /Magnus
More information about the build-dev
mailing list