Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 331806 - Missing names and licenses for source features
Summary: Missing names and licenses for source features
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Releng (show other bugs)
Version: 4.1   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 4.1 M7   Edit
Assignee: Kim Moir CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 340903 342027 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-12-03 13:47 EST by John Arthorne CLA
Modified: 2011-04-20 17:06 EDT (History)
4 users (show)

See Also:


Attachments
Screen shot (25.52 KB, image/png)
2010-12-03 13:49 EST, John Arthorne CLA
no flags Details
patch (2.06 KB, patch)
2010-12-03 15:59 EST, Kim Moir CLA
no flags Details | Diff
mylyn/context/zip (614 bytes, application/octet-stream)
2010-12-03 16:00 EST, Kim Moir CLA
no flags Details
patch (1.31 KB, patch)
2010-12-14 08:38 EST, Kim Moir CLA
no flags Details | Diff
patch (2.70 KB, patch)
2011-04-13 17:11 EDT, Kim Moir CLA
no flags Details | Diff
patch (2.23 KB, patch)
2011-04-13 17:21 EDT, Kim Moir CLA
no flags Details | Diff
patch (3.71 KB, patch)
2011-04-18 14:20 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 John Arthorne CLA 2010-12-03 13:47:54 EST
4.1 stream build: I20101202-2256

The source features are missing names and licenses. The result is that you get a goofy license with text "license" when you attempt to upgrade to this build.
Comment 1 John Arthorne CLA 2010-12-03 13:49:12 EST
Created attachment 184499 [details]
Screen shot
Comment 2 John Arthorne CLA 2010-12-03 13:52:46 EST
I have no idea where these are defined but I'm assuming you do, Kim.
Comment 3 Kim Moir CLA 2010-12-03 15:59:59 EST
Created attachment 184513 [details]
patch

Don't think that this is related, but  as an aside Andrew mentioned that your features now include the new Lucene bundle.  You need to update the features to include them and generate the source for the new versions.
Comment 4 Kim Moir CLA 2010-12-03 16:00:10 EST
Created attachment 184514 [details]
mylyn/context/zip
Comment 5 Kim Moir CLA 2010-12-14 08:38:48 EST
Created attachment 185128 [details]
patch
Comment 6 Kim Moir CLA 2010-12-14 08:46:14 EST
I've released the patch and tagged the sdk feature for the next 4.1 build.
Comment 7 Kim Moir CLA 2011-02-17 15:57:06 EST
Andrew, are there any plans to use shared licenses in the 4.1 stream like we do in the 3.7 stream?  Not sure why this error is occurring in 4.1.
Comment 8 Kim Moir CLA 2011-04-06 19:54:13 EDT
*** Bug 342027 has been marked as a duplicate of this bug. ***
Comment 9 Kim Moir CLA 2011-04-13 16:36:01 EDT
So I looked at this today and the problem is that the 3.7 source features that the 4.1 build just consumes from the 3.7 build repos are missing files.  For instance, they are missing feature.properties file which defines the feature name and that's why it shows up as blank in the ui.  Looking at the 4.1 map files there are a number of binary source features consumed, for instance

feature@org.eclipse.jdt=p2IU,id=org.eclipse.jdt.feature.jar,repository=ECLIPSE_PLATFORM_REPO
feature@org.eclipse.pde=p2IU,id=org.eclipse.pde.feature.jar,repository=ECLIPSE_PLATFORM_REPO
feature@org.eclipse.cvs=p2IU,id=org.eclipse.cvs.feature.jar,repository=ECLIPSE_PLATFORM_REPO
plugin@org.eclipse.cvs=p2IU,id=org.eclipse.cvs,repository=ECLIPSE_PLATFORM_REPO
feature@org.eclipse.help=p2IU,id=org.eclipse.help.feature.jar,repository=ECLIPSE_PLATFORM_REPO
feature@org.eclipse.equinox.p2.user.ui=p2IU,id=org.eclipse.equinox.p2.user.ui.feature.jar,repository=ECLIPSE_PLATFORM_REPO
feature@org.eclipse.equinox.p2.core.feature=p2IU,id=org.eclipse.equinox.p2.core.feature.feature.jar,repository=ECLIPSE_PLATFORM_REPO
feature@org.eclipse.equinox.p2.extras.feature=p2IU,id=org.eclipse.equinox.p2.extras.feature.feature.jar,repository=ECLIPSE_PLATFORM_REPO
feature@org.eclipse.equinox.p2.rcp.feature=p2IU,id=org.eclipse.equinox.p2.rcp.feature.feature.jar,repository=ECLIPSE_PLATFORM_REPO

However, I don't seem the associated source features. You can't just generate source features from binary features, you need the actually source artifacts to do that.  Andrew, shouldn't the 3.7 source features that are reconsumed by the 4.1 build just be added to the 4.1 map files?
Comment 10 Andrew Niefer CLA 2011-04-13 16:46:34 EDT
*** Bug 340903 has been marked as a duplicate of this bug. ***
Comment 11 Andrew Niefer CLA 2011-04-13 16:54:30 EDT
Yeah we should add them to the map file, that could be enough to fix this
Comment 12 Kim Moir CLA 2011-04-13 17:11:25 EDT
Created attachment 193198 [details]
patch
Comment 13 Kim Moir CLA 2011-04-13 17:21:33 EDT
Created attachment 193200 [details]
patch
Comment 14 Kim Moir CLA 2011-04-13 17:22:40 EDT
Okay, I've updated the maps.
Comment 15 Kim Moir CLA 2011-04-18 13:56:16 EDT
Hmmm, it seems like the same problem still exists in 4.1 I20110415-1030.  Andrew, should I have to tweak the build.properties of the 4.1 sdk feature to stop generating the source features?
Comment 16 Andrew Niefer CLA 2011-04-18 14:20:51 EDT
Created attachment 193513 [details]
patch

It looks like the Fetch phase will skip fetching features for which there is a generate.feature@ properties in the containing feature's build.properties.  So, yes it looks like we should remove those generate.feature properties for any source feature that we don't need to recreate.

Here is a patch that removes the following from org.eclipse.sdk
generate.feature@org.eclipse.jdt.source
generate.feature@org.eclipse.pde.source
generate.feature@org.eclipse.pde.p2.source
generate.feature@org.eclipse.cvs.source
generate.feature@org.eclipse.help.source

and from org.eclipse.platform:
generate.feature@org.eclipse.equinox.p2.user.ui.source

The previous change to the map files is still required.
Comment 17 Kim Moir CLA 2011-04-18 14:57:29 EDT
Thanks Andrew, patch released and tagged for next i-build.
Comment 18 Kim Moir CLA 2011-04-20 17:06:20 EDT
Verified that this is fixed in 4.1 build I20110419-2200.