Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 343804 - Update API versioning information for org.eclipse.wst.xml.xpath.(core|ui)
Summary: Update API versioning information for org.eclipse.wst.xml.xpath.(core|ui)
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.xpath (show other bugs)
Version: 3.3   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.3 RC1   Edit
Assignee: Jesper Moller CLA
QA Contact: Jesper Moller CLA
URL:
Whiteboard: PMC_approved
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-26 04:56 EDT by Jesper Moller CLA
Modified: 2011-05-13 13:25 EDT (History)
3 users (show)

See Also:
jesper: pmc_approved? (david_williams)
jesper: pmc_approved? (raghunathan.srinivasan)
jesper: pmc_approved? (naci.dai)
jesper: pmc_approved? (deboer)
jesper: pmc_approved? (neil.hauge)
kaloyan: pmc_approved+
jesper: pmc_approved? (cbridgha)
d_a_carver: review+


Attachments
Patch to update versioning information for XPath plugins where needed (13.62 KB, patch)
2011-04-28 03:01 EDT, Jesper Moller CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jesper Moller CLA 2011-04-26 04:56:14 EDT
A few subtle changes in org.eclipse.wst.xml.xpath.core and org.eclipse.wst.xml.xpath.ui requires a version bump from 1.1.101 to 1.2, and a few additions of @since JavaDoc markers.
Comment 1 Nitin Dahyabhai CLA 2011-04-28 00:44:55 EDT
Which subtle changes?  We're fast running out of time to do it if needed.
Comment 2 Jesper Moller CLA 2011-04-28 03:01:55 EDT
Created attachment 194232 [details]
Patch to update versioning information for XPath plugins where needed

Actually, I was going to wait to after the S build to submit this for PMC review, to not add noise, but here it is:

The fix is to add "@since" tags on new elements since the Helios baseline. In the org.eclipse.wst.xsl.xpath.code bundle, a minor version bump was needed, which was reflected in the org.eclipse.wst.xsl.xpath.ui manifest file as well.
In one instance, an API exception rule was appropropriate to not introduce deprecated code.

* Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such. 

This updates the versioning information to the API tooling build is free of errors in the shipped code. Since this is also the API baseline for WTP 3.3 I'm thinking this is as important as the compiled code.

* Is there a work-around? If so, why do you believe the work-around is insufficient? 

No work-around in the current tooling.

* How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added? 

Local build of API tooling against Helios baseline reveals no errors. No JUnit test applicable.

* Give a brief technical overview. Who has reviewed this fix? 

No review (yet)

* What is the risk associated with this fix?

The risk is very low, since only tooling and versioning information is changed. No actual run-time changes will occur.
Comment 3 Jesper Moller CLA 2011-04-28 03:03:55 EDT
Hi David, asking for review for the versioning tags.
Comment 4 David Carver CLA 2011-04-28 10:00:34 EDT
Only thing missing is to make sure that the POM version numbers are updated as well if you are changing the plugin version.
Comment 5 Jesper Moller CLA 2011-04-28 14:49:58 EDT
PMC review information:

* Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug"
(requested by an adopter) please document it as such. 

This updates the versioning information, so that the API tooling build is free of
errors in the shipped code. Since this is also the API baseline for WTP 3.3, I think it is important to get right.

* Is there a work-around? If so, why do you believe the work-around is
insufficient? 

No work-around in the current tooling.

* How has the fix been tested? Is there a test case attached to the bugzilla
record? Has a JUnit Test been added? 

Local build of API tooling against Helios baseline reveals no errors. No JUnit
test applicable.

* Give a brief technical overview. Who has reviewed this fix? 

David Carver reviewed this.

* What is the risk associated with this fix?

The risk is very low, since only tooling and versioning information is changed.
No actual run-time changes will occur, except for the minor version bump.

If the review is successfull, I'll commit and release as soon as M7 is declared.
Comment 6 Kaloyan Raev CLA 2011-04-29 02:03:55 EDT
Which Milestone do you target this bug? M7 or RC1?
Comment 7 Jesper Moller CLA 2011-04-29 03:16:41 EDT
(In reply to comment #6)
> Which Milestone do you target this bug? M7 or RC1?

Targetting for RC1 (didn't see those in the dropdown when I filed)
Comment 8 Jesper Moller CLA 2011-05-02 20:48:31 EDT
Committed and released