Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 348095 - [publisher] new publisher bundle should be a jar not a folder
Summary: [publisher] new publisher bundle should be a jar not a folder
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: p2 (show other bugs)
Version: 3.7   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: Juno M1   Edit
Assignee: Pascal Rapicault CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 350076 (view as bug list)
Depends on:
Blocks: 349784
  Show dependency tree
 
Reported: 2011-06-02 11:59 EDT by Pascal Rapicault CLA
Modified: 2011-06-23 11:21 EDT (History)
2 users (show)

See Also:


Attachments
patch (630 bytes, patch)
2011-06-23 11:21 EDT, Andrew Niefer CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pascal Rapicault CLA 2011-06-02 11:59:45 EDT
When I merged the publisher changes back in head, I overlooked the detail on the format on disk and it resulted that it is a folder instead of being a jar like the publisher was. 
This seems to cause issues to the Virgo build that can't deal with the inner jars.
Given that this was an accident, I'll fix it.
Comment 1 Katya Stoycheva CLA 2011-06-03 03:39:28 EDT
Virgo build issues are actually caused by the fact that classes in the inner jar cannot be referred to from outside the publisher jar/folder at compile time. So Virgo components which use publisher classes fail to compile.
In that sense having jar instead of folder won't solve the issues if the inner jar remains as it is. 
It would be helpful if the new publisher jar has similar file structure to 3.7M6 org.eclipse.equinox.p2.publisher_1.2.0.v20110228.jar for example (directly containing class files rather than pdepublisher.jar).
Comment 2 Pascal Rapicault CLA 2011-06-12 00:25:10 EDT
Fixed in HEAD.

### Eclipse Workspace Patch 1.0
#P org.eclipse.equinox.p2.publisher.eclipse
Index: META-INF/MANIFEST.MF
===================================================================
RCS file: /cvsroot/rt/org.eclipse.equinox/p2/bundles/org.eclipse.equinox.p2.publisher.eclipse/META-INF/MANIFEST.MF,v
retrieving revision 1.2
diff -u -r1.2 MANIFEST.MF
--- META-INF/MANIFEST.MF	1 Apr 2011 17:32:23 -0000	1.2
+++ META-INF/MANIFEST.MF	12 Jun 2011 04:24:39 -0000
@@ -2,8 +2,6 @@
 Bundle-ManifestVersion: 2
 Bundle-Name: %bundleName
 Bundle-SymbolicName: org.eclipse.equinox.p2.publisher.eclipse;singleton:=true
-Bundle-ClassPath: pdepublishing.jar
-Eclipse-BundleShape: dir
 Bundle-Version: 1.0.0.qualifier
 Bundle-Activator: org.eclipse.pde.internal.publishing.Activator
 Bundle-ActivationPolicy: lazy
Comment 3 DJ Houghton CLA 2011-06-22 12:53:07 EDT
*** Bug 350076 has been marked as a duplicate of this bug. ***
Comment 4 Andrew Niefer CLA 2011-06-23 11:05:26 EDT
This bundle is still in bad shape.  It is still a directory, but instead of the nested  pdepublishing.jar, the class files are directly on disk.

We likely need to update the feature and/or put Eclipse-BundleShape:jar
Comment 5 Andrew Niefer CLA 2011-06-23 11:21:01 EDT
Created attachment 198474 [details]
patch
Comment 6 Andrew Niefer CLA 2011-06-23 11:21:46 EDT
Ok, this should do it.