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