CompileJavaModule.gmk overrides values from a custom extension gmk

[Jason Yong]

Hi Eric,

I've removed the SETUP changes as requested...



On a side note, I noticed that the attachment got stripped out in the post 
to the mailing list. Should I actually be copying and pasting the entire 
diff in the message? Its a couple of hundred lines... or is somewhere to 
put the attachment? 


Regards,

Jason Yong
CEng MEng MIET
Software Engineer, IBM Runtimes Technology
IBM Hybrid Cloud


Phone: 44-1962-815256
E-mail: yongja at uk.ibm.com
Find me on:  and within IBM on:  


Hursley Park
Hursley, SO212JN
United Kingdom





From:   Erik Joelsson <erik.joelsson at oracle.com>
To:     Jason Yong <Jason_Yong at uk.ibm.com>
Cc:     build-dev at openjdk.java.net
Date:   30/08/2017 14:43
Subject:        Re: CompileJavaModule.gmk overrides values from a custom 
extension gmk



Hello,

Changing the assignment on COPY, CLEAN and FLAGS variables makes sense, 
but please revert the SETUP variables as those are not lists but single 
value types.

Otherwise this looks good to me.

/Erik


On 2017-08-30 15:37, Jason Yong wrote:
> Hi Eric,
>
> With regards to the OCA I believe IBM has signed a contributors 
agreement
> which should cover me for that.
>
>
> So here's the mercurial export of the CompileJavaModule.java with my
> changes in
>
>
>
>
> Regards,
>
>
> Jason Yong
>
> CEng MEng MIET
> Software Engineer, IBM Runtimes Technology
> IBM Hybrid Cloud
>
>
> Phone: 44-1962-815256
> E-mail: yongja at uk.ibm.com
> Find me on:  and within IBM on:
>
>
> Hursley Park
> Hursley, SO212JN
> United Kingdom
>
>
>
>
>
> From:   Erik Joelsson <erik.joelsson at oracle.com>
> To:     Jason Yong <Jason_Yong at uk.ibm.com>
> Cc:     build-dev at openjdk.java.net
> Date:   30/08/2017 13:47
> Subject:        Re: CompileJavaModule.gmk overrides values from a custom
> extension gmk
>
>
>
> If you have signed the OCA, you can post your proposed change here and I
> or someone else will sponsor it once we agree that it looks good.
> /Erik
>
> On 2017-08-30 14:27, Jason Yong wrote:
> Thanks Eric,
>
> Is the next step is to get a sponsor for the change or should I post my
> proposed change first?
>
> Jason
>
> Regards,
>
> Jason Yong
> CEng MEng MIET
> Software Engineer, IBM Runtimes Technology
> IBM Hybrid Cloud
>
>
> Phone: 44-1962-815256
> E-mail: yongja at uk.ibm.com
> Find me on:  and within IBM on:
>
>
> Hursley Park
> Hursley, SO212JN
> United Kingdom
>
>
>
>
>
>
> From:        Erik Joelsson <erik.joelsson at oracle.com>
> To:        Jason Yong <Jason_Yong at uk.ibm.com>, 
build-dev at openjdk.java.net
> Date:        29/08/2017 12:55
> Subject:        Re: CompileJavaModule.gmk overrides values from a custom
> extension gmk
>
>
>
> Hello Jason,
>
> Your suggestion makes sense. The only reason these variables have :=
> today is that we (at Oracle) haven't had a need for appending to those
> particular variables (yet).
>
> /Erik
>
>
> On 2017-08-29 11:31, Jason Yong wrote:
>> Hello,
>> 
>> I've had an issue where I've had a custom extension to
> CompileJavaModules.gmk with the variable java.base_COPY set to files 
that
> I wanted to be copied across but its value was overwritten by
> CompileJavaModules.gmk.
>> 
>> I would like to propose changes that would allow a custom extensions to
> update variables listed in CompileJavaModules.gmk. This issue is similar
> to bug JDK-8064372 (
> 
https://urldefense.proofpoint.com/v2/url?u=https-3A__bugs.openjdk.java.net_browse_JDK-2D8064372&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=_pd0JudnAMtzyOP3NkOCP7ozDRbZ9ukki8lLmogKLJI&m=qG_YEjgXnzBfNvN5ztSbjIVP3nZ5SaZCibl7SHJjTfc&s=2mMmeC6jKTJu1OgK0Lssib1LKQvOFX3BwzIcJAebSeU&e=

> ) but affects all the other variables such as:
>> 
>> java.activation_SETUP
>> java.base_ADD_JAVAC_FLAGS
>> java.base_COPY
>> java.base_CLEAN
>> etc
>> 
>> The fix is also similar, changing := to += allowing the custom 
extension
> to append to the variable if already set and create it if its not.
>> I would appreciate any feedback and help on what the next steps would
> be.
>> Thanks
>>
>> Jason
>>
>>
>> Jason Yong
>> CEng MEng MIET
>> Software Engineer, IBM Runtime Technologies
>> IBM Hybrid Cloud
>> Phone: 44-1962-815256
>> E-mail: yongja at uk.ibm.com
>> Find me on:  and within IBM on:
>> Unless stated otherwise above:
>> IBM United Kingdom Limited - Registered in England and Wales with 
number
> 741598.
>> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6
> 3AU
>
>
>
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 
3AU
>
>
>
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 
3AU




Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 
741598. 
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU