Control: tag -1 patch

On Sun, Dec 27, 2015 at 10:48:51PM +0100, Laurent Bigonville wrote:
> Package: sbuild
> Version: 0.67.0-1
> Severity: important
 
> This is probably also related to the perl transition.

No, just a regression introduced in the last upload.

> sbuild doesn't want to build a source package with only linux-any
> architecture binary packages:
> 
> Check architectures
> -------------------
> 
> Bareword "amd64" not allowed while "strict subs" in use at -e line 4.
> Bareword "linux" not allowed while "strict subs" in use at -e line 4.
> Bareword "any" not allowed while "strict subs" in use at -e line 4.
> Execution of -e aborted due to compilation errors.
> setools_3.3.8+20151215-1.dsc: amd64 not in arch list or does not match any 
> arch wildcards: linux-any -- skipping

The attached proposed patch should fix this.
-- 
Niko Tyni   [email protected]
>From bc5a0813896a97c8908d346e2b32c25e1ce117d1 Mon Sep 17 00:00:00 2001
From: Niko Tyni <[email protected]>
Date: Mon, 28 Dec 2015 00:30:49 +0200
Subject: [PATCH] Fix quoting when checking architectures

The variables got interpolated into $command without any quoting,
essentially becoming barewords.

Regression introduced in 9162dd0d727facb2ebab0ab7008603c5fa8c36a1

Closes: #809175
---
 lib/Sbuild/Build.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/Sbuild/Build.pm b/lib/Sbuild/Build.pm
index a6ac74c..95245ca 100644
--- a/lib/Sbuild/Build.pm
+++ b/lib/Sbuild/Build.pm
@@ -1139,7 +1139,7 @@ sub fetch_source_files {
 		use strict;
 		use warnings;
 		use Dpkg::Arch;
-		if (Dpkg::Arch::debarch_is($host_arch, $a)) {
+		if (Dpkg::Arch::debarch_is('$host_arch', '$a')) {
 		    exit 0;
 		}
 		exit 1;
-- 
2.6.4

Reply via email to