[GitHub] [poi] robmv opened a new pull request #198: Remove jdk.charset module dependency for spreadsheets generation

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] [poi] robmv opened a new pull request #198: Remove jdk.charset module dependency for spreadsheets generation

GitBox

robmv opened a new pull request #198:
URL: https://github.com/apache/poi/pull/198


   Move double byte charset utility code to scratchpad...hwpf that is the only code that currently requires them. This allows using spreadsheet related code without pulling jdk.charset to custom built JREs using jlink.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [poi] robmv commented on pull request #198: Remove jdk.charset module dependency for spreadsheets generation

GitBox

robmv commented on pull request #198:
URL: https://github.com/apache/poi/pull/198#issuecomment-731599873


   The build failure looks unrelated to my changes
   
   ```
   /home/travis/build/apache/poi/build.xml:938: javac doesn't support the "release" attribute
   ```


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [poi] kiwiwings commented on pull request #198: Remove jdk.charset module dependency for spreadsheets generation

GitBox
In reply to this post by GitBox

kiwiwings commented on pull request #198:
URL: https://github.com/apache/poi/pull/198#issuecomment-731607535


   > The build failure looks unrelated to my changes
   >
   > ```
   > /home/travis/build/apache/poi/build.xml:938: javac doesn't support the "release" attribute
   > ```
   
   If you get that warning, your javac is not running in Java 9+ mode. I thought I've changed the build in a way, that it recognizes the JVM. So it uses classpath mode for Java 8 and modulepath mode for Java 9+. if this doesn't work for your environment, I need to investigate what's the difference there ... with your help.
   
   anyway with this warning and your proposed change, I think the tests are running in classpath mode and therefore aren't significant ...


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [poi] robmv commented on pull request #198: Remove jdk.charset module dependency for spreadsheets generation

GitBox
In reply to this post by GitBox

robmv commented on pull request #198:
URL: https://github.com/apache/poi/pull/198#issuecomment-731628527


   > If you get that warning, your javac is not running in Java 9+ mode.
   
   My build using Java 8 says: `Support for javac --release has been added in Java9 ignoring it`.
   
   The build machines say the Ant version is:  `Using Ant: Apache Ant(TM) version 1.9.6 compiled on July 20 2018`
   
    This is a [message from current Ant](https://github.com/apache/ant/blob/a8645a151bc706259fb1789ef587d05482d98612/src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java#L402) that is [not present in that old Ant](https://github.com/apache/ant/blob/rel/1.9.6/src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java)
   
   An Ant update on the build machines should fix it


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [poi] robmv edited a comment on pull request #198: Remove jdk.charset module dependency for spreadsheets generation

GitBox
In reply to this post by GitBox

robmv edited a comment on pull request #198:
URL: https://github.com/apache/poi/pull/198#issuecomment-731628527


   > If you get that warning, your javac is not running in Java 9+ mode.
   
   My build using Java 8 says: `Support for javac --release has been added in Java9 ignoring it`.
   
   The build machines say the Ant version is:  `Using Ant: Apache Ant(TM) version 1.9.6 compiled on July 20 2018`
   
    This is a [message from current Ant](https://github.com/apache/ant/blob/a8645a151bc706259fb1789ef587d05482d98612/src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java#L402) that is [not present in that old Ant](https://github.com/apache/ant/blob/rel/1.9.6/src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java)
   
   An Ant update on the build machines should fix it
   
   Edit: My Ant is `Apache Ant(TM) version 1.10.5 compiled on July 10 2018`


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [poi] robmv commented on pull request #198: Remove jdk.charset module dependency for spreadsheets generation

GitBox
In reply to this post by GitBox

robmv commented on pull request #198:
URL: https://github.com/apache/poi/pull/198#issuecomment-734036119


   > If you get that warning
   
   Just for clarifying, I don't get that error, in my machine the build is ok. The Travis CI display that error, see my previous message about the Travis CI using and older version of Ant than me


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]