Reviewer needed: fix for class RegularFileObject + added new regression test
Dr Andrew John Hughes
ahughes at redhat.com
Tue Mar 29 05:40:09 PDT 2011
On 11:36 Tue 29 Mar , Pavel Tisnovsky wrote:
> Dr Andrew John Hughes wrote:
> > With that change, it's good to go in.
>
> Andrew, thank you for your review. Pushed to HEAD.
>
> > I would guess you want this in the 1.10.x series too. If so, can you
> > post a version for 1.10 and I'll approve it for the branch too? Thanks.
> >
>
> You are right. hg diff generated against latest IcedTea6-1.10 is stored
> in attachment and ChangeLog entry is almost the same as for 1.10 (I only
> fixed typo & date):
>
> 2011-03-29 Pavel Tisnovsky <ptisnovs at redhat.com>
>
> * Makefile.am: Add new patch.
> * patches/regular-file-object-patch:
> Patch which fixes the behaviour of JavaFileManager
> and JavaFileObject classes. This patch also contains
> regression test for check if the fix is correct.
>
>
> Can you please review this change?
>
> Thank you in advance,
> Pavel
The ChangeLog file has the wrong filename in it still.
Also, can you please list the fix under 'Bug Fixes' in NEWS.
Something like:
* Fixed regression in JavaFileObject.getName() behaviour caused by 6885123
Thanks.
> diff -r 2b50c801e31b Makefile.am
> --- a/Makefile.am Sat Mar 26 08:27:56 2011 +0100
> +++ b/Makefile.am Tue Mar 29 11:29:17 2011 +0200
> @@ -325,7 +325,8 @@
> patches/pr600-arm-jvm.cfg.patch \
> patches/jaxp-serial-version-uid.patch \
> patches/openjdk/7023591-AAShapePipe.patch \
> - patches/openjdk/7027667-AAShapePipeRegTest.patch
> + patches/openjdk/7027667-AAShapePipeRegTest.patch \
> + patches/revert-6885123.patch
>
> if WITH_ALT_HSBUILD
> ICEDTEA_PATCHES += \
> diff -r 2b50c801e31b patches/revert-6885123.patch
> --- /dev/null Thu Jan 01 00:00:00 1970 +0000
> +++ b/patches/revert-6885123.patch Tue Mar 29 11:29:17 2011 +0200
> @@ -0,0 +1,310 @@
> +--- openjdk.old/langtools/src/share/classes/com/sun/tools/javac/file/RegularFileObject.java 2011-02-28 17:07:28.000000000 +0100
> ++++ openjdk/langtools/src/share/classes/com/sun/tools/javac/file/RegularFileObject.java 2011-03-25 10:16:13.646180000 +0100
> +@@ -78,7 +78,7 @@
> +
> + //@Override
> + public String getName() {
> +- return file.getPath();
> ++ return this.name;
> + }
> +
> + //@Override
> +--- /dev/null 1970-01-01 01:00:00.000000000 +0100
> ++++ openjdk/jdk/test/javax/tools/JavaFileManager/JavaFileManagerTest.java 2011-03-28 15:51:43.000000000 +0200
> +@@ -0,0 +1,119 @@
> ++/*
> ++ * Copyright 2011 Red Hat, Inc. All Rights Reserved.
> ++ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
> ++ *
> ++ * This code is free software; you can redistribute it and/or modify it
> ++ * under the terms of the GNU General Public License version 2 only, as
> ++ * published by the Free Software Foundation.
> ++ *
> ++ * This code is distributed in the hope that it will be useful, but WITHOUT
> ++ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
> ++ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
> ++ * version 2 for more details (a copy is included in the LICENSE file that
> ++ * accompanied this code).
> ++ *
> ++ * You should have received a copy of the GNU General Public License version
> ++ * 2 along with this work; if not, write to the Free Software Foundation,
> ++ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
> ++ */
> ++
> ++
> ++import java.util.*;
> ++import java.io.*;
> ++import java.nio.charset.*;
> ++
> ++import javax.tools.*;
> ++import javax.tools.JavaFileManager.*;
> ++import javax.tools.JavaFileObject.*;
> ++
> ++public class JavaFileManagerTest {
> ++
> ++ /**
> ++ * Test data files are stored in a subdirectory named "test_data".
> ++ */
> ++ private static final String TEST_DATA_DIRECTORY = "test_data";
> ++
> ++ /**
> ++ * Charset used by JavaFileManager.
> ++ */
> ++ private static final Charset FILE_MANAGER_CHARSET = Charset.forName("US-ASCII");
> ++
> ++ /**
> ++ * Locale used by JavaFileManager.
> ++ */
> ++ private static final Locale FILE_MANAGER_LOCALE = (java.util.Locale)null;
> ++
> ++ /**
> ++ * Location of file objects being tested.
> ++ */
> ++ private static final Location FILE_OBJECTS_LOCATION = javax.tools.StandardLocation.SOURCE_PATH;
> ++
> ++ public static DiagnosticListener<JavaFileObject> listener = new DiagnosticCollector<JavaFileObject>();
> ++
> ++ private void doTest() throws IOException {
> ++ StandardJavaFileManager fileManager = getStandardFileManager();
> ++ fileManager.setLocation(FILE_OBJECTS_LOCATION, Collections.singleton(getSourceDirPathFile()));
> ++
> ++ checkFileManagerLocation(fileManager.getLocation(FILE_OBJECTS_LOCATION));
> ++ checkFileManagerGetName(fileManager);
> ++ }
> ++
> ++ private StandardJavaFileManager getStandardFileManager() {
> ++ return ToolProvider.getSystemJavaCompiler().getStandardFileManager(
> ++ listener, FILE_MANAGER_LOCALE, FILE_MANAGER_CHARSET);
> ++ }
> ++
> ++ public static File getSourceDirPathFile() {
> ++ return new File(new File("."), TEST_DATA_DIRECTORY);
> ++ }
> ++
> ++ /**
> ++ * Check if there's "./test_data" location included in a locations argument.
> ++ * @param locations set of locations returned by fileManager.getLocation()
> ++ */
> ++ private void checkFileManagerLocation(Iterable<? extends File> locations) {
> ++ for (File file : locations) {
> ++ if (".".equals(file.getParent()) && TEST_DATA_DIRECTORY.equals(file.getName())) {
> ++ System.out.println("checkFileManagerLocation: ok");
> ++ return;
> ++ }
> ++ }
> ++ throw new RuntimeException("fileManager.getLocation() returns wrong locations! " + locations.toString());
> ++ }
> ++
> ++ /**
> ++ * Check if method JavaFileManager.getName() returns correct file name.
> ++ * @param fileManager
> ++ * @throws IOException
> ++ */
> ++ private void checkFileManagerGetName(JavaFileManager fileManager) throws IOException {
> ++ Map<Kind, String> kinds = new HashMap<Kind, String>();
> ++ kinds.put(Kind.SOURCE, "Test.java");
> ++ kinds.put(Kind.CLASS, "Test.class");
> ++ kinds.put(Kind.HTML, "Test.html");
> ++ kinds.put(Kind.OTHER, "Test.txt");
> ++ for (Map.Entry<Kind, String> kind : kinds.entrySet()) {
> ++ System.out.println("Testing JavaFileManager for kind " + kind.getKey());
> ++ Iterable<JavaFileObject> fileObjects = fileManager.list(
> ++ FILE_OBJECTS_LOCATION, "", Collections.singleton(kind.getKey()), false);
> ++ checkIfFileObjectsContainName(fileObjects, kind.getValue());
> ++ }
> ++ }
> ++
> ++ private void checkIfFileObjectsContainName(Iterable<JavaFileObject> fileObjects, String name) {
> ++ //System.out.println(fileObjects);
> ++ for (JavaFileObject fileObject : fileObjects) {
> ++ //System.out.println(fileObject.getName());
> ++ if (name.equals(fileObject.getName())) {
> ++ System.out.println("JavaFileObject.getName(): ok ('" + fileObject.getName() + "')");
> ++ return;
> ++ }
> ++ }
> ++ throw new RuntimeException("JavaFileObject.getName() returns wrong name!");
> ++ }
> ++
> ++ public static void main(String argv[]) throws IOException {
> ++ new JavaFileManagerTest().doTest();
> ++ }
> ++
> ++}
> +--- /dev/null 1970-01-01 01:00:00.000000000 +0100
> ++++ openjdk/jdk/test/javax/tools/JavaFileManager/JavaFileManagerTest.sh 2011-03-28 15:47:23.000000000 +0200
> +@@ -0,0 +1,153 @@
> ++#!/bin/sh
> ++
> ++#
> ++# Copyright 2011 Red Hat, Inc. All Rights Reserved.
> ++# DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
> ++#
> ++# This code is free software; you can redistribute it and/or modify it
> ++# under the terms of the GNU General Public License version 2 only, as
> ++# published by the Free Software Foundation.
> ++#
> ++# This code is distributed in the hope that it will be useful, but WITHOUT
> ++# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
> ++# FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
> ++# version 2 for more details (a copy is included in the LICENSE file that
> ++# accompanied this code).
> ++#
> ++# You should have received a copy of the GNU General Public License version
> ++# 2 along with this work; if not, write to the Free Software Foundation,
> ++# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
> ++
> ++#
> ++# This script is the actual launcher of JavaFileManagerTest.java
> ++#
> ++
> ++# @test
> ++# @summary Test that check proper behaviour of JavaFileManager and
> ++# JavaFileObject classes.
> ++
> ++
> ++
> ++# check arguments set by JTreg
> ++function checkArgs() {
> ++ if [ "${TESTSRC}" = "" ]
> ++ then
> ++ echo "TESTSRC not set. Test cannot execute. Failed."
> ++ exit 1
> ++ fi
> ++ echo "TESTSRC=${TESTSRC}"
> ++ if [ "${TESTJAVA}" = "" ]
> ++ then
> ++ echo "TESTJAVA not set. Test cannot execute. Failed."
> ++ exit 1
> ++ fi
> ++ echo "TESTJAVA=${TESTJAVA}"
> ++ if [ "${TESTCLASSES}" = "" ]
> ++ then
> ++ echo "TESTCLASSES not set. Test cannot execute. Failed."
> ++ exit 1
> ++ fi
> ++}
> ++
> ++
> ++
> ++# print paths to test sources and test classes
> ++function printPaths() {
> ++ echo "TESTCLASSES=${TESTCLASSES}"
> ++ echo "CLASSPATH=${CLASSPATH}"
> ++}
> ++
> ++
> ++
> ++# copy of all necessarry files into work directory
> ++function copyWorkFiles() {
> ++ cp -r "${TESTSRC}/test_data" "${TESTCLASSES}/"
> ++ result=$?
> ++ if [ $result -eq 0 ]
> ++ then
> ++ echo "Copy of work files was successful."
> ++ else
> ++ echo "copy of work files have failed."
> ++ exit $result
> ++ fi
> ++
> ++ cp "${TESTSRC}/JavaFileManagerTest.java" "${TESTCLASSES}"
> ++ result=$?
> ++ if [ $result -eq 0 ]
> ++ then
> ++ echo "Copy of work files was successful."
> ++ else
> ++ echo "copy of work files have failed."
> ++ exit $result
> ++ fi
> ++}
> ++
> ++
> ++
> ++# compilation of Test class which is stored in test_data subdirectory
> ++function compileTestClass() {
> ++ COMPILE="${TESTJAVA}/bin/javac ${TESTCLASSES}/test_data/Test.java"
> ++ echo ${COMPILE}
> ++ ${COMPILE}
> ++ result=$?
> ++
> ++ if [ $result -eq 0 ]
> ++ then
> ++ echo "Compilation of the test class was successful."
> ++ else
> ++ echo "Compilation of the test class have failed."
> ++ # Cleanup
> ++ rm -rf ${TESTCLASSES}/
> ++ exit $result
> ++ fi
> ++}
> ++
> ++
> ++
> ++# compilation of the Test itself
> ++function compileTest() {
> ++ COMPILE="${TESTJAVA}/bin/javac ${TESTCLASSES}/JavaFileManagerTest.java"
> ++ echo ${COMPILE}
> ++ ${COMPILE}
> ++ result=$?
> ++
> ++ if [ $result -eq 0 ]
> ++ then
> ++ echo "Compilation of the test case was successful."
> ++ else
> ++ echo "Compilation of the test case failed."
> ++ # Cleanup
> ++ rm -rf ${TESTCLASSES}/
> ++ exit $result
> ++ fi
> ++}
> ++
> ++
> ++
> ++# run the test
> ++function runTest() {
> ++ cd ${TESTCLASSES}
> ++ RUNCMD="${TESTJAVA}/bin/java JavaFileManagerTest"
> ++ echo ${RUNCMD}
> ++ ${RUNCMD}
> ++ result=$?
> ++
> ++ if [ $result -eq 0 ]
> ++ then
> ++ echo "Execution successful"
> ++ else
> ++ echo "Execution of the test case failed."
> ++ fi
> ++ rm -rf ${TESTCLASSES}/
> ++ exit $result
> ++}
> ++
> ++
> ++
> ++checkArgs
> ++printPaths
> ++copyWorkFiles
> ++compileTestClass
> ++compileTest
> ++runTest
> ++
> +--- /dev/null 1970-01-01 01:00:00.000000000 +0100
> ++++ openjdk/jdk/test/javax/tools/JavaFileManager/test_data/Test.html 2011-03-25 18:19:45.000000000 +0100
> +@@ -0,0 +1,5 @@
> ++<html>
> ++<body>
> ++42
> ++</body>
> ++</html>
> +--- /dev/null 1970-01-01 01:00:00.000000000 +0100
> ++++ openjdk/jdk/test/javax/tools/JavaFileManager/test_data/Test.java 2011-03-28 14:53:50.000000000 +0200
> +@@ -0,0 +1,5 @@
> ++package test_data;
> ++
> ++public class Test {
> ++ // nothing here
> ++}
> +--- /dev/null 1970-01-01 01:00:00.000000000 +0100
> ++++ openjdk/jdk/test/javax/tools/JavaFileManager/test_data/Test.txt 2011-03-28 16:15:09.000000000 +0200
> +@@ -0,0 +1,2 @@
> ++42
> ++
--
Andrew :)
Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)
Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: F5862A37 (https://keys.indymedia.org/)
Fingerprint = EA30 D855 D50F 90CD F54D 0698 0713 C3ED F586 2A37
More information about the distro-pkg-dev
mailing list