Followup-For: Bug #624530
Package: pbuilder
Version: 0.200

found 624530 0.199+nmu2
severity 624530 important
tags 624530 patch
thanks

Hi,

The changes introduced in 0.199+nmu2 (arch-wildcard-part2.patch in
Message #159 of Bug#363193) breaks the cross building of packages
(not only i386 packages on amd64, but also amd64 ones on i386 running
on amd64 kernel).
The checkarchitecture must be done inside the build place.
That patch does not actually check anything, and what's worse, breaks
the cross building.
So I propose reverting back the changes.

Thanks,
-- 
YOSHINO Yoshihito <yy.y.ja...@gmail.com>
diff -ur pbuilder-0.200/pbuilder-buildpackage pbuilder-0.200.new/pbuilder-buildpackage
--- pbuilder-0.200/pbuilder-buildpackage	2011-06-25 08:06:40.000000000 +0900
+++ pbuilder-0.200.new/pbuilder-buildpackage	2011-07-27 15:13:05.073598440 +0900
@@ -31,9 +31,6 @@
     exit 1;
 fi;
 
-# check if this package should be built at all
-checkarchitecture "$PACKAGENAME"
-
 if [ -n "$BUILDUSERNAME" -a -n "$BUILDUSERID" ]; then
     SUTOUSER="env LOGNAME=$BUILDUSERNAME su -p $BUILDUSERNAME"
     DEBBUILDOPTS="${DEBBUILDOPTS:+$DEBBUILDOPTS }-rfakeroot"
diff -ur pbuilder-0.200/pbuilder-buildpackage-funcs pbuilder-0.200.new/pbuilder-buildpackage-funcs
--- pbuilder-0.200/pbuilder-buildpackage-funcs	2011-06-25 08:06:40.000000000 +0900
+++ pbuilder-0.200.new/pbuilder-buildpackage-funcs	2011-07-27 15:13:05.073598440 +0900
@@ -30,25 +30,6 @@
     done
 }
 
-function checkarchitecture () {
-    local DSCFILE="$1"
-    local ARCHES="$(cat $DSCFILE | grep Architecture | sed 's/^[^:]\+:\s*//')"
-    local ARCH=$(dpkg-architecture -qDEB_HOST_ARCH)
-    local VALID_ARCH
-    for d in $ARCHES; do
-        if dpkg-architecture -a$ARCH -i$d; then
-            VALID_ARCH="yes"
-            break
-        fi
-    done
-    if [ ! $VALID_ARCH ] && [ ! "$ARCHES" = "all" ]; then
-        local msg="E: $ARCH not in arch list or does not match any "
-        msg="$msg arch wildcards: $ARCHES"
-        log $msg
-        exit 2
-    fi
-}
-
 function checkbuilddep () {
     # call satisfydepends
     local BUILDOPT="--binary-all"

Reply via email to