RFR: JDK-8034199 Add 'reconfigure' target for re-creating a configuration (original) (raw)
Erik Joelsson erik.joelsson at oracle.com
Thu Feb 13 10:30:53 UTC 2014
- Previous message (by thread): RFR: JDK-8034199 Add 'reconfigure' target for re-creating a configuration
- Next message (by thread): RFR: JDK-8034199 Add 'reconfigure' target for re-creating a configuration
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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
- Previous message (by thread): RFR: JDK-8034199 Add 'reconfigure' target for re-creating a configuration
- Next message (by thread): RFR: JDK-8034199 Add 'reconfigure' target for re-creating a configuration
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]