Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 183849

Summary: www-servers/tomcat-5.5-r6 doesn't DEPEND on ant-trax
Product: Gentoo Linux Reporter: Petteri Räty (RETIRED) <betelgeuse>
Component: [OLD] ServerAssignee: William L. Thomson Jr. (RETIRED) <wltjr>
Status: RESOLVED FIXED    
Severity: normal CC: java
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: patch

Description Petteri Räty (RETIRED) gentoo-dev 2007-07-01 16:00:19 UTC
mirja@tinkkis ~ $ grep ant-trax /var/cache/edb/dep/usr/portage/www-servers/tomcat-5.5.23-r6
mirja@tinkkis ~ $

mirja@tinkkis /usr/portage/www-servers/tomcat $ grep ant-trax *
tomcat-5.5.23-r1.ebuild:#WANT_ANT_TASKS="ant-trax"
tomcat-5.5.23-r1.ebuild:        || ( dev-java/ant-trax dev-java/ant-tasks ) "
tomcat-5.5.23-r1.ebuild:        ANT_TASKS="ant-trax" eant ${antflags}
tomcat-5.5.23-r6.ebuild:#WANT_ANT_TASKS="ant-trax"
tomcat-5.5.23-r6.ebuild:        ANT_TASKS="ant-trax" eant ${antflags}
tomcat-6.0.13-r3.ebuild:WANT_ANT_TASKS="ant-trax"

the DEPEND || atom was dropped at some point but while doing that the WANT_ANT_TASKS for ant-trax was not uncommented.
Comment 1 William L. Thomson Jr. (RETIRED) gentoo-dev 2007-07-01 17:07:08 UTC
(In reply to comment #0)
>
> the DEPEND || atom was dropped at some point but while doing that the
> WANT_ANT_TASKS for ant-trax was not uncommented.

Yeah I was going to switch back to using the automatic stuff provided by the new split ant. However there was some class path pollution problems of the past. Not sure if they still remain. So wasn't really able to switch to the new stuff, and left the ebuild in limbo between the two ;) 

Fixed ebuild and committed to tree. Thanks for reporting.

Comment 2 Vlastimil Babka (Caster) (RETIRED) gentoo-dev 2007-07-01 17:18:05 UTC
Aw, going back to || () deps when split-ant is stable? Please no, we want to get rid of ant-tasks one day...
Comment 3 Vlastimil Babka (Caster) (RETIRED) gentoo-dev 2007-07-01 17:19:14 UTC
Created attachment 123551 [details]
patch

Like this.

WANT_ANT_TASKS=ant-trax will both bring the needed dep and prevent classpath pollution
Comment 4 Vlastimil Babka (Caster) (RETIRED) gentoo-dev 2007-07-01 17:20:56 UTC
btw 6.0 is also done the way I proposed, and it's right
Comment 5 William L. Thomson Jr. (RETIRED) gentoo-dev 2007-07-01 17:31:10 UTC
Well 6.0.x does not have admin webapp. Which is the problem 5.5.x had that caused the custom ANT_TASKS stuff to be introduced in the first place. Although I had already started going the direction of uncommenting WANT_ANT_TASKS and removing all the stuff it adds automatically.  Just committed that, so we should be good to go now. Would have committed sooner, but was bogged down in conversation :( While the bug got reopened. 

Oh well. Interesting how it was missed in the last 5 revisions :)