Bug 1676319

Summary:	jgit: missing javapackage-tools dependency (for /bin/build-classpath)
Product:	[Fedora] Fedora	Reporter:	Todd Zullinger <tmz>
Component:	eclipse-jgit	Assignee:	Mat Booth <mat.booth>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	akurtako, andjrobins, eclipse-sig, jerboaa, mat.booth, rgrunber
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	eclipse-jgit-5.2.0-5.fc30	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2019-02-12 13:20:28 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:	1676296

Description Todd Zullinger 2019-02-12 01:01:04 UTC

Installing jgit on a rawhide system without the '*modular*' repositories installed results in a broken jgit.  This does require the most recent changes in koji.  I ran into it while building git, which requires jgit to run some compatibility tests.

The result is:

  $ jgit -h
  /usr/bin/jgit: line 63: build-classpath: command not found
  Error: Could not find or load main class org.eclipse.jgit.pgm.Main

I filed a PR as well, but wanted to open a ticket in case others run into this.  I'm not sure if there's more fallout from the changes which have caused javapackage-tools to not be pulled in by jgit.  I think this is due to a large number of java packages moving to modules, so there might be a better place to fix things.

Though since the jgit wrapper script requires build-classpath it seems reasonable to have a hard requirement on javapackages-tools rather than relying on indirect deps.

Comment 1 Todd Zullinger 2019-02-12 01:03:32 UTC

Oh, the PR I filed is: https://src.fedoraproject.org/rpms/eclipse-jgit/pull-request/1 (which will need an updated commit message before merging, if the fix is agreeable).

Comment 2 Mat Booth 2019-02-12 13:20:28 UTC

Thanks, I merged and started a build: https://koji.fedoraproject.org/koji/taskinfo?taskID=32762737