[MNG-6380] Option -Dstyle.color=always doesn't force color output

[gnodet]

No description provided.

[gnodet]

I need to release jansi before merging this PR.

[slachiewicz]

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?

[michael-o]

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

[gnodet]

I've pushed commits to leverage today's changes to Jansi.

@slachiewicz I'm not sure to understand your question, could you rephrase please ?

[gnodet]

Rebased on top of #69

[gnodet]

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 apache/maven@151e349

[elharo]

Running through jenkins on branch gnodet

[elharo]

JAnsi looks released. Can this be merged?

[elharo]

should this be 2.3.1 now?

[gnodet]

We could, but the 2.3.0/2.3.1 only brings two additional native methods specifically for mvnd and no other changes, so there is no real benefit either.

[MartinKanters]

@gnodet I see that jansi was already upgraded in #69. Is this work still needed? Is it required or nice to have for the Maven Core 4.0.0 (alpha) release eventually?

[gnodet]

@MartinKanters jansi is a dependency both maven-shared and core maven, so it needs to be upgraded in both. In addition, MNG-6380 is resolved with the commits in this PR and not simply the upgrade of jansi.

[gnodet]

Rebased and squashed.

[elharo]

checking jenkins: https://ci-builds.apache.org/job/Maven/job/maven-box/job/maven-shared-utils/job/MNG-6380/

[elharo]

jenkins passed.