Bug 955715 - maven-assembly-plugin: spurious dependencies
Summary: maven-assembly-plugin: spurious dependencies
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: maven-assembly-plugin
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Mikolaj Izdebski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-04-23 15:26 UTC by Florian Weimer
Modified: 2013-06-14 14:01 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-06-13 13:13:45 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
pom.diff (780 bytes, patch)
2013-04-23 15:26 UTC, Florian Weimer
no flags Details | Diff

Description Florian Weimer 2013-04-23 15:26:46 UTC
Created attachment 739063 [details]
pom.diff

I've built maven-assembly-plugin with the attached patch, removing the dependencies on saxpath and jaxen, and the build succeeded.  Are these dependencies really still needed?

Comment 1 Stanislav Ochotnicky 2013-04-24 06:25:39 UTC
These deps would be required for tests. But the tests are disabled anyway because of easymock incompatibilities. Tests require rework or indeed those BRs can be removed

Comment 2 Mikolaj Izdebski 2013-06-13 13:13:45 UTC
(In reply to Florian Weimer from comment #0)
> I've built maven-assembly-plugin with the attached patch, removing the
> dependencies on saxpath and jaxen, and the build succeeded.  Are these
> dependencies really still needed?

%mvn_build is called with -f argument.  This, among other things, causes XMvn to remove all dependencies with scope "test" from the POM during package build.

There are 2 POM files installed. One in /usr/share/maven-effective-poms/ and the other in /usr/share/maven-poms/.  The second POM is not used for anything.  Grepping the first POM for jaxen and saxpath will confirm that these dependencies are indeed removed automatically during package build.

Closing as NOTABUG.

Comment 3 Florian Weimer 2013-06-14 14:01:54 UTC
Actually, this was fixed in 2.4-6β€”and I was looking at a Git checkout of 2.4-5. 8-)


Note You need to log in before you can comment on or make changes to this bug.