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

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

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


Reply | Threaded
Open this post in threaded view
|

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

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]


Reply | Threaded
Open this post in threaded view
|

[GitHub] [maven-shared-utils] michael-o commented on pull request #67: [MNG-6380] Update the wrapped jansi streams

GitBox
In reply to this post by 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]


Reply | Threaded
Open this post in threaded view
|

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

GitBox
In reply to this post by 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]


Reply | Threaded
Open this post in threaded view
|

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

GitBox
In reply to this post by 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]


Reply | Threaded
Open this post in threaded view
|

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

GitBox
In reply to this post by 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]


Reply | Threaded
Open this post in threaded view
|

[GitHub] [maven-shared-utils] gnodet commented on pull request #67: [MNG-6380] Option -Dstyle.color=always doesn't force color output

GitBox
In reply to this post by 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]