RFR: JDK-8215952: NetBeans pre-build failed due to the incorrect configuration

Alan Bateman Alan.Bateman at oracle.com
Thu Jan 3 08:07:30 UTC 2019


On 03/01/2019 07:22, Fu Jie wrote:
> Thanks Alan for your review and valuable advice.
>
> I think it's worth keeping a NB project in the repo since it seems 
> more convenient for IDE developers.
> And to keep configurations.xml current is also important.
>
> I made a patch to fix the issue mentioned by Alan.
> Webrev: http://cr.openjdk.java.net/~aoqi/8215952/webrev.01/
> Thanks Ao Qi for posting the webrev.
>
> The main idea is that configurations.xml can be split into two parts: 
> the manually configured part and the automatically generated part.
> Only the manually configured part is checked into the repo which is 
> now less than 80 lines.
> And the other part will be generated automatically by NetBeans the 
> first time when the project is opened.
> The complete configurations.xml would be more than 20k lines after the 
> automatically generated part was filled in.
>
> With this patch, configurations.xml becomes source-code independent 
> and there is no need to update it anymore.
> Would you please review it and give me some advice?
This looks much better. Is the intent that someone using this project 
can edit all the sources in the JDK or just hotspot and the JNI/native 
code in the libraries? I ask because I don't understand how NB will 
locate the generated sources in gensrc, also not clear how NB will deal 
with java sources that are platform specific.

-Alan



More information about the build-dev mailing list