Package: src:apg
Followup-For: Bug #833141

Daniel Shahaf wrote on Mon, Aug 01, 2016 at 10:56:05 +0000:
> -     tar --create --file - --directory $(CURDIR)/php/apgonline/ . | gzip 
> --no-name > php.tar.gz
> +     tar --create --file - --directory $(CURDIR)/php/apgonline/ 
> --mode=u=rwX,go=rX . \
> +             | gzip --no-name > php.tar.gz

Reiner noticed that --sort=name is also needed here to account for
varying readdir() order on some platforms.  Here's a replacement patch:

[[[
diff --git a/debian/rules b/debian/rules
index e313383..b4f9477 100755
--- a/debian/rules
+++ b/debian/rules
@@ -9,7 +9,8 @@ override_dh_auto_build:
 override_dh_auto_install:
        make install INSTALL_PREFIX=$(CURDIR)/debian/apg/usr
        mv $(CURDIR)/debian/apg/usr/bin/apg $(CURDIR)/debian/apg/usr/lib/apg/apg
-       tar --create --file - --directory $(CURDIR)/php/apgonline/ . | gzip 
--no-name > php.tar.gz
+       tar --create --file - --directory $(CURDIR)/php/apgonline/ \
+               --mode=u=rwX,go=rX --sort=name | gzip --no-name > php.tar.gz
        install -D --mode=0644 php.tar.gz 
$(CURDIR)/debian/apg/usr/share/doc/apg/php.tar.gz
        rm php.tar.gz
        install -D --mode=0755 $(CURDIR)/debian/apg.wrapper 
$(CURDIR)/debian/apg/usr/bin/apg
]]]

Cheers,

Daniel

Reply via email to