Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug
Bug#: 183849
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: William L. Thomson Jr. (RETIRED) <wltjr@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Petteri Räty <betelgeuse@gentoo.org>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
tomcat.patch patch text/plain Vlastimil Babka (Caster) 2007-07-01 17:19 0000 1.10 KB Details
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 183849 depends on: Show dependency tree
Bug 183849 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-07-01 16:00 0000 
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 From William L. Thomson Jr. (RETIRED) 2007-07-01 17:07:08 0000 ------- 
(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 From Vlastimil Babka (Caster) 2007-07-01 17:18:05 0000 ------- 
Aw, going back to || () deps when split-ant is stable? Please no, we want to
get rid of ant-tasks one day...

------- Comment #3 From Vlastimil Babka (Caster) 2007-07-01 17:19:14 0000 ------- 
Created an attachment (id=123551) [details]
patch

Like this.

WANT_ANT_TASKS=ant-trax will both bring the needed dep and prevent classpath
pollution

------- Comment #4 From Vlastimil Babka (Caster) 2007-07-01 17:20:56 0000 ------- 
btw 6.0 is also done the way I proposed, and it's right

------- Comment #5 From William L. Thomson Jr. (RETIRED) 2007-07-01 17:31:10 0000 ------- 
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 :)
Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug