maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (MJAVADOC-544) Changed behaviour of Javadoc for temporary files encoding (options, argfile, ...)
Date Fri, 30 Nov 2018 10:13:00 GMT


ASF GitHub Bot commented on MJAVADOC-544:

rfscholte commented on a change in pull request #10: [MJAVADOC-544] - Changed behaviour of
Javadoc for temporary files encoding (options, argfile, ...)

 File path: src/main/java/org/apache/maven/plugins/javadoc/
 @@ -4368,9 +4378,15 @@ private void addCommandLineOptions( Commandline cmd, List<String>
arguments, Fil
         options.append( StringUtils.join( arguments.iterator(),
                                           SystemUtils.LINE_SEPARATOR ) );
+        /* default to platform encoding */
+        String encoding = null;
+        if ( JAVA_VERSION.compareTo( SINCE_JAVADOC_9 ) >= 0 )
 Review comment:
   Please switch to `isAtLeast()` or `isBefore()`, which are better readable.

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

> Changed behaviour of Javadoc for temporary files encoding (options, argfile, ...)
> ---------------------------------------------------------------------------------
>                 Key: MJAVADOC-544
>                 URL:
>             Project: Maven Javadoc Plugin
>          Issue Type: Bug
>          Components: javadoc
>    Affects Versions: 3.0.1
>         Environment: JDK9+, Windows
>            Reporter: Michael Stumpf
>            Priority: Major
> With JDK9 the behaviour of javadoc has apparently changed (at least on Windows platform).
> In a project with special characters (umlaut) in the organization name the content of
the options file was written with the default platform encoding (cp1252).
> As a result, the javadoc generation fails on javadoc execution with "Exit code: 1 - javadoc:
error - cannot read Input length = 1".
> With JDK8 javadoc generation works.
> It also succeeds with JDK9 and setting the environment variable: JAVA_TOOL_OPTIONS=-Dfile.encoding=UTF-8
> However, this should not be necessary, and maven-javadoc-plugin should work out of the
> h6. Cause:
> After some tests it appears to be the case that the JDK9 Javadoc reads @-files always
with UTF-8 charset, regardless of platform encoding.
> I looked into the issue and prepared a patch for the plugin when working with JDK9+ and
added two tests for the temporary files options and argfile.
> To my knowledge, the temporary file "packages" cannot contain special characters, since
this isn't allowed in the java specification, so no workaround is added.
> The file javadoc.bat is not affected (a test with a JDK in the directory "C:\jdk9äöüß"
worked with the file being platform encoded).
> h6. 
> Resolution:
> I'll try to create a pull request from

This message was sent by Atlassian JIRA

View raw message