[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: simple next step for getting .buildinfo files into Debian



Niels Thykier:
> Holger Levsen:
>> Hi ftp folks,
>>
> 
> Hi,
> 
>> while we still appreciate your comments on this proposal as last week 
>> described in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=763822 I'd like 
>> to make a intermediate very simple proposal, so that reproducible builds in 
>> Debian get one step forward:
>>
> 
> I have started a git branch, build-info-support, available from:
> 
> * ssh://release.debian.org/~nthykier/dak
> 
> [...]
> 
> Thanks,
> ~Niels
> 
> 

Hi,

Thanks to h01ger and jwilk, who spotted a ".changes" that should have
been a ".buildinfo" in regexes.py.  I have rebased the patches (and
resigned them) on to master.  The changed patch is also attached.

Thanks,
~Niels


From ce901421acfc505d9d5787940dc964dceed1b32c Mon Sep 17 00:00:00 2001
From: Niels Thykier <niels@thykier.net>
Date: Mon, 14 Dec 2015 20:46:28 +0000
Subject: [PATCH 1/2] daklib/upload.py: Silently accept and discard .buildinfo
 files

Besides some very trivial validation of the filename, accept
.buildinfo files without really looking at them.

Signed-off-by: Niels Thykier <niels@thykier.net>
---
 daklib/regexes.py | 4 ++++
 daklib/upload.py  | 2 ++
 2 files changed, 6 insertions(+)

diff --git a/daklib/regexes.py b/daklib/regexes.py
index ae8fd91..16dd7ca 100644
--- a/daklib/regexes.py
+++ b/daklib/regexes.py
@@ -141,6 +141,10 @@ re_file_source = re.compile(_re_file_prefix + r'(?:(?:\.orig(?:-[a-zA-Z0-9-]+)?|
 # Groups: package, version
 re_file_orig = re.compile(_re_file_prefix + r'\.orig(?:-[a-zA-Z0-9-]+)?(?:\.tar\.(?:bz2|gz|xz)|\.asc)')
 
+# Match buildinfo file
+# Groups: package, version, suffix
+re_file_buildinfo = re.compile(_re_file_prefix + r'_(?P<suffix>[a-zA-Z0-9+]+)\.buildinfo')
+
 ######################################################################
 # Patterns matching fields                                           #
 ######################################################################
diff --git a/daklib/upload.py b/daklib/upload.py
index b78d100..4c10f45 100644
--- a/daklib/upload.py
+++ b/daklib/upload.py
@@ -374,6 +374,8 @@ class Changes(object):
         for f in self.files.itervalues():
             if re_file_dsc.match(f.filename) or re_file_source.match(f.filename) or re_file_binary.match(f.filename):
                 continue
+            if re_file_buildinfo.match(f.filename):
+                continue
             if f.section != 'byhand' and f.section[:4] != 'raw-':
                 raise InvalidChangesException("{0}: {1} looks like a byhand package, but is in section {2}".format(self.filename, f.filename, f.section))
             byhand.append(f)
-- 
2.6.2

Attachment: signature.asc
Description: OpenPGP digital signature


Reply to: