Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 40720 - JRE System Library -> Alternative Library Can not be accepted.
Summary: JRE System Library -> Alternative Library Can not be accepted.
Status: RESOLVED WONTFIX
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows 2000
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Martin Aeschlimann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-07-24 11:34 EDT by MG CLA
Modified: 2003-08-07 04:03 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description MG CLA 2003-07-24 11:34:16 EDT
Project->properties->Java Build Path-> Add Library -> JRE System Library ->
Alternative Library -> Finish.  The Alt lib is added to the libraries tab but
the OK button is not highlighted so I can't finalize that change.  The only
options are click cancel or click the close dialog which both undo changes.
Comment 1 Philipe Mulet CLA 2003-07-24 11:53:10 EDT
Weird, it works for me just fine.
Comment 2 Martin Aeschlimann CLA 2003-07-25 12:25:32 EDT
works for me too.
what's strange is that even I use the default JRE the alternate JRE options is
enabled but with an empty field.

moving to debug, suspect a strange container is added to the build path.
Comment 3 Darin Wright CLA 2003-07-25 12:47:54 EDT
If you're using M2, this is likely a side effect of bug 40673.
Comment 4 MG CLA 2003-07-25 13:20:22 EDT
Yes I am using e3m2.
Comment 5 Darin Wright CLA 2003-07-25 13:33:02 EDT
Please try the patch provided in bug 40673 to see if this solves the problem.
Comment 6 MG CLA 2003-07-25 14:54:29 EDT
I tried it same result.  I select the JRE system library, it shows up in the
java build path but the OK button is greyed out so I can not accept the change.  

From the project menu, select properties, java build path, libraries tab, add
library, JRE system library, next, alternative jre, select from the combo box,
finish.  The jre is now listed in the build path but the OK button to accept
this change is greyed out.
Comment 7 Darin Wright CLA 2003-07-28 10:20:58 EDT
Unable to reproduce the problem. What JRE/version are you adding to the 
buildpath? Do it's libraries appear correctly when you expand the JRE entry 
(after you add it to the buildpath)?
Comment 8 MG CLA 2003-07-28 12:33:49 EDT
The problem seems to be that I have JRE_LIB=x/rt.jar and using the add library
button I was adding JRE System library which also contained that rt.jar.

If I remove the JRE_LIB then I can accept the changes.  I am not sure how you
could make this obvious?  There must be some kind of check which prevents me
from accepting this change so maybe you could display a warning telling my why I
will be unable to accept it, or what I have to do in order to accept this change.
Comment 9 Darin Wright CLA 2003-07-28 12:39:09 EDT
Moving back to JDT-UI (see previous comment from bug reporter)
Comment 10 Dirk Baeumer CLA 2003-07-31 06:01:30 EDT
Martin, can you please comment.
Comment 11 Martin Aeschlimann CLA 2003-08-06 12:12:14 EDT
I can reproduce what you say but there is an error message in the header that
explains the problem. ('Duplicated entry '...rt.jar'')

I think it's good enough. What do you think?
Comment 12 MG CLA 2003-08-06 13:39:26 EDT
Oh you are right.  I never saw that little message...   probably just too
focused on why the Ok button was greyed out.

I don't think I would have ever thought to look at the title of the tab to
determine the source of my error.  

Could you change the background of the error text to a light red instead of the
grey.  That would definitely have caught my attention, they grey just seems to
blend that message in with everything else.
Comment 13 Martin Aeschlimann CLA 2003-08-06 13:44:50 EDT
The rendering of the error message is done by the desktop (Platform UI).
I agree with you that having the error on the top is strange.
Can you file a new PR against Platform UI?

Closing this one
Comment 14 MG CLA 2003-08-06 14:06:06 EDT
Moved to Bug 41209
Comment 15 Martin Aeschlimann CLA 2003-08-07 04:03:03 EDT
Thanks a lot!