[Bug] The command line 'mvn versions:set' returns exit code 1 with the last maven version 3.8.2 #483

RomainPruvostMHH · 2021-08-24T15:31:38Z

Hi,

I have a shell script who runs the command line 'mvn versions:set' and verify the exit code. With my previous Maven 3.6.3, everything works fine.
But I recently upgraded maven to 3.8.2, and now the command line 'mvn versions:set' returns exit code 1 systematically. I precise that the result of the mvn command is BUILD SUCCESS of course.

  mvn versions:set -DgenerateBackupPoms=false -DnewVersion=$mvn_new_version
  exit_code=$?
  if [ $exit_code -ne 0 ]; then
    echo "ERROR : cmd mvn versions:set -DnewVersion=$mvn_new_version failed"
    exit 10
  fi

Any idea ?

Kind regards,
Romain

The text was updated successfully, but these errors were encountered:

slawekjaranowski · 2021-08-24T16:07:12Z

Can you show output of:

mvn -V versions:set -DnewVersion=1.2.3; echo "result=$?"

I can't see a problem - works for me 😄

RomainPruvostMHH · 2021-08-24T17:05:42Z

Hi @slawekjaranowski,

I reproduce the problem with the minimum of files.

With maven 3.6.3, it's ok :

With maven 3.8.2, it's KO :

slawekjaranowski · 2021-08-25T13:16:31Z

ok, last simply test:

 mvn -v; echo "result=$?"

RomainPruvostMHH · 2021-08-25T14:15:51Z

slawekjaranowski · 2021-08-25T16:28:28Z

I don't have any more quick idea ... can you share your test project

RomainPruvostMHH · 2021-08-26T07:13:42Z

See in attachment my test project. But it is very simple. Not sure it can help you.
test-maven-3.8.2.zip

slawekjaranowski · 2021-08-26T19:41:07Z

I can't reproduce wrong behavior on macos system ... so maybe it is connected with windows os

can you test on maven 3.8.1?

you can also try to write on "Maven User List" or "Maven Developer List"
http://maven.apache.org/mailing-lists.html

slachiewicz · 2021-08-26T20:43:51Z

I can reproduce the issue with Windows/MinGW - please send mail to the Maven Developers list

sylwe@DESKTOP-7BCBELM MINGW64 /c/tmp/test-maven-3.8.2
$ uname -a
MINGW64_NT-10.0-22000 DESKTOP-7BCBELM 3.1.7-340.x86_64 2020-10-23 13:08 UTC x86_64 Msys

raphaelNguyen · 2021-09-03T01:27:35Z

I also have had this problem. I was on Maven 3.8.1 which was fine. After upgrading to 3.8.2, I started having my scripts failing and noticed the problem. Was able to reproduce it with a very minimalist project with no code and just a pom as follows:

<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0"
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
  <modelVersion>4.0.0</modelVersion>
  <groupId>com.test</groupId>
  <artifactId>test</artifactId>
  <version>1.0</version>
</project>

All these are done on the same version of versions-maven-plugin so maybe this is some weird interaction with the new version of maven? Please tell me if I can provide any more information. I'm running maven on git bash for windows.

Same with the previous posts on this issue, the output of set looks identical to the same run on maven 3.8.1. Build was successful as indicated by maven. Only the exit code is 1 instead of 0.

slawekjaranowski · 2021-09-03T06:21:47Z

Only out of my curiosity - can anyone trigger build on master branch - now GitHub runners should have Maven 3.8.2

slawekjaranowski · 2021-09-07T06:55:17Z

You can also create issue at Apache Jira for Maven project:
https://issues.apache.org/jira/projects/MNG

RomainPruvostMHH · 2021-09-07T07:01:09Z

@slawekjaranowski So, you are sure the problem is at Maven and is not at versions plugin ?

slawekjaranowski · 2021-09-07T07:24:53Z

I'm not sure - where is the problem but Maven 3.8.2 exposes it ...
I don't have access to host with windows os so I can't reproduce it

Maybe someone else can help with it, I think it is good idea to post on Maven Developers list or create issue at Maven jira.

I heard that new Maven version 3.8.x will be prepared to the end of month - so maybe fix (if needed) will be catched

RomainPruvostMHH · 2021-09-07T07:52:37Z

Issue at Apache Maven :

https://issues.apache.org/jira/browse/MNG-7234

michael-o · 2021-09-07T10:32:01Z

Need to take a look at it. This is weird. There have been virtually no changes to the startup scripts of Maven in 3.8.2.

michael-o · 2021-09-07T10:58:07Z

Hmmm:

$ ~/apache-maven-3.8.2/bin/mvn -V versions:set -DnewVersion=3 ; echo $?
Apache Maven 3.8.2 (ea98e05a04480131370aa0c110b8c54cf726c06f)
Maven home: /net/home/osipovmi/apache-maven-3.8.2
Java version: 1.8.0_302, vendor: OpenJDK BSD Porting Team, runtime: /usr/local/openjdk8/jre
Default locale: de_DE, platform encoding: UTF-8
OS name: "freebsd", version: "12.2-stable", arch: "amd64", family: "unix"
[INFO] Scanning for projects...
[INFO]
[INFO] ---------------------------< com.test:test >----------------------------
[INFO] Building test 2
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- versions-maven-plugin:2.8.1:set (default-cli) @ test ---
[INFO] Local aggregation root: /net/home/osipovmi
[INFO] Processing change of com.test:test:2 -> 3
[INFO] Processing com.test:test
[INFO]     Updating project com.test:test
[INFO]         from version 2 to 3
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  5.197 s
[INFO] Finished at: 2021-09-07T12:56:26+02:00
[INFO] ------------------------------------------------------------------------
0

Will try MinGW...

michael-o · 2021-09-07T11:06:08Z

MinGW:

$ /c/Entwicklung/Programme/apache-maven-3.8.2/bin/mvn -V versions:set -DnewVersion=4 ; echo $?
Apache Maven 3.8.2 (ea98e05a04480131370aa0c110b8c54cf726c06f)
Maven home: C:\Entwicklung\Programme\apache-maven-3.8.2
Java version: 1.8.0_275, vendor: Azul Systems, Inc., runtime: C:\Program Files\Zulu\zulu-8\jre
Default locale: de_DE, platform encoding: Cp1252
OS name: "windows 10", version: "10.0", arch: "amd64", family: "windows"
[INFO] Scanning for projects...
[INFO]
[INFO] ---------------------------< com.test:test >----------------------------
[INFO] Building test 2
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- versions-maven-plugin:2.8.1:set (default-cli) @ test ---
[INFO] Local aggregation root: C:\Users\osipovmi
[INFO] Processing change of com.test:test:2 -> 4
[INFO] Processing com.test:test
[INFO]     Updating project com.test:test
[INFO]         from version 2 to 4
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  4.563 s
[INFO] Finished at: 2021-09-07T13:03:19+02:00
[INFO] ------------------------------------------------------------------------
1

Confirmed, but don't understand it yet.

michael-o · 2021-09-07T13:00:12Z

OK, here is the bug (swallowed exceptions) and it in Cygwin only. @gnodet needs to take a look.

Invalid handle...

michael-o · 2021-09-07T13:25:31Z

Note that happens only in MinGW. It does not fail in PowerShell as well as Windows Command Prompt.

michael-o · 2021-09-08T07:44:04Z

Debugged a bit further, MinGW (MSYS2) assignes handles 1112 to both stdout and stderr while PowerShell does 84 and 88. Something is either flawed in MSYS2 or in Jansi. So without @gnodet no progress.

gnodet · 2021-09-08T08:24:54Z

@michael-o which call throws the invalid handle exception exactly ?
The uninstall should not close the streams, so the handle should not become invalid...
The 1112 is the file descriptor of the underlying FileOutputStream ?

michael-o · 2021-09-08T12:14:54Z

When I enter flushBuffer() specifically out.write(buf, 0, count); fails with the invalid handle.
1112 is not the FD, but the handle. The fd is -1 in this case. I guess Windows uses the handle.

gnodet · 2021-09-09T13:35:38Z

I think the problem is located here:
https://github.com/codehaus-plexus/plexus-interactivity/blob/master/plexus-interactivity-api/src/main/java/org/codehaus/plexus/components/interactivity/DefaultOutputHandler.java#L53

The DefaultOutputHandler should not close the consoleWriter because it will also close the underlying System.out print stream. Those system streams are not supposed to be closed directly.

gnodet · 2021-09-09T16:11:30Z

See codehaus-plexus/plexus-interactivity#3 for a possible fix.

michael-o · 2021-09-09T18:28:51Z

Thank you @gnodet I will check tomorrow.

michael-o · 2021-09-09T19:04:38Z

Checked now. I can confirm @gnodet's analysis and it correct. Thank you for that. So this is not a bug in Maven or Jansi, they just trigger the bug in Plexus Interactivity.

michael-o · 2021-09-11T07:45:53Z

Plexus Interactivity 1.1 has been pushed to Central. Please update at the earliest convenience.

pigelvy · 2022-07-13T08:31:40Z

This issue has been fixed since r2.9.0 by #488.

To assert that, execute the following commands:

path/to/maven/3.8.2+/bin/mvn org.codehaus.mojo:versions-maven-plugin:2.8.1:set -DnewVersion=XYZ; echo $? returns 1
path/to/maven/3.8.2+/bin/mvn org.codehaus.mojo:versions-maven-plugin:2.9.0:set -DnewVersion=XYZ; echo $? returns 0

IMO, this issue could be closed.

slachiewicz closed this as completed Jul 13, 2022

imagejan mentioned this issue Jan 24, 2023

release-version.sh failing on Windows Git Bash scijava/scijava-scripts#49

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Bug] The command line 'mvn versions:set' returns exit code 1 with the last maven version 3.8.2 #483

[Bug] The command line 'mvn versions:set' returns exit code 1 with the last maven version 3.8.2 #483

RomainPruvostMHH commented Aug 24, 2021

slawekjaranowski commented Aug 24, 2021

RomainPruvostMHH commented Aug 24, 2021

slawekjaranowski commented Aug 25, 2021

RomainPruvostMHH commented Aug 25, 2021

slawekjaranowski commented Aug 25, 2021

RomainPruvostMHH commented Aug 26, 2021

slawekjaranowski commented Aug 26, 2021

slachiewicz commented Aug 26, 2021

raphaelNguyen commented Sep 3, 2021 •

edited

slawekjaranowski commented Sep 3, 2021

slawekjaranowski commented Sep 7, 2021

RomainPruvostMHH commented Sep 7, 2021

slawekjaranowski commented Sep 7, 2021

RomainPruvostMHH commented Sep 7, 2021

michael-o commented Sep 7, 2021

michael-o commented Sep 7, 2021 •

edited

michael-o commented Sep 7, 2021 •

edited

michael-o commented Sep 7, 2021

michael-o commented Sep 7, 2021

michael-o commented Sep 8, 2021

gnodet commented Sep 8, 2021

michael-o commented Sep 8, 2021

gnodet commented Sep 9, 2021

gnodet commented Sep 9, 2021

michael-o commented Sep 9, 2021

michael-o commented Sep 9, 2021

michael-o commented Sep 11, 2021

pigelvy commented Jul 13, 2022 •

edited

[Bug] The command line 'mvn versions:set' returns exit code 1 with the last maven version 3.8.2 #483

[Bug] The command line 'mvn versions:set' returns exit code 1 with the last maven version 3.8.2 #483

Comments

RomainPruvostMHH commented Aug 24, 2021

slawekjaranowski commented Aug 24, 2021

RomainPruvostMHH commented Aug 24, 2021

slawekjaranowski commented Aug 25, 2021

RomainPruvostMHH commented Aug 25, 2021

slawekjaranowski commented Aug 25, 2021

RomainPruvostMHH commented Aug 26, 2021

slawekjaranowski commented Aug 26, 2021

slachiewicz commented Aug 26, 2021

raphaelNguyen commented Sep 3, 2021 • edited

slawekjaranowski commented Sep 3, 2021

slawekjaranowski commented Sep 7, 2021

RomainPruvostMHH commented Sep 7, 2021

slawekjaranowski commented Sep 7, 2021

RomainPruvostMHH commented Sep 7, 2021

michael-o commented Sep 7, 2021

michael-o commented Sep 7, 2021 • edited

michael-o commented Sep 7, 2021 • edited

michael-o commented Sep 7, 2021

michael-o commented Sep 7, 2021

michael-o commented Sep 8, 2021

gnodet commented Sep 8, 2021

michael-o commented Sep 8, 2021

gnodet commented Sep 9, 2021

gnodet commented Sep 9, 2021

michael-o commented Sep 9, 2021

michael-o commented Sep 9, 2021

michael-o commented Sep 11, 2021

pigelvy commented Jul 13, 2022 • edited

raphaelNguyen commented Sep 3, 2021 •

edited

michael-o commented Sep 7, 2021 •

edited

michael-o commented Sep 7, 2021 •

edited

pigelvy commented Jul 13, 2022 •

edited