Closing bug #134192. Original patch provided by Paul Bredbury <brebs@sent.com> and edited by me for inclusion to maintain Games team coding standards.

Package-Manager: portage-2.1_rc2-r1

3 files changed, 49 insertions, 43 deletions

diff --git a/games-fps/ut2004-data/ChangeLog b/games-fps/ut2004-data/ChangeLog
index cdd0afada3e0..4fc6b40ddb10 100644
--- a/games-fps/ut2004-data/ChangeLog
+++ b/games-fps/ut2004-data/ChangeLog
@@ -1,6 +1,12 @@
 # ChangeLog for games-fps/ut2004-data
 # Copyright 1999-2006 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/games-fps/ut2004-data/ChangeLog,v 1.16 2006/04/17 21:56:58 wolf31o2 Exp $
+# $Header: /var/cvsroot/gentoo-x86/games-fps/ut2004-data/ChangeLog,v 1.17 2006/05/24 15:12:12 wolf31o2 Exp $
+
+  24 May 2006; Chris Gianelloni <wolf31o2@gentoo.org>
+  ut2004-data-3186-r3.ebuild:
+  Closing bug #134192. Original patch provided by Paul Bredbury
+  <brebs@sent.com> and edited by me for inclusion to maintain Games team
+  coding standards.
 
   17 Apr 2006; Chris Gianelloni <wolf31o2@gentoo.org>
   ut2004-data-3186-r3.ebuild:
diff --git a/games-fps/ut2004-data/Manifest b/games-fps/ut2004-data/Manifest
index b86adb2d2c8c..9fdac0601b2f 100644
--- a/games-fps/ut2004-data/Manifest
+++ b/games-fps/ut2004-data/Manifest
@@ -1,4 +1,15 @@
-MD5 82e9d9293f4d88c5405ca6fe7992a5c2 ChangeLog 3158
-MD5 68b329da9893e34099c7d8ad5cb9c940 files/digest-ut2004-data-3186-r3 1
+EBUILD ut2004-data-3186-r3.ebuild 7959 RMD160 927d3bc5dc3d320e97e9d6baf440ea084a2a4aab SHA1 e6b34cf04984862e5476c6011e35e520b11e87fd SHA256 7b61ef3c89e6023eb378dab82e9d67e7b56e3e818e5cf06efb8937fd28343b4c
+MD5 9fbf6225b3aa4fc66773ee3575041236 ut2004-data-3186-r3.ebuild 7959
+RMD160 927d3bc5dc3d320e97e9d6baf440ea084a2a4aab ut2004-data-3186-r3.ebuild 7959
+SHA256 7b61ef3c89e6023eb378dab82e9d67e7b56e3e818e5cf06efb8937fd28343b4c ut2004-data-3186-r3.ebuild 7959
+MISC ChangeLog 3400 RMD160 7bce805a69d7309fde68325f6d0398d061f315ee SHA1 4ce6bc27a17ddcd888f0a8237821f5c39aac48cb SHA256 5275c5f1a62401c8a6ba0235b8a48268c3fc89aa7364166738d29fc2a7069753
+MD5 76e5fbee96983c5a3a22b486d1daa070 ChangeLog 3400
+RMD160 7bce805a69d7309fde68325f6d0398d061f315ee ChangeLog 3400
+SHA256 5275c5f1a62401c8a6ba0235b8a48268c3fc89aa7364166738d29fc2a7069753 ChangeLog 3400
+MISC metadata.xml 254 RMD160 877a1dc36df2be58926bc51041093feafdefe429 SHA1 45f0334cd127c4a1b4ece41f7098ab70046e378a SHA256 2916aab684ec3908fffb515065eeb2c481cde563380b682646add582bed5f535
 MD5 11a7e1491b1ac042ac7216b26b4ad1fb metadata.xml 254
-MD5 43683d9373433429478be027b2511dd8 ut2004-data-3186-r3.ebuild 8292
+RMD160 877a1dc36df2be58926bc51041093feafdefe429 metadata.xml 254
+SHA256 2916aab684ec3908fffb515065eeb2c481cde563380b682646add582bed5f535 metadata.xml 254
+MD5 68b329da9893e34099c7d8ad5cb9c940 files/digest-ut2004-data-3186-r3 1
+RMD160 c0da025038ed83c687ddc430da9846ecb97f3998 files/digest-ut2004-data-3186-r3 1
+SHA256 01ba4719c80b6fe911b091a7c05124b64eeece964e09c058ef8f9805daca546b files/digest-ut2004-data-3186-r3 1
diff --git a/games-fps/ut2004-data/ut2004-data-3186-r3.ebuild b/games-fps/ut2004-data/ut2004-data-3186-r3.ebuild
index 7595d3f16dc0..0e36d89b37bb 100644
--- a/games-fps/ut2004-data/ut2004-data-3186-r3.ebuild
+++ b/games-fps/ut2004-data/ut2004-data-3186-r3.ebuild
@@ -1,6 +1,6 @@
 # Copyright 1999-2006 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/games-fps/ut2004-data/ut2004-data-3186-r3.ebuild,v 1.6 2006/04/18 14:14:23 wolf31o2 Exp $
+# $Header: /var/cvsroot/gentoo-x86/games-fps/ut2004-data/ut2004-data-3186-r3.ebuild,v 1.7 2006/05/24 15:12:12 wolf31o2 Exp $
 
 inherit games games-ut2k4mod
 
@@ -23,15 +23,15 @@ dir=${GAMES_PREFIX_OPT}/ut2004
 Ddir=${D}/${dir}
 
 grabdirs() {
-	local srcdir
+	local d srcdir
 
 	for d in {Music,Sounds,Speech,StaticMeshes,Textures} ; do
 		srcdir=${CDROM_ROOT}/$1${d}
 		# Is flexible to handle CD_ROOT vs CD_ROOT_1 mixups
 		[[ -d "${srcdir}" ]] || srcdir=${CDROM_ROOT}/${d}
 		if [[ -d "${srcdir}" ]] ; then
-			echo "Copying ${srcdir}"
-			cp -r "${srcdir}" "${Ddir}" || die "copying ${srcdir}"
+			insinto "${dir}"
+			doins -r "${srcdir}" || die "doins ${srcdir} failed"
 		fi
 	done
 }
@@ -80,46 +80,36 @@ pkg_setup() {
 src_unpack() {
 	unpack_makeself "${CDROM_ROOT}"/linux-installer.sh \
 		|| die "unpacking linux installer"
-	use x86 && tar -xf "${S}"/linux-x86.tar
-	use amd64 && tar -xf "${S}"/linux-amd64.tar
+	use x86 && unpack ./linux-x86.tar
+	use amd64 && unpack ./linux-amd64.tar
 }
 
 src_install() {
-	dodir "${dir}"/System/editorres
+	local diskno srcdir varname
 
 	# Disk 1
 	einfo "Copying files from Disk 1..."
-	cp -r "${CDROM_ROOT}/${DISK1}"{Animations,ForceFeedback,Help,KarmaData,Maps,Sounds,Web} "${Ddir}" \
-		|| die "copying files"
-	cp -r "${CDROM_ROOT}/${DISK1}"System/{editorres,*.{bat,bmp,dat,det,est,frt,ini,int,itt,kot,md5,smt,tmt,u,ucl,upl,url}} \
-		"${Ddir}"/System || die "copying files"
-	mkdir -p "${Ddir}"/Manual || die "creating manual folder"
-	cp "${CDROM_ROOT}/${DISK1}"Manual/Manual.pdf "${Ddir}"/Manual \
+	insinto "${dir}"
+	doins -r "${CDROM_ROOT}/${DISK1}"{Animations,ForceFeedback,Help,KarmaData,Maps,Sounds,Web} \
+		|| die "copying directories"
+	insinto "${dir}"/System
+	doins -r "${CDROM_ROOT}/${DISK1}"System/{editorres,*.{bat,bmp,dat,det,est,frt,ini,int,itt,kot,md5,smt,tmt,u,ucl,upl,url}} \
+		|| die "copying System files"
+	insinto "${dir}"/Manual
+	doins "${CDROM_ROOT}/${DISK1}"Manual/Manual.pdf \
 		|| die "copying manual"
-	mkdir -p "${Ddir}"/Benchmark/Stuff || die "creating benchmark folders"
-	cp -r "${CDROM_ROOT}/${DISK1}"Benchmark/Stuff/* "${Ddir}"/Benchmark/Stuff \
-		|| die "copying benchmark files"
-	cdrom_load_next_cd
-
-	# Disk 2
-	einfo "Copying files from Disk 2..."
-	grabdirs "${DISK2}"
-	cdrom_load_next_cd
-
-	# Disk 3
-	einfo "Copying files from Disk 3..."
-	grabdirs "${DISK3}"
-	cdrom_load_next_cd
-
-	# Disk 4
-	einfo "Copying files from Disk 4..."
-	grabdirs "${DISK4}"
+	insinto "${dir}"/Benchmark/Stuff
+	doins -r "${CDROM_ROOT}/${DISK1}"Benchmark/Stuff/* \
+		|| die "copying Benchmark files"
 	cdrom_load_next_cd
 
-	# Disk 5
-	einfo "Copying files from Disk 5..."
-	grabdirs "${DISK5}"
-	cdrom_load_next_cd
+	for diskno in 2 3 4 5 ; do
+		einfo "Copying files from Disk ${diskno}..."
+		varname="DISK${diskno}"
+		srcdir=${!varname}
+		grabdirs "${srcdir}"
+		cdrom_load_next_cd
+	done
 
 	# Disk 6
 	einfo "Copying files from Disk 6..."
@@ -128,8 +118,8 @@ src_install() {
 	# Create empty files in Benchmark
 	for j in {CSVs,Logs,Results}
 	do
-		mkdir -p "${Ddir}/Benchmark/${j}" || die "creating folders"
-		touch "${Ddir}/Benchmark/${j}"/DO_NOT_DELETE.ME || die "creating files"
+		dodir "${dir}/Benchmark/${j}"
+		touch "${Ddir}/Benchmark/${j}"/DO_NOT_DELETE.ME || die "creating dummy files"
 	done
 
 	# Install extra help files
@@ -206,8 +196,7 @@ src_install() {
 
 	# Installing documentation/icon
 	dodoc "${S}"/README.linux || die "dodoc README.linux"
-	insinto /usr/share/pixmaps
-	doins "${S}"/ut2004.xpm || die "copying pixmap"
+	doicon "${S}"/ut2004.xpm || die "doicon ut2004.xpm"
 	insinto "${dir}"
 	doins "${S}"/{README.linux,ut2004.xpm} || die "copying readme/icon"