Community
Participate
Working Groups
The 3 facet install delegates for EE 5 modules need to add the EAR Libraries classpath container. Previously this container was being added by another thread, but that logic changed with bug 200369. Because that bug was originally coded up for 1.5.5 where there were no EE 5 facet install delegates, the EE 5 facet install delegates were overlooked when checking this into the 2.x stream. It is surprising that no one noticed this until now (Rob found it).
Created attachment 79016 [details] patch This patch adds the EAR Libraries classpath container for the EE 5 EJB, Application Client, and Web facet install delegates.
I have tested this patch with the UI. Rob, Chuck, could you also look this patch over and test.
Works as expected (tested all Java EE 5 project types); thanks for generating this patch so quickly, Jason. Up'ing the priority since I feel this one must be fixed before 2.0.1 goes out.
Adding as a hotbug request; although there is a simple work around for this (just add the cp container via the JDT UI), users have never previously been required to manually add this cp container so the regression here is going to cause serious confusion for Java EE 5 users and, since the fix is low risk, seems like one to repair before 2.0.1 ships.
If possible, I think we should include this in 201; all users creating new EE 5 projects will encounter this problem if we don't include it in 201. Furthermore, this is a very simple and straight forward fix.
I tested this patch and it appears very safe, and is fixing an ugly problem. We need to add a junit to catch this in the future (Checking for the correct classpath)
code dropped to 2.0.1 and released for build.
