Bugzilla Bug 45402

The ebuilds for 
   dev-java/commons-validator, 
   dev-java/struts-legacy and
   dev-java/struts

use the statement

  echo "commons-logging.jar=`java-config --classpath=commons-logging`" \
       | sed s/.*:// >> build.properties

The sed call deletes everything up to the last colon, including the "commons-logging.jar=" part. This is not intended and causes a faulty build.properties file, which in turn causes ant to fail.

Changing the above line to

  echo "commons-logging.jar=\
`java-config --classpath=commons-logging | sed s/.*://`" >> build.properties

should make it work as expected.

BTW:
As I understand the ebuild-HOWTO, sys-apps/sed should be added to the build dependencies as the ebuild uses it.
Furthermore, the runtime dependencies should be the build dependencies without ant, jikes and junit (if used).
Commons-validator doesn't run with just an JRE but needs commons-collections, for example.


I'm using ant-1.5.4.

Portage 2.0.50-r1 (default-x86-1.4, gcc-3.3.2, glibc-2.3.2-r9, 2.4.20-gentoo-r9)
=================================================================
System uname: 2.4.20-gentoo-r9 i686 AMD Athlon(TM) XP 2200+
Gentoo Base System version 1.4.3.13
ccache version 2.3 [enabled]
Autoconf: sys-devel/autoconf-2.58-r1
Automake: sys-devel/automake-1.7.7
ACCEPT_KEYWORDS="x86"
AUTOCLEAN="yes"
CFLAGS="-march=athlon-xp -O3 -pipe -fomit-frame-pointer"
CHOST="i686-pc-linux-gnu"
COMPILER="gcc3"
CONFIG_PROTECT="/etc /etc/tomcat /usr/X11R6/lib/X11/xkb /usr/kde/2/share/config /usr/kde/3.1/share/config /usr/kde/3.2/share/config /usr/kde/3/share/config /usr/share/config /usr/share/texmf/dvipdfm/config/ /usr/share/texmf/dvips/config/ /usr/share/texmf/tex/generic/config/ /usr/share/texmf/tex/platex/config/ /usr/share/texmf/xdvi/ /var/qmail/control"
CONFIG_PROTECT_MASK="/etc/gconf /etc/terminfo /etc/env.d"
CXXFLAGS="-march=athlon-xp -O3 -pipe -fomit-frame-pointer"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoaddcvs ccache sandbox"
GENTOO_MIRRORS="ftp://sunsite.informatik.rwth-aachen.de/pub/Linux/gentoo ftp://ftp.uni-erlangen.de/pub/mirrors/gentoo http://gentoo.tiscali.nl/gentoo/"
MAKEOPTS="-j2"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/local/portage"
SYNC="rsync://rsync.europe.gentoo.org/gentoo-portage"
USE="X aalib acpi alsa apm avi berkdb bonobo cdr clamav crypt cups dga directfb doc dv dvb dvd emacs encode esd faad fam ffmpeg flash foomaticdb gd gd-external gdbm gif gimpprint gphoto2 gpm gtk gtk2 gtkhtml guile imap imlib innodb java jdepend jikes jpeg jsch junit kde ldap libg++ libwww mad maildir md5sum mikmod motif mozilla moznocompose moznoirc mozsvg mpeg mysql ncurses nls oggvorbis opengl oss pam pda pdflib perl png postgres ppds python qt quicktime radeon readline samba sasl scanner sdl slang spell ssl svga tcpd tetex tiff truetype video_cards_radeon vim-with-x x86 xface xml2 xmms xv zlib"

Fixed these ebuilds to use more suitable runtime dependencies. Sed is part of
the base system, and it is highly unlikely that Gentoo will even boot without
it, so it needs not be added explicitly.