Bug 327817 - ecjsrc.zip should be ecjsrc.jar
Summary: ecjsrc.zip should be ecjsrc.jar
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.7 M3   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 315844
  Show dependency tree
 
Reported: 2010-10-14 14:34 EDT by Kim Moir CLA
Modified: 2010-10-28 14:48 EDT (History)
1 user (show)

See Also:

Attachments
patch (1.89 KB, patch)
2010-10-14 15:17 EDT, Kim Moir CLA 		no flags Details | Diff
Proposed fix (5.28 KB, patch)
2010-10-14 15:30 EDT, Olivier Thomann CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Kim Moir CLA 2010-10-14 14:34:20 EDT 
I'll attach a patch. This is in support of bug 315844.  The featuresandBundles publisher doesn't support jars in the zip format and it's better to be consistent and have both ecj and ecjsrc files in jar format.
Comment 1 Kim Moir CLA 2010-10-14 15:17:31 EDT 
Created attachment 180907 [details]
patch
Comment 2 Olivier Thomann CLA 2010-10-14 15:30:00 EDT 
Created attachment 180908 [details]
Proposed fix

Should not this patch be also done for compiler.tool and the jdt.core bundle scripts ?
Kim, could you please double-check it ?
Comment 3 Kim Moir CLA 2010-10-14 15:45:51 EDT 
Looks good to me Olivier, thanks
Comment 4 Olivier Thomann CLA 2010-10-14 20:38:00 EDT 
Released for 3.7M3.
Comment 5 Olivier Thomann CLA 2010-10-26 14:47:23 EDT 
Will be verified once compiler.apt has been built with the corresponding changes.
Comment 6 Olivier Thomann CLA 2010-10-28 14:48:22 EDT 
Verified with I20101027-1800.