Re: RFR: 8011814/8013271/8013272: Three improvements to J2SE Netbeans project
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
Re: RFR: 8011814/8013271/8013272: Three improvements to J2SE Netbeans project
Hi Mike, The changes look good to me. Best Lance On Apr 29, 2013, at 10:11 PM, 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 Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
RFR: 8011814/8013271/8013272: Three improvements to J2SE Netbeans project
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