Bug#881217: python-stetl: please make the build reproducible

2017-11-08 Thread Sebastiaan Couwenberg
Control: tags -1 pending

Hi Chris,

Thanks for the patch and forwarding it upstream.

I've applied it in git and a new upload to unstable will follow shortly.

Kind Regards,

Bas

-- 
 GPG Key ID: 4096R/6750F10AE88D4AF1
Fingerprint: 8182 DE41 7056 408D 6146  50D1 6750 F10A E88D 4AF1



Bug#881217: python-stetl: please make the build reproducible

2017-11-08 Thread Chris Lamb
forwarded 881217 https://github.com/geopython/stetl/pull/64
thanks

I've forwarded this upstream here:

  https://github.com/geopython/stetl/pull/64


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#881217: python-stetl: please make the build reproducible

2017-11-08 Thread Chris Lamb
Source: python-stetl
Version: 1.1+ds-1
Severity: wishlist
Tags: patch
User: reproducible-bui...@lists.alioth.debian.org
Usertags: timestamps
X-Debbugs-Cc: reproducible-b...@lists.alioth.debian.org

Hi,

Whilst working on the Reproducible Builds effort [0], we noticed
that python-stetl could not be built reproducibly.

This is because it embeds the current build path in its own
documentation.

Patch attached.

 [0] https://reproducible-builds.org/


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-
--- a/debian/patches/reproducible_build.patch   1970-01-01 01:00:00.0 
+0100
--- b/debian/patches/reproducible_build.patch   2017-11-08 20:59:42.184958126 
+
@@ -0,0 +1,24 @@
+Description: Make the build reproducible
+Author: Chris Lamb 
+Last-Update: 2017-11-08
+
+--- python-stetl-1.1+ds.orig/stetl/filters/templatingfilter.py
 python-stetl-1.1+ds/stetl/filters/templatingfilter.py
+@@ -139,7 +139,7 @@ class Jinja2TemplatingFilter(TemplatingF
+ # Applying Decorator pattern with the Config class to provide
+ # read-only config values from the configured properties.
+ 
+-@Config(ptype=str, default=[os.getcwd()], required=False)
++@Config(ptype=str, default=None, required=False)
+ def template_search_paths(self):
+ """
+ List of directories where to search for templates, default is current 
working directory only.
+@@ -207,7 +207,7 @@ class Jinja2TemplatingFilter(TemplatingF
+ raise e
+ 
+ # Load and Init Template once
+-loader = FileSystemLoader(self.template_search_paths)
++loader = FileSystemLoader(self.template_search_paths or [os.getcwd()])
+ self.jinja2_env = Environment(loader=loader, 
extensions=['jinja2.ext.do'], lstrip_blocks=True, trim_blocks=True)
+ 
+ # Register additional Filters on the template environment by updating 
the filters dict:
--- a/debian/patches/series 1970-01-01 01:00:00.0 +0100
--- b/debian/patches/series 2017-11-08 20:59:40.688948464 +
@@ -0,0 +1 @@
+reproducible_build.patch