Bug 542432 - Adding a feature to the product should not add the version number
Summary: Adding a feature to the product should not add the version number
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.10   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: 4.11 M3   Edit
Assignee: Julian Honnen CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday, helpwanted, usability
: 543060 (view as bug list)
Depends on:
Blocks:
 
Reported: 2018-12-05 07:49 EST by Lars Vogel CLA
Modified: 2019-02-15 04:51 EST (History)
5 users (show)

See Also:


Attachments
Screenshot (26.65 KB, image/png)
2018-12-05 07:49 EST, Lars Vogel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2018-12-05 07:49:26 EST
Created attachment 276830 [details]
Screenshot

AFAIK it is good practice to not add the version number for features in the product as the build qualifier will regulary change. 

But if I use the "Add..." button in the product editor, the version number is added. 

I suggest to add new feature entries without version number in the editor.
Comment 1 Lars Vogel CLA 2019-01-08 10:50:23 EST
Julian, something for you?
Comment 2 Eclipse Genie CLA 2019-01-10 08:20:16 EST
New Gerrit change created: https://git.eclipse.org/r/134875
Comment 3 Lars Vogel CLA 2019-01-11 04:10:16 EST
Thanks, Julian.
Comment 5 Wim Jongman CLA 2019-01-11 09:05:28 EST
Hey guys I agree with the fact that we should have an easy way of clearing the version number but I think this implementation is a little bit too hasty.

There are many cases where you want to have a specific version number. Also if you leave the version to .qualifier then the build will fill in the specific version number. 

For example, this avoids the problem that Tycho unintentionally pulls in old plugins from the local .m2 directory. This is just one case where I see this failing but it does not look like you considered all the corner cases of the product build.

Please consider a version toggle instead like we have fixed the root toggle [1].

Also, the properties dialog should have a convenient way of restoring the version number.
Comment 6 Lars Vogel CLA 2019-01-11 09:35:25 EST
(In reply to Wim Jongman from comment #5)

> There are many cases where you want to have a specific version number. Also
> if you leave the version to .qualifier then the build will fill in the
> specific version number. 

For features a version range is supported. I think the whole scenario you are describing is validate for features which we have not change. For products it is a simple text match which is very limited so I typically recommend to not use feature versions in products.
Comment 7 Julian Honnen CLA 2019-01-11 09:41:30 EST
(In reply to Wim Jongman from comment #5)
> Also, the properties dialog should have a convenient way of restoring the
> version number.

How about adding content assist to the properties dialog? It could suggest available versions with current qualifier and ".qualifier".
Comment 8 Lars Vogel CLA 2019-01-11 10:11:54 EST
(In reply to Julian Honnen from comment #7)
> (In reply to Wim Jongman from comment #5)
> > Also, the properties dialog should have a convenient way of restoring the
> > version number.
> 
> How about adding content assist to the properties dialog? It could suggest
> available versions with current qualifier and ".qualifier".

AFAIK .qualifier is not supported for product configuration. Is that not true?
Comment 9 Julian Honnen CLA 2019-01-11 10:20:28 EST
(In reply to Lars Vogel from comment #8)
> AFAIK .qualifier is not supported for product configuration. Is that not
> true?

Seems like it is widely used:
https://github.com/search?q=.qualifier+extension%3Aproduct&type=Code&ref=advsearch&l=&l=
Comment 10 Wim Jongman CLA 2019-01-11 11:02:40 EST
I was looking in the EPP packages. I was convinced they heavily used version but they don't. 

However, they do use .qualifier for their own features.

https://github.com/eclipse/epp.packages/tree/master/packages
Comment 11 Lars Vogel CLA 2019-01-11 11:05:03 EST
(In reply to Wim Jongman from comment #10)
> I was looking in the EPP packages. I was convinced they heavily used version
> but they don't. 
> 
> However, they do use .qualifier for their own features.
> 
> https://github.com/eclipse/epp.packages/tree/master/packages

For own features it will work as both .qualifiers are replaced during the build.
Comment 12 Wim Jongman CLA 2019-01-11 11:12:23 EST
(In reply to Julian Honnen from comment #7)
> (In reply to Wim Jongman from comment #5)
> > Also, the properties dialog should have a convenient way of restoring the
> > version number.
> 
> How about adding content assist to the properties dialog? It could suggest
> available versions with current qualifier and ".qualifier".

+1. I agree with the version number blanking [1] but it would be nice if we have some means of restoring the version number in case needed. No one wants to type a version number.

Maybe a button next to the list "Set version number" that would just pick the latest known version number? Just brainstorming. I'm happy with any solution.

[1] https://bugs.eclipse.org/bugs/show_bug.cgi?id=543060
Comment 13 Wim Jongman CLA 2019-01-11 11:13:38 EST
*** Bug 543060 has been marked as a duplicate of this bug. ***
Comment 14 Vikas Chandra CLA 2019-02-15 04:49:32 EST
Can bug reporter or bug assignee please verify this defect for 4.11M3?