Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 327452 - Removing @noextend should generate API problem on MANIFEST.MF
Summary: Removing @noextend should generate API problem on MANIFEST.MF
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: API Tools (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 4.12 M1   Edit
Assignee: Vikas Chandra CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on:
Blocks: 546336
  Show dependency tree
 
Reported: 2010-10-11 10:29 EDT by Dani Megert CLA
Modified: 2019-04-11 09:33 EDT (History)
2 users (show)

See Also:


Attachments
Tentative fix (3.08 KB, patch)
2019-03-22 05:57 EDT, Vikas Chandra CLA
no flags Details | Diff
Fixed for noinstantiate as well (3.78 KB, patch)
2019-03-22 06:24 EDT, Vikas Chandra CLA
no flags Details | Diff
Updated fix (9.60 KB, patch)
2019-03-22 12:52 EDT, Vikas Chandra CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dani Megert CLA 2010-10-11 10:29:20 EDT
N20101010-2000

0. check out 'org.eclipse.ui.ide' from HEAD
1. change bundle version of 'org.eclipse.ui.ide' to 3.7.0
2. set API baseline to 3.6
3. open org.eclipse.ui.ide.OpenWithMenu
4. remove @noextend from class
5. save
==> no error on MANIFEST.MF

Expected: problem reported on MANIFEST.MF to increase minor version number.
Comment 1 Darin Wright CLA 2010-10-12 12:00:54 EDT
In this case, an API restriction has been relaxed (a class that was previously not extensible can now be extended). No new API has been added. Why should the minor version be increased? (the steps already say to increase it to 3.7.0).
Comment 2 Dani Megert CLA 2010-10-12 12:05:15 EDT
(In reply to comment #1)
> In this case, an API restriction has been relaxed (a class that was previously
> not extensible can now be extended). No new API has been added. Why should the
> minor version be increased?
Since I can now extend that class, more API is available to me. Same as if a new API got added: there, you can also say, that the API is more relaxed, but still, we increase the service number.
Comment 3 Darin Wright CLA 2010-10-12 12:13:19 EDT
In this case, don't you want to omit step 1 (which already manually increased the minor version)? i.e. the bundle version is 3.6.100, next you increase the API by removing the @noextend tag. The tooling should detect that the version should be increased to 3.7.0 form 3.6.100?
Comment 4 Dani Megert CLA 2010-10-12 12:16:15 EDT
(In reply to comment #3)
> In this case, don't you want to omit step 1 (which already manually increased
> the minor version)? i.e. the bundle version is 3.6.100, next you increase the
> API by removing the @noextend tag. The tooling should detect that the version
> should be increased to 3.7.0 form 3.6.100?
Mmh - I'm confused about my steps from comment 0. Let me check again and report back tomorrow.
Comment 5 Dani Megert CLA 2010-10-13 03:19:15 EDT
(In reply to comment #4)
> (In reply to comment #3)
> > In this case, don't you want to omit step 1 (which already manually increased
> > the minor version)? i.e. the bundle version is 3.6.100, next you increase the
> > API by removing the @noextend tag. The tooling should detect that the version
> > should be increased to 3.7.0 form 3.6.100?
> Mmh - I'm confused about my steps from comment 0. Let me check again and 
> report back tomorrow.

Sorry, the steps got copied from another bug. Step 1. needs to be removed.
Comment 6 Eclipse Genie CLA 2019-03-15 11:55:53 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 7 Dani Megert CLA 2019-03-21 11:35:11 EDT
Vikas, please check whether this is still an issue,.
Comment 8 Vikas Chandra CLA 2019-03-22 02:35:07 EDT
This still is a problem. Even a removal of @noimplement on interface is not forcing a minor version increase. However removal of @noreference works as I expected it to work.
Comment 9 Vikas Chandra CLA 2019-03-22 05:57:52 EDT
Created attachment 277961 [details]
Tentative fix

Pending items
1) Add messages
2) Add test cases
Comment 10 Vikas Chandra CLA 2019-03-22 06:24:49 EDT
Created attachment 277962 [details]
Fixed for noinstantiate as well

Pending items
1) Messages
2) Test cases
Comment 11 Vikas Chandra CLA 2019-03-22 12:52:28 EDT
Created attachment 277970 [details]
Updated fix
Comment 12 Dani Megert CLA 2019-03-22 12:53:31 EDT
(In reply to Vikas Chandra from comment #11)
> Created attachment 277970 [details] [diff]
> Updated fix
Since when did we switch back from Gerrit to text patches?
Comment 13 Eclipse Genie CLA 2019-03-24 07:35:22 EDT
New Gerrit change created: https://git.eclipse.org/r/139379
Comment 15 Vikas Chandra CLA 2019-04-09 00:24:40 EDT
verified in 
Version: 2019-06 (4.12)
Build id: I20190406-1800
Comment 16 Eclipse Genie CLA 2019-04-09 01:23:13 EDT
New Gerrit change created: https://git.eclipse.org/r/140267