[GitHub] [maven-shared-utils] gnodet opened a new pull request #67: [MNG-6380] Update the wrapped jansi streams

[GitBox]

gnodet opened a new pull request #67:
URL: https://github.com/apache/maven-shared-utils/pull/67


   


----------------------------------------------------------------
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]

[GitBox]

gnodet commented on pull request #67:
URL: https://github.com/apache/maven-shared-utils/pull/67#issuecomment-737055511


   I need to release jansi before merging this PR.


----------------------------------------------------------------
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]

[GitBox]

michael-o commented on pull request #67:
URL: https://github.com/apache/maven-shared-utils/pull/67#issuecomment-737179842


   I will have a look at this combined with other Maven issues in a few days.


----------------------------------------------------------------
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]

[GitBox]

gnodet commented on pull request #67:
URL: https://github.com/apache/maven-shared-utils/pull/67#issuecomment-737546957


   I've pushed commits to leverage today's changes to Jansi.
   
   @slachiewicz I'm not sure to understand your question, could you rephrase please ?


----------------------------------------------------------------
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]

[GitBox]

gnodet commented on pull request #67:
URL: https://github.com/apache/maven-shared-utils/pull/67#issuecomment-764529402


   Rebased on top of #69


----------------------------------------------------------------
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]

[GitBox]

gnodet commented on pull request #67:
URL: https://github.com/apache/maven-shared-utils/pull/67#issuecomment-764529402


   Rebased on top of #69


----------------------------------------------------------------
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]

[GitBox]

gnodet commented on pull request #67:
URL: https://github.com/apache/maven-shared-utils/pull/67#issuecomment-765203066


   > I saw that something have changed in relation to unpacking libs. What native lib version will be used if we unpack (for maven core) lib to dedicated directory and set env variables also?
   
   The native libraries location have changed a bit with Jansi 2.x, so we need to properly change the maven distribution, see https://github.com/apache/maven/pull/433/commits/151e3494fec519197fb55df97e79f60bb2ef1d24


----------------------------------------------------------------
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]