Bug 1194110

Summary: build-classpath fails for some jars
Product: [Fedora] Fedora Reporter: Zbigniew Jędrzejewski-Szmek <zbyszek>
Component: javapackages-toolsAssignee: Mikolaj Izdebski <mizdebsk>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 22CC: java-sig-commits, mat.booth, mizdebsk, msimacek, msrb
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-02-19 09:16:07 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 902086    

Description Zbigniew Jędrzejewski-Szmek 2015-02-19 03:34:17 UTC
Description of problem:
This came up in review of elasticsearch (#902086). The launcher script fails because build-classpath fails for some classes. It seems it is missing a dependency on xmvn:

$ /usr/bin/build-classpath com.carrotsearch:hppc
/usr/bin/build-classpath: Unable to execute xmvn-resolve.
/usr/bin/build-classpath: Make sure that XMvn is installed and M2_HOME is set correctly.

/usr/bin/build-classpath: error: Some specified jars were not found

$ sudo dnf install /usr/bin/xmvn-resolve
...
$ 
$ /usr/bin/build-classpath com.carrotsearch:hppc
/usr/share/java/hppc.jar

Version-Release number of selected component (if applicable):
javapackages-tools-4.4.0-2.fc23.noarch

Comment 1 Mikolaj Izdebski 2015-02-19 09:16:07 UTC
This is intentional. xmvn-resolve is optional dependency of bulid-classpath - typically build-classpath is used with JAR names as arguments (for example: hppc), but if you want to use it with artifact coordinates then you need to install xmvn-resolve yourself. This is to reduce dependencies for common use cases.

I've just documented this dependency upstream in manual pages: https://github.com/mizdebsk/javapackages/commit/979b7790

Comment 2 Zbigniew Jędrzejewski-Szmek 2015-02-19 13:04:52 UTC
Thanks, we'll add the dependency.