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