From e17e251af7b4ab58efacdf325471635206d2d263 Mon Sep 17 00:00:00 2001 From: Kumar Srinivasan Date: Thu, 14 Oct 2010 09:36:05 -0700 Subject: [PATCH] 6991164: pack source needs vendor rebranding changes (jdk7 only) Reviewed-by: ohair, jrose --- .../com/sun/java/util/jar/pack/Utils.java | 4 +++- jdk/test/tools/pack200/PackageVersionTest.java | 16 +++++++++++++++- 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/jdk/src/share/classes/com/sun/java/util/jar/pack/Utils.java b/jdk/src/share/classes/com/sun/java/util/jar/pack/Utils.java index 2dc47c41464..33524325d8e 100644 --- a/jdk/src/share/classes/com/sun/java/util/jar/pack/Utils.java +++ b/jdk/src/share/classes/com/sun/java/util/jar/pack/Utils.java @@ -216,7 +216,9 @@ class Utils { // Returns the Max Version String of this implementation static String getVersionString() { - return "Pack200, Vendor: Sun Microsystems, Version: " + + return "Pack200, Vendor: " + + System.getProperty("java.vendor") + + ", Version: " + Constants.JAVA6_PACKAGE_MAJOR_VERSION + "." + Constants.JAVA6_PACKAGE_MINOR_VERSION; } diff --git a/jdk/test/tools/pack200/PackageVersionTest.java b/jdk/test/tools/pack200/PackageVersionTest.java index 344aadd6b61..538b92ecafc 100644 --- a/jdk/test/tools/pack200/PackageVersionTest.java +++ b/jdk/test/tools/pack200/PackageVersionTest.java @@ -24,7 +24,7 @@ /* * @test - * @bug 6712743 + * @bug 6712743 6991164 * @summary verify package versions * @compile -XDignore.symbol.file Utils.java PackageVersionTest.java * @run main PackageVersionTest @@ -40,6 +40,7 @@ import java.io.PrintStream; import java.util.jar.JarFile; import java.util.jar.Pack200; import java.util.jar.Pack200.Packer; +import java.util.jar.Pack200.Unpacker; public class PackageVersionTest { private static final File javaHome = new File(System.getProperty("java.home")); @@ -60,6 +61,7 @@ public class PackageVersionTest { createClassFile("Test6"); createClassFile("Test7"); + verify6991164(); verifyPack("Test5.class", JAVA5_PACKAGE_MAJOR_VERSION, JAVA5_PACKAGE_MINOR_VERSION); @@ -75,6 +77,18 @@ public class PackageVersionTest { JAVA5_PACKAGE_MINOR_VERSION); } + static void verify6991164() { + Unpacker unpacker = Pack200.newUnpacker(); + String versionStr = unpacker.toString(); + String expected = "Pack200, Vendor: " + + System.getProperty("java.vendor") + ", Version: " + + JAVA6_PACKAGE_MAJOR_VERSION + "." + JAVA6_PACKAGE_MINOR_VERSION; + if (!versionStr.equals(expected)) { + System.out.println("Expected: " + expected); + System.out.println("Obtained: " + versionStr); + throw new RuntimeException("did not get expected string " + expected); + } + } static void createClassFile(String name) { createJavaFile(name);