Very simply: quake4-data ebuild does not quote $CDROM_ROOT, thus installation will fail if the path has any spaces. Attached ebuild properly quotes the variable to fix this problem, which is inline with other ebuilds providing similar functionality (doom3-data, etc.). Reproducible: Always Steps to Reproduce:
Created attachment 237415 [details] quake4-data/quake4-data-1.0.2147.12.ebuild
for the record, please submit diffs to the current tree ebuilds, but thanks! io /tmp # diff -ru /usr/portage/games-fps/quake4-data/quake4-data-1.0.2147.12.ebuild test --- /usr/portage/games-fps/quake4-data/quake4-data-1.0.2147.12.ebuild 2008-09-26 21:05:58.000000000 +0200 +++ test 2010-07-04 18:37:24.000000000 +0200 @@ -33,20 +33,20 @@ Setup/Data/q4base/pak007.pk4 insinto "${dir}"/q4base einfo "Copying files from Disk 1..." - doins ${CDROM_ROOT}/Setup/Data/q4base/pak01{0,1,2}.pk4 \ - ${CDROM_ROOT}/Setup/Data/q4base/zpak*.pk4 \ + doins "${CDROM_ROOT}"/Setup/Data/q4base/pak01{0,1,2}.pk4 \ + "${CDROM_ROOT}"/Setup/Data/q4base/zpak*.pk4 \ || die "copying pak010->pak012 and zpack*" cdrom_load_next_cd einfo "Copying files from Disk 2..." - doins ${CDROM_ROOT}/Setup/Data/q4base/pak00{1,2,3}.pk4 \ + doins "${CDROM_ROOT}"/Setup/Data/q4base/pak00{1,2,3}.pk4 \ || die "copying pak001->pak003" cdrom_load_next_cd einfo "Copying files from Disk 3..." - doins ${CDROM_ROOT}/Setup/Data/q4base/pak00{4,5,6}.pk4 \ + doins "${CDROM_ROOT}"/Setup/Data/q4base/pak00{4,5,6}.pk4 \ || die "copying pak004->pak006" cdrom_load_next_cd einfo "Copying files from Disk 4..." - doins ${CDROM_ROOT}/Setup/Data/q4base/pak00{7,8,9}.pk4 \ + doins "${CDROM_ROOT}"/Setup/Data/q4base/pak00{7,8,9}.pk4 \ || die "copying pak007->pak009" find ${Ddir} -exec touch '{}' \;
Sure, I can do that going forward. You're the first person I've seen request a diff; didn't realize they were preferred.
in portage. thanks for the bug report and patch.
