Bug 917619 - eclipse: Please add Maven depmaps for org.eclipse.jetty.orbit artifacts
Summary: eclipse: Please add Maven depmaps for org.eclipse.jetty.orbit artifacts
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: eclipse
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Krzysztof Daniel
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 917618
TreeView+ depends on / blocked
 
Reported: 2013-03-04 12:20 UTC by Mikolaj Izdebski
Modified: 2014-01-13 00:27 UTC (History)
9 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-03-05 09:54:20 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Proposed patch (1.88 KB, patch)
2013-03-04 12:23 UTC, Mikolaj Izdebski
no flags Details | Diff
Proposed patch (1.88 KB, patch)
2013-03-04 13:02 UTC, Mikolaj Izdebski
no flags Details | Diff

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. :-(


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