As stated in the current jedit-4.2-r2.ebuild, jedit-4.2 currently does not compile with jdk-1.6 (because of ambigous references to a class named SplashScreen which appeared in jdk-1.6 i think). I tried to fix this issue and will now attach two patches : the first to replace ambigous references by explicite ones, the second makes the ebuild use the first and updates DEPEND. They both seem to work for me.
Created attachment 128847 [details, diff] jdk-1.6 compat patch
Created attachment 128849 [details, diff] jedit-4.2-r2 update patch
(In reply to comment #1) > Created an attachment (id=128847) [edit] > jdk-1.6 compat patch > Please submit this to jedit upstream and post the url back here. This way we are not stuck maintaining this patch.
(In reply to comment #3) > Please submit this to jedit upstream and post the url back here. Well i can try if you insist, but i am quiet sure that they will ignore my request as they already fixed this a long time ago by tweaking import statements [1]. Their answer will be "use jdk-1.4 or jdk1.5 to compile jedit-4.2, or go with jedit-4.3pre10", don't you think ? 1. http://jedit.svn.sourceforge.net/viewvc/jedit/jEdit/trunk/org/gjt/sp/jedit/GUIUtilities.java?r1=5317&r2=5345
(In reply to comment #4) > (In reply to comment #3) > > Please submit this to jedit upstream and post the url back here. > > Well i can try if you insist, but i am quiet sure that they will ignore my > request as they already fixed this a long time ago by tweaking import > statements [1]. Guess we didn't know that, we wouldn't ask to submit your patch upstream otherwise. Of course, looking if the problem is already fixed in CVS/betas before submitting is wise, glad you did. > Their answer will be "use jdk-1.4 or jdk1.5 to compile jedit-4.2, or go with > jedit-4.3pre10", don't you think ? Sure we don't expect them to fix 4.2, the point was so that we don't have to keep patching future versions. Seems we won't then, fine.
4.3pre11 was added some time ago which doesn't have this problem anymore. Anyway, I've also applied your patch to 4.2-r2 because it's current stable. Thanks!
