[pacman-dev] CVS update of pacman-lib/scripts (makepkg)

[dan at archlinux.org]

    Date: Tuesday, February 6, 2007 @ 01:54:13
  Author: dan
    Path: /home/cvs-pacman/pacman-lib/scripts

Modified: makepkg (1.42 -> 1.43)

* Added some backwards compatability stuff to makepkg, along with helpful
  warnings so people aren't caught off guard in the future.
* Added a temp fix for a bug in file 4.19 where bziped tarballs were
  returning type "application/empty".


---------+
 makepkg |   27 +++++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)


Index: pacman-lib/scripts/makepkg
diff -u pacman-lib/scripts/makepkg:1.42 pacman-lib/scripts/makepkg:1.43
--- pacman-lib/scripts/makepkg:1.42	Sun Feb  4 13:28:21 2007
+++ pacman-lib/scripts/makepkg	Tue Feb  6 01:54:13 2007
@@ -120,6 +120,16 @@
 		elif [ "$lc" = "!$needle" ]; then
 			echo "n"
 			return
+		# START DEPRICATED
+		# TODO This code should be removed in the next release of makepkg
+		elif [ "$lc" = "no$needle" ]; then
+			warning "Options beginning with 'no' will be depricated in the next version of makepkg!"
+			plain "Please replace 'no' with '!': no$needle -> !$needle."
+			echo "n"
+			return
+		elif [ "$lc" = "keepdocs" -a "$needle" = "docs" ]; then
+			warning "Option 'keepdocs' may not work as intended. Please replace with 'docs'."
+		# END DEPRICATED
 		fi
 	done
 	# fall back to makepkg.conf options
@@ -542,17 +552,16 @@
 			fakeroot -- $0 -F $ARGLIST
 			exit $?
 		else
-			warning "Fakeroot is not installed.  Building as an unprivileged user"
-			plain "will result in non-root ownership of the packaged files."
-			plain "Install the fakeroot package to correctly build as a non-root"
-			plain "user."
+			warning "Fakeroot is not installed. Building as an unprivileged user"
+			plain "will result in non-root ownership of the packaged files. Install"
+			plain "the fakeroot package to correctly build as a non-root user."
 			plain ""
 			sleep 1
 		fi
 	else
 		warning "Running makepkg as an unprivileged user will result in non-root"		
-		plain "ownership of the packaged files.  Try using the fakeroot"
-		plain "environment.  ('fakeroot' in BUILDENV in makepkg.conf)"
+		plain "ownership of the packaged files. Try using the fakeroot environment"
+		plain "by placing 'fakeroot' in the BUILDENV array in makepkg.conf."
 		plain ""
 		sleep 1
 	fi
@@ -738,6 +747,12 @@
 		file_type=$(file -biz "$file")
 		unset cmd
 		case "$file_type" in
+			# START REMOVE
+			# TODO remove this with release of file v4.20, temp bugfix
+			*application/empty*)
+			cmd="tar -xf $file" ;;
+			warning "file had trouble detecting type, assuming tar"
+			# END REMOVE
 			*application/x-tar*)
 			cmd="tar -xf $file" ;;
 			*application/x-zip*)