tag 634595 + patch
thanks

Hi Racke!

In order to fix this problem, I propose to apply attached patch. I hope
my assumption is correct that only the install target needs the 022
umask - it builds for me, but I did not do a before/after analysis of
the file permissions in the packages built.

WM


> Relevant part:
>>  fakeroot debian/rules clean
>> dh_testdir
>> if [ ! -x /usr/bin/openssl ]; then \
>>              echo "You need to install openssl first"; \
>>              exit 1; \
>>      fi
>> if [ `umask` != "0022" ]; then echo "You need to set umask to 022 in order 
>> to compile/build courier"; exit 1; fi
>> You need to set umask to 022 in order to compile/build courier
>> make: *** [check] Error 1

diff --git a/debian/rules b/debian/rules
index 7879771..8b21188 100755
--- a/debian/rules
+++ b/debian/rules
@@ -63,8 +63,6 @@ check:
 		exit 1; \
 	fi
 
-# check umask
-	if [ `umask` != "0022" ]; then echo "You need to set umask to 022 in order to compile/build courier"; exit 1; fi
 	chmod +x debian/fixlinks
 	chmod +x debian/courier_perms
 	chmod +x debian/correctpermissions.pl
@@ -93,7 +91,10 @@ clean: check
 #can be removed later
 	rm -f debian/permissions
 
-install: check stamp-build
+install:
+	umask 0022; $(MAKE) -f debian/rules install.real
+
+install.real: check stamp-build
 	dh_testroot
 	dh_clean -k
 	dh_installdirs
@@ -346,4 +347,4 @@ binary-arch: stamp-build stamp-install
 
 binary: binary-indep binary-arch
 
-.PHONY: check build clean binary-indep binary-arch binary install
+.PHONY: check build clean binary-indep binary-arch binary install install.real

Reply via email to