Bug 100049 – kde-meta.eclass + java

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Thomas Matthijs (RETIRED) 2005-07-23 11:06:49 UTC]

virtual/jdk provides also provide virtual/jdk
you won't be able to compile anything with a virtual/jre

please apply this patch

[Comment 1 Thomas Matthijs (RETIRED) 2005-07-23 11:07:23 UTC]

Created attachment 64142 [details, diff]
kde-meta + java

[Comment 2 Thomas Matthijs (RETIRED) 2005-07-23 12:41:33 UTC]

note: patch still has double ending ) )

[Comment 3 Thomas Matthijs (RETIRED) 2005-07-23 14:46:52 UTC]

noticed some comments in the kde ebuilds with 'java' and 'stupid' in the same 
sentance:p

is there anything using java besides kdejava/qtjava you know off? 
you can always poke me on irc if you're having java questions/issues

[Comment 4 Gregorio Guidi (RETIRED) 2005-07-26 07:10:04 UTC]

Created attachment 64351 [details, diff]
kde-meta-java.patch

In fact there's nothing in kdebase that needs java to compile (qtjava/kdejava
are dealt separately), all it needs is the "java" executable at runtime.
So the appropriate fix probably is the one above, I will commit it if it's ok.

[Comment 5 Thomas Matthijs (RETIRED) 2005-07-26 07:19:06 UTC]

looks good too me, except it is prefered to dep on the min version it needs of 
virtual/{jre,jdk}
if unsure safest bet is just putting >=1.4

[Comment 6 Dan Armak (RETIRED) 2005-07-27 04:37:24 UTC]

OK with me. The original code in kde-meta.eclass was due to the fact that for 
some reason I couldn't satisfy myself that the only thing in kdebase using 
java was konqueror + nsplugins. 
Looking at it again it seems clear that this is the case, so the dep should 
be moved outside the eclass and into these two ebuilds. OK? 

[Comment 7 Gregorio Guidi (RETIRED) 2005-07-27 11:47:33 UTC]

Ok, feel free to commit. 

[Comment 8 Gregorio Guidi (RETIRED) 2005-07-29 01:36:57 UTC]

I saw that konqueror now has the java dependency, I removed the java code from 
kde-meta.eclass so this should be fixed now.