First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 189815
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Java team <java@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Yoann Pannier <gentoo-bugs.e@umsar.org>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
jedit-4.2-jdk-1.6.patch jdk-1.6 compat patch patch Yoann Pannier 2007-08-22 10:04 0000 805 bytes Details | Diff
jedit-4.2-r2.patch jedit-4.2-r2 update patch patch Yoann Pannier 2007-08-22 10:05 0000 653 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 189815 depends on: Show dependency tree
Bug 189815 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

Description:   Opened: 2007-08-22 10:03 0000 
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.

------- Comment #1 From Yoann Pannier 2007-08-22 10:04:30 0000 ------- 
Created an attachment (id=128847) [edit]
jdk-1.6 compat patch

------- Comment #2 From Yoann Pannier 2007-08-22 10:05:12 0000 ------- 
Created an attachment (id=128849) [edit]
jedit-4.2-r2 update patch

------- Comment #3 From Petteri Räty 2007-08-22 10:05:30 0000 ------- 
(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.

------- Comment #4 From Yoann Pannier 2007-08-22 10:47:11 0000 ------- 
(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

------- Comment #5 From Vlastimil Babka (Caster) 2007-08-22 11:06:42 0000 ------- 
(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.

------- Comment #6 From Vlastimil Babka (Caster) 2007-12-20 10:41:32 0000 ------- 
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!
First Last Prev Next    No search results available      Search page      Enter new bug