Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 275316

Summary: [publisher] CU versions should match product, not host
Product: [Eclipse Project] Equinox Reporter: Andrew Niefer <aniefer>
Component: p2Assignee: Andrew Niefer <aniefer>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: aniefer, irbull, pascal
Version: 3.5Flags: pascal: review+
dj.houghton: review+
Target Milestone: 3.5 RC2   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
uses product version
none
updated patch includes pde.build none

Description Andrew Niefer CLA 2009-05-07 11:04:57 EDT
(some?) CUs are currently generated with a version that matches their host bundle.  These should instead get a version that matches the product.  This will allow a new version of the product to change configurations without changing versions of the included bundle.

See ConfigCUsAction.publishBundleCUs and BundlesAction.createBundleConfigurationUnit

Ian can you take a look?
Comment 1 Pascal Rapicault CLA 2009-05-08 20:44:19 EDT
The real issue here is that if a CU matches the IU version then it is not possible to ever deliver a new version of the CU.
Comment 2 Ian Bull CLA 2009-05-13 19:13:15 EDT
Created attachment 135713 [details]
uses product version

This uses the version passed to the ConfigCUsAction for the CUs. This version is the product version, in product publishing.

I also put a check to ensure the version has been set.  If not, it defaults to the bundle version.
Comment 3 Andrew Niefer CLA 2009-05-13 19:40:49 EDT
Slipping to RC2, I just realized the same applies to the default CUs generated by pde.build.
Comment 4 Ian Bull CLA 2009-05-19 19:23:21 EDT
Andrew, I'll bounce this over this over to you.  (Unless you want me to take a stab at the PDE Build side of this).
Comment 5 Andrew Niefer CLA 2009-05-21 11:27:59 EDT
Created attachment 136663 [details]
updated patch includes pde.build

Updated patch includes CUs generated by pde.build to set start levels.
Comment 6 DJ Houghton CLA 2009-05-21 16:39:43 EDT
Released.