Bug#852888: sx: FTBFS: Test failures

2017-02-07 Thread Roger Shimizu
Put the latest release to debomatic amd64, to reproduce the FTBFS:
 - http://debomatic-amd64.debian.net/distribution#unstable/sx/2.0+ds-3/buildlog

And the package after my patch:
 - 
http://debomatic-amd64.debian.net/distribution#unstable/sx/2.0+ds-3.1/buildlog

So it confirms the fix.
And I also uploaded the fixed package to mentors:
 - https://mentors.debian.net/package/sx
 - https://mentors.debian.net/debian/pool/main/s/sx/sx_2.0+ds-3.1.dsc

Hope it's convenient for you to sponsor the upload.
If not, I'll ask mentors list for help this week. Thank you!

Cheers,
-- 
Roger Shimizu, GMT +9 Tokyo
PGP/GPG: 4096R/6C6ACD6417B3ACB1



Bug#852888: sx: FTBFS: Test failures

2017-02-07 Thread Roger Shimizu
Control: tag -1 patch

> The full build log is available from:
>http://aws-logs.debian.net/2017/01/28/sx_2.0+ds-3_unstable.log

The build log of AWS shows the error is:
nginx: [alert] could not open error log file: open()
"/var/log/nginx/error.log" failed (13: Permission denied)
...
+ /<>/sx-2.0+ds/server/sx-test-5uNoeXXd/bin/sxinit
--batch-mode --port=8013 --no-ssl
--auth-file=/<>/sx-2.0+ds/server/sx-test-5uNoeXXd/var/lib/sxserver/data/admin.key
--config-dir=/<>/sx-2.0+ds/server/sx-test-5uNoeXXd/.sx
sx://localhost
Segmentation fault

And logs of buildd [0][1][2] seems simply skipped the nginx test:
  PASS: test/hdist-test
  PASS: test/blob-test
  SKIP: test/run-nginx-test.sh

[0] https://buildd.debian.org/status/package.php?p=sx
[1] 
https://buildd.debian.org/status/fetch.php?pkg=sx&arch=i386&ver=2.0%2Bds-3&stamp=1481129412&raw=0
[2] 
https://buildd.debian.org/status/fetch.php?pkg=sx&arch=arm64&ver=2.0%2Bds-3&stamp=1481129348&raw=0

However, local "gbp buildpackage" result is the same as AWS log.
Since test/run-nginx-test.sh was previously skipped on buildd already,
so I think it should be OK to add a patch to prevent it running in
local gbp environment.

Please kindly help to sponsor the upload, before it get removed from stretch.
Thank you!

Cheers,
-- 
Roger Shimizu, GMT +9 Tokyo
PGP/GPG: 4096R/6C6ACD6417B3ACB1
From: Roger Shimizu 
Date: Tue, 7 Feb 2017 18:44:30 +0900
Subject: [PATCH] debian/patches: Add a patch to skip test/run-nginx-test.sh

---
 .../patches/04-Skip-test-run-nginx-test.sh.patch   | 36 ++
 debian/patches/series  |  1 +
 2 files changed, 37 insertions(+)
 create mode 100644 debian/patches/04-Skip-test-run-nginx-test.sh.patch

diff --git a/debian/patches/04-Skip-test-run-nginx-test.sh.patch b/debian/patches/04-Skip-test-run-nginx-test.sh.patch
new file mode 100644
index 000..840b8fa
--- /dev/null
+++ b/debian/patches/04-Skip-test-run-nginx-test.sh.patch
@@ -0,0 +1,36 @@
+From: Roger Shimizu 
+Date: Tue, 7 Feb 2017 18:43:29 +0900
+Subject: Skip test/run-nginx-test.sh
+
+---
+ server/Makefile.am | 2 +-
+ server/Makefile.in | 3 +--
+ 2 files changed, 2 insertions(+), 3 deletions(-)
+
+diff --git a/server/Makefile.am b/server/Makefile.am
+index 4ac0137..9b65eaf 100644
+--- a/server/Makefile.am
 b/server/Makefile.am
+@@ -155,7 +155,7 @@ check_PROGRAMS = test/printerrno
+ 
+ check_SCRIPTS = test/runvg.sh test/run-nginx-test.sh test/fcgi-test.pl
+ EXTRA_DIST += $(check_SCRIPTS)
+-TESTS = test/hdist-test test/blob-test test/run-nginx-test.sh
++TESTS = test/hdist-test test/blob-test
+ 
+ test_printerrno_SOURCES = test/printerrno.c
+ 
+diff --git a/server/Makefile.in b/server/Makefile.in
+index 46b761c..cb717f2 100644
+--- a/server/Makefile.in
 b/server/Makefile.in
+@@ -97,8 +97,7 @@ sbin_PROGRAMS = src/fcgi/sx.fcgi$(EXEEXT) \
+ 	src/tools/sxreport-server/sxreport-server$(EXEEXT) \
+ 	src/tools/sxadm/sxadm$(EXEEXT)
+ check_PROGRAMS = test/printerrno$(EXEEXT)
+-TESTS = test/hdist-test$(EXEEXT) test/blob-test$(EXEEXT) \
+-	test/run-nginx-test.sh
++TESTS = test/hdist-test$(EXEEXT) test/blob-test$(EXEEXT)
+ subdir = .
+ ACLOCAL_M4 = $(top_srcdir)/aclocal.m4
+ am__aclocal_m4_deps = $(top_srcdir)/m4/ax_append_compile_flags.m4 \
diff --git a/debian/patches/series b/debian/patches/series
index 89b8d64..ce1189c 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,3 +1,4 @@
 01-OpenSSL.patch
 02-OpenSSL.patch
 03-OpenSSL.patch
+04-Skip-test-run-nginx-test.sh.patch


Bug#852888: sx: FTBFS: Test failures

2017-01-28 Thread Lucas Nussbaum
Source: sx
Version: 2.0+ds-3
Severity: serious
Tags: stretch sid
User: debian...@lists.debian.org
Usertags: qa-ftbfs-20170128 qa-ftbfs
Justification: FTBFS on amd64

Hi,

During a rebuild of all packages in sid, your package failed to build on
amd64.

Relevant part (hopefully):
> Makefile:2232: recipe for target 'test-suite.log' failed
> make[6]: *** [test-suite.log] Error 1
> make[6]: Leaving directory '/<>/sx-2.0+ds/server'
> Makefile:2338: recipe for target 'check-TESTS' failed
> make[5]: *** [check-TESTS] Error 2
> make[5]: Leaving directory '/<>/sx-2.0+ds/server'
> Makefile:2583: recipe for target 'check-am' failed
> make[4]: *** [check-am] Error 2
> make[4]: Leaving directory '/<>/sx-2.0+ds/server'
> Makefile:2118: recipe for target 'check-recursive' failed
> make[3]: *** [check-recursive] Error 1
> make[3]: Leaving directory '/<>/sx-2.0+ds/server'
> Makefile:420: recipe for target 'check-recursive' failed
> make[2]: *** [check-recursive] Error 1
> make[2]: Leaving directory '/<>/sx-2.0+ds'
> debian/rules:20: recipe for target 'override_dh_auto_test' failed

The full build log is available from:
   http://aws-logs.debian.net/2017/01/28/sx_2.0+ds-3_unstable.log

A list of current common problems and possible solutions is available at
http://wiki.debian.org/qa.debian.org/FTBFS . You're welcome to contribute!

About the archive rebuild: The rebuild was done on EC2 VM instances from
Amazon Web Services, using a clean, minimal and up-to-date chroot. Every
failed build was retried once to eliminate random failures.