Code review request for 7052122: Update JDK_MINOR_VERSION for JDK 8

Joe Darcy joe.darcy at oracle.com
Wed Jun 8 03:32:54 UTC 2011


Hello.

Please review this change to bump the JDK 8 version number from 1.7 to 1.8:

    7052122: Update JDK_MINOR_VERSION for JDK 8
    http://cr.openjdk.java.net/~darcy/7052122.0/

Patch below followed by post-GPL text of new langtools test.

Thanks,

-Joe

--- old/make/common/shared/Defs.gmk    2011-06-07 20:28:41.000000000 -0700
+++ new/make/common/shared/Defs.gmk    2011-06-07 20:28:41.000000000 -0700
@@ -195,8 +195,8 @@
 endif
 
 ifndef JDK_MINOR_VERSION
-  JDK_MINOR_VERSION      = 7
-  PREVIOUS_MINOR_VERSION = 6
+  JDK_MINOR_VERSION      = 8
+  PREVIOUS_MINOR_VERSION = 7
 endif
 
 ifndef JDK_MICRO_VERSION
--- old/make/docs/Makefile    2011-06-07 20:28:42.000000000 -0700
+++ new/make/docs/Makefile    2011-06-07 20:28:41.000000000 -0700
@@ -223,6 +223,9 @@
 ifeq ($(JDK_MINOR_VERSION),6)
   JDK_IS_FCS = true
 endif
+ifeq ($(JDK_MINOR_VERSION),7)
+  JDK_IS_FCS = true
+endif
 ifeq ($(JDK_IS_FCS),false)
   ifneq ($(MILESTONE), fcs)
     DRAFT_HEADER = 
<br><strong>DRAFT $(MILESTONE)-$(BUILD_NUMBER)</strong>

/*
 * @test
 * @bug 7025809
 * @summary Test latest and latestSupported
 * @author  Joseph D. Darcy
 */

import javax.lang.model.SourceVersion;
import static javax.lang.model.SourceVersion.*;

/**
 * Verify latest[Supported] behavior.
 */
public class TestSourceVersion {
    public static void main(String... args) {
        if (SourceVersion.latest() != RELEASE_8 ||
            SourceVersion.latestSupported() != RELEASE_8)
            throw new RuntimeException("Unexpected release value(s) 
found:\n" +
                                       "latest:\t" + 
SourceVersion.latest() + "\n" +
                                       "latestSupported:\t" + 
SourceVersion.latestSupported());
    }
}




More information about the build-dev mailing list