RFR: 8011814/8013271/8013272: Three improvements to J2SE Netbeans project

[Mike Duigou]

As a followup to this issue it's been pointed out that I pushed this patch with source-level set to "1.8" which is not supported by Netbeans 7.2 or 7.3. Source 1.8 is supported by the development version of Netbeans which is what I use primarily.

There's a problem here:

- The JDK repo now contains Java 8 source and over time will include much more Java 8 source (lambdas, default methods, static methods on interfaces, etc.).

- The release versions of Netbeans don't understand Java 8 source.


So we have to choose between using release versions of Netbeans and accept broken source parsing and using dev versions of Netbeans and the likely bugs (and missing extensions) that will be encountered.

Should we change the source level back to 1.7 for now?  My vote is "No", but I'm not the decider.

Mike

On Apr 29 2013, at 19:11 , Mike Duigou wrote:

> Hello All;
> 
> This is a review for three changes to the J2SE Netbeans project. If necessary I can break this up into three separate patches but I would rather not if possible.
> 
> 
> http://cr.openjdk.java.net/~mduigou/JDK-8011814/0/webrev/
> 
> 
> 8011814: Add testng.jar to Netbeans projects test compile classpath
> 
> An increasing number of jtreg tests now use TestNG. This change adds the TestNG jar from you JTReg installation to the tests classpath. The location of JTReg is specified in build.properties using jtreg.home or from the environment via JT_HOME.
> 
> 
> 8013271: Add OS X sources to J2SE Netbeans project
> 
> Adds as source entry for Apple OS X sources to match the Unix and Windows entries. I checked the trademark with the Apple Trademarks page to make sure I got it correct.
> 
> 
> 8013272: JDK Netbeans projects should use ASCII encoding for sources
> 
> The build scripts compile all OpenJDK java sources using the US-ASCII encoding. This change causes Netbeans to respect this encoding. Whether US-ASCII is the awesomest encoding is certainly debatable, but all editors and IDEs should use what the compiler uses.
> 
> 
> Thanks for reviewing,
> 
> Mike