Bug 917619

Summary: eclipse: Please add Maven depmaps for org.eclipse.jetty.orbit artifacts
Product: [Fedora] Fedora Reporter: Mikolaj Izdebski <mizdebsk>
Component: eclipseAssignee: Krzysztof Daniel <kdaniel>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: akurtako, andjrobins, jerboaa, kdaniel, mbenitez, msrb, overholt, rgrunber, swagiaal
Target Milestone: ---Keywords: Patch, Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-03-05 09:54:20 UTC Type: ---
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: 917618    
Attachments:
Description Flags
Proposed patch
none
Proposed patch none

Description Mikolaj Izdebski 2013-03-04 12:20:16 UTC
Description of problem:
Eclipse relocated various artifacts under their own groupId -
org.eclipse.jetty.orbit. Please additional depmaps to appropriate
artifacts in eclipse package.

The following artifacts need to have depmaps added with groupId
org.eclipse.jetty.orbit:
  * org.eclipse.jdt.core

For more information on how to add extra mappings see Java packaging
guidelines[1], especially the part describing %add_maven_depmap macro[3].

Version-Release number of selected component (if applicable):
4.3.0-0.21.git3fd9eca

Additional info:
[1] http://fedoraproject.org/wiki/Packaging:Java
[2] http://fedoraproject.org/wiki/Packaging:Guidelines
[3] http://fedoraproject.org/wiki/Packaging:Java#add_maven_depmap_macro

Comment 1 Mikolaj Izdebski 2013-03-04 12:23:04 UTC
Created attachment 704931 [details]
Proposed patch

Comment 2 Mikolaj Izdebski 2013-03-04 13:02:18 UTC
Created attachment 704960 [details]
Proposed patch

Comment 3 Krzysztof Daniel 2013-03-04 14:50:22 UTC
The *orbit* names are allowed for names that are not managed by Eclipse foundation, but are from other foundations and are used by Eclipse.

The JDT is a core Eclipse bundle, and therefore it should not be mapped to org.eclipse.jetty.orbit: .

Comment 4 Mikolaj Izdebski 2013-03-04 15:17:09 UTC
Yet eclipse packages (like Jetty 9 RC3, released 3 days ago) require this depmap.

./jetty-9.0.0.RC3/pom.xml-      <dependency>
./jetty-9.0.0.RC3/pom.xml:        <groupId>org.eclipse.jetty.orbit</groupId>
./jetty-9.0.0.RC3/pom.xml-        <artifactId>org.eclipse.jdt.core</artifactId>
./jetty-9.0.0.RC3/pom.xml-        <version>3.7.1</version>
./jetty-9.0.0.RC3/pom.xml-      </dependency>

Please reconsider adding it.

Comment 5 Krzysztof Daniel 2013-03-05 09:26:41 UTC
reopening for investigation.

Comment 6 Krzysztof Daniel 2013-03-05 09:54:20 UTC
Released the rebased version of a patch. Due to the lack of central Eclipse maven repository, projects started to create their own poms and names. There may be more mappings needed soon. :-(