Skip to content

[STORM-2893] fix storm distribution build by using POSIX tar#2511

Merged
asfgit merged 1 commit intoapache:masterfrom
erikdw:STORM-2893
Jan 12, 2018
Merged

[STORM-2893] fix storm distribution build by using POSIX tar#2511
asfgit merged 1 commit intoapache:masterfrom
erikdw:STORM-2893

Conversation

@erikdw
Copy link
Contributor

@erikdw erikdw commented Jan 10, 2018

There is a known issue with the maven-assembly-plugin version 2.5+ where it fails with errors like the following:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:2.6:single (default) on project final-package: Execution default of goal org.apache.maven.plugins:maven-assembly-plugin:2.6:single failed: group id '906175167' is too big ( > 2097151 ). Use STAR or POSIX extensions to overcome this limit -> [Help 1]

This became an issue with change 84a4314 when the maven-assembly-plugin
was upgraded from version 2.2.2 to 2.6, as inherited from the apache pom version 18.

To fix this we set the assembly plugin's tarLongFileMode configuration setting to posix.

There is a known issue with the maven-assembly-plugin version 2.5+ where it fails with errors like the following:

```
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:2.6:single (default) on project final-package: Execution default of goal org.apache.maven.plugins:maven-assembly-plugin:2.6:single failed: group id '906175167' is too big ( > 2097151 ). Use STAR or POSIX extensions to overcome this limit -> [Help 1]
```

This became an issue with change 84a4314 when the maven-assembly-plugin
was upgraded from version 2.2.2 to 2.6, as inherited from the apache pom version 18.

To fix this we set the assembly plugin's `tarLongFileMode` configuration setting to `posix`.
Copy link
Contributor

@revans2 revans2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@srdo
Copy link
Contributor

srdo commented Jan 12, 2018

+1. Thanks @erikdw, merged to master.

@asfgit asfgit merged commit 08d7d8d into apache:master Jan 12, 2018
@erikdw
Copy link
Contributor Author

erikdw commented Jan 12, 2018

@revans2 & @srdo : thx for the quick review-and-merge!

@erikdw erikdw deleted the STORM-2893 branch January 12, 2018 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants