Skip to content

[MEAR-182] skipClassPathModification in case of undefined Class-Path entry of MANFIEST.mf#28

Closed
mabrarov wants to merge 1 commit intoapache:masterfrom
mabrarov:MEAR-182_undefined_Class-Path
Closed

[MEAR-182] skipClassPathModification in case of undefined Class-Path entry of MANFIEST.mf#28
mabrarov wants to merge 1 commit intoapache:masterfrom
mabrarov:MEAR-182_undefined_Class-Path

Conversation

@mabrarov
Copy link
Copy Markdown
Contributor

@mabrarov mabrarov commented Dec 6, 2020

[MEAR-182] - Don't add MANFIEST.mf Class-Path entry for EAR module if skipClassPathModification is true and original MANFIEST.mf of the EAR module artifact doesn't contain Class-Path entry. Refer to that comment in MEAR-182 for description of the bug which this pull request fixes.

@mabrarov mabrarov force-pushed the MEAR-182_undefined_Class-Path branch from dd780f1 to 8ecc190 Compare December 6, 2020 18:11
… skipClassPathModification is true and original MANFIEST.mf of EAR module artifact doesn't contain Class-Path entry.
@mabrarov mabrarov force-pushed the MEAR-182_undefined_Class-Path branch from 8ecc190 to 30fbe32 Compare December 6, 2020 18:13
@mabrarov
Copy link
Copy Markdown
Contributor Author

mabrarov commented Dec 6, 2020

@hboutemy, it would be nice to include this PR and PR #22 into 3.2.0 release of Maven EAR Plugin.

@hboutemy
Copy link
Copy Markdown
Member

MEAR-182 https://issues.apache.org/jira/browse/MEAR-182 is an old issue closed for release 2.10
please create a new issue (we'll link it to MEAR-182), and update the commit (with rebasing, please, to have a simpler Git history)

@mabrarov
Copy link
Copy Markdown
Contributor Author

Declining in favor of pull request #29 with fixed source branch name.

@mabrarov mabrarov closed this Dec 20, 2020
@jira-importer
Copy link
Copy Markdown

Resolve #359

1 similar comment
@jira-importer
Copy link
Copy Markdown

Resolve #359

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.

3 participants