Bug#964137: clazy: autopkgtests failure on 32bit

2020-07-02 Thread Gianfranco Costamagna
Source: clazy
Version: 1.7-2
severity: serious
tags: patch

Hello, the following patch should be sufficient to make it work also in 
autopkgtests.
(note: I'm testing it right now on Ubuntu)
https://launchpad.net/ubuntu/+source/clazy/1.7-2ubuntu1
if the version migrates it means the patch is correct, otherwise please have a 
look at the Ubuntu package before integrating!

--- clazy-1.7/debian/changelog  2020-07-01 10:44:27.0 +0200
+++ clazy-1.7/debian/changelog  2020-07-02 13:10:00.0 +0200
@@ -1,3 +1,10 @@
+clazy (1.7-2.1) unstable; urgency=medium
+
+  * Bring back patch to exclude inefficient-qlist check on autopkgtests
+running on 32bit (Closes: #-1).
+
+ -- Gianfranco Costamagna   Thu, 02 Jul 2020 
13:10:00 +0200
+
 clazy (1.7-2) unstable; urgency=medium
 
   * Exclude the inefficient-qlist check again, unfortunately on all the 32bit
diff -Nru clazy-1.7/debian/tests/run-tests clazy-1.7/debian/tests/run-tests
--- clazy-1.7/debian/tests/run-tests2018-11-27 07:23:27.0 +0100
+++ clazy-1.7/debian/tests/run-tests2020-07-02 13:09:59.0 +0200
@@ -2,9 +2,17 @@
 
 set -e
 
+DEB_HOST_ARCH_BITS=$(dpkg-architecture -qDEB_HOST_ARCH_BITS)
+
 # show some facts about clang/clang++, so it is easier to debug issues
 clang -E -x c - -v < /dev/null
 clang++ -E -x c++ - -v < /dev/null
 
 cd tests
-./run_tests.py --verbose
+# wrong size reported, although the test correctly identifies the issue
+# https://bugs.kde.org/show_bug.cgi?id=423728
+if [ $DEB_HOST_ARCH_BITS = 32 ]; then
+  ./run_tests.py --verbose --exclude inefficient-qlist
+else
+  ./run_tests.py --verbose
+fi



Bug#964137: clazy: autopkgtests failure on 32bit

2020-07-02 Thread Gianfranco Costamagna
Hello,

a smarter approach might be this one:
https://invent.kde.org/sdk/clazy/commit/dbde38f1a67c2968c951629a5f5629c9847078bd

(untested)

G.



Bug#964137: clazy: autopkgtests failure on 32bit

2020-07-02 Thread Pino Toscano
severity 964137 important
tag 964137 - patch
thanks

In data giovedì 2 luglio 2020 13:12:54 CEST, Gianfranco Costamagna ha scritto:
> Source: clazy
> Version: 1.7-2
> severity: serious

There are no 32bit architectures for autopkgtest in Debian, so this
is definitely *not* a RC bug.

> Hello, the following patch should be sufficient to make it work also in 
> autopkgtests.

The patch is basically applying the workaround used in debian/rules
also for autopkgtest; weird, flex, but I'd rather use the upstream
patch that does the same.

I will backport it after the current clazy 1.7 migrates to testing,
since there is no reason it should be prevented to migrate.

-- 
Pino Toscano

signature.asc
Description: This is a digitally signed message part.