Bug#997802: ddnet: autopkgtest regression on armhf: output to stderr

2021-10-26 Thread Graham Inggs 
On Sun, 24 Oct 2021 at 22:30, Paul Gevers  wrote:
> I copied some of the output at the bottom of this report. It seems that
> the actual test passes, but there's output on stderr and the default
> behavior of autopkgtest is to fail on output to stderr. If in general
> output to stderr is harmless to your test, consider adding the
> allow-stderr restriction. Otherwise, please fix your test.

The autopkgtest on ppc64el [1] also fails due to output on stderr.


[1] https://ci.debian.net/packages/d/ddnet/testing/ppc64el/


/tmp/autopkgtest-lxc.e6vtmdil/downtmp/build.Hvt/src/src/game/collision.cpp:
In member function ‘void CCollision::Init(CLayers*)’:
/tmp/autopkgtest-lxc.e6vtmdil/downtmp/build.Hvt/src/src/game/collision.cpp:66:10:
note: the layout of aggregates containing vectors with 8-byte
alignment has changed in GCC 5
   66 |  m_Width = m_pLayers->GameLayer()->m_Width;
  |  ^
[ 59%] Building CXX object CMakeFiles/game-shared.dir/src/game/extrainfo.cpp.o
[ 59%] Building CXX object CMakeFiles/game-shared.dir/src/game/gamecore.cpp.o
/tmp/autopkgtest-lxc.e6vtmdil/downtmp/build.Hvt/src/src/game/gamecore.cpp:
In member function ‘void CCharacterCore::Reset()’:
/tmp/autopkgtest-lxc.e6vtmdil/downtmp/build.Hvt/src/src/game/gamecore.cpp:70:6:
note: the layout of aggregates containing vectors with 2-byte
alignment has changed in GCC 5
   70 | void CCharacterCore::Reset()
  |  ^~
[ 62%] Building CXX object CMakeFiles/game-shared.dir/src/game/layers.cpp.o
[ 62%] Building CXX object
CMakeFiles/game-shared.dir/src/game/localization.cpp.o
/tmp/autopkgtest-lxc.e6vtmdil/downtmp/build.Hvt/src/src/game/localization.cpp:
In member function ‘void CLocalizationDatabase::AddString(const char*,
const char*, const char*)’:
/tmp/autopkgtest-lxc.e6vtmdil/downtmp/build.Hvt/src/src/game/localization.cpp:39:6:
note: the layout of aggregates containing vectors with 8-byte
alignment has changed in GCC 5
   39 | void CLocalizationDatabase::AddString(const char *pOrgStr,
const char *pNewStr, const char *pContext)
  |  ^

Bug#997802: ddnet: autopkgtest regression on armhf: output to stderr

2021-10-24 Thread Paul Gevers 
Source: ddnet
Version: 15.5.4-1
X-Debbugs-CC: debian...@lists.debian.org
Severity: serious
User: debian...@lists.debian.org
Usertags: regression

Dear maintainer(s),

With a recent upload of ddnet the autopkgtest of ddnet fails in testing
when that autopkgtest is run with the binary packages of ddnet from
unstable. It passes when run with only packages from testing. In tabular
form:

   passfail
ddnet  from testing15.5.4-1
all others from testingfrom testing

I copied some of the output at the bottom of this report. It seems that
the actual test passes, but there's output on stderr and the default
behavior of autopkgtest is to fail on output to stderr. If in general
output to stderr is harmless to your test, consider adding the
allow-stderr restriction. Otherwise, please fix your test.

Currently this regression is blocking the migration to testing [1]. Can
you please investigate the situation and fix it?

More information about this bug and the reason for filing it can be found on
https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation

Paul

[1] https://qa.debian.org/excuses.php?package=ddnet

https://ci.debian.net/data/autopkgtest/testing/armhf/d/ddnet/16147062/log.gz

[ 91%] Building CXX object CMakeFiles/testrunner.dir/src/test/unix.cpp.o
[ 91%] Building CXX object CMakeFiles/testrunner.dir/src/test/uuid.cpp.o
[ 94%] Building CXX object
CMakeFiles/testrunner.dir/src/engine/client/blocklist_driver.cpp.o
[ 94%] Building CXX object
CMakeFiles/testrunner.dir/src/engine/client/http.cpp.o
[ 94%] Building CXX object
CMakeFiles/testrunner.dir/src/engine/client/serverbrowser.cpp.o
[ 97%] Building CXX object
CMakeFiles/testrunner.dir/src/engine/client/serverbrowser_http.cpp.o
In file included from /usr/include/c++/10/vector:72,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/engine/graphics.h:13,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/engine/textrender.h:8,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/game/client/ui.h:7,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/engine/serverbrowser.h:8,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/engine/client/serverbrowser_http.cpp:8:
/usr/include/c++/10/bits/vector.tcc: In member function ‘void
std::vector<_Tp, _Alloc>::_M_realloc_insert(std::vector<_Tp,
_Alloc>::iterator, _Args&& ...) [with _Args =
{CServerBrowserHttp::CEntry}; _Tp = CServerBrowserHttp::CEntry; _Alloc =
std::allocator]’:
/usr/include/c++/10/bits/vector.tcc:426:7: note: parameter passing for
argument of type ‘std::vector::iterator’
changed in GCC 7.1
  426 |   vector<_Tp, _Alloc>::
  |   ^~~
/usr/include/c++/10/bits/vector.tcc: In static member function ‘static
bool CServerBrowserHttp::Parse(json_value*,
std::vector*, std::vector*)’:
/usr/include/c++/10/bits/vector.tcc:121:21: note: parameter passing for
argument of type
‘__gnu_cxx::__normal_iterator >’ changed in GCC 7.1
  121 |_M_realloc_insert(end(), std::forward<_Args>(__args)...);
  |~^~~
[ 97%] Building CXX object
CMakeFiles/testrunner.dir/src/engine/client/serverbrowser_ping_cache.cpp.o
[ 97%] Building CXX object
CMakeFiles/testrunner.dir/src/engine/client/sqlite.cpp.o
[100%] Building CXX object
CMakeFiles/testrunner.dir/src/engine/server/name_ban.cpp.o
[100%] Building CXX object
CMakeFiles/testrunner.dir/src/game/server/teehistorian.cpp.o
[100%] Linking CXX executable testrunner
[100%] Built target testrunner
[100%] tests


[...]

autopkgtest [18:23:02]: test unittest.sh:  - - - - - - - - - - stderr -
- - - - - - - - -
In file included from /usr/include/c++/10/vector:72,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/engine/graphics.h:13,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/engine/textrender.h:8,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/game/client/ui.h:7,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/engine/serverbrowser.h:8,
 from
/tmp/autopkgtest-lxc.6korsrvz/downtmp/build.XVn/src/src/engine/client/serverbrowser_http.cpp:8:
/usr/include/c++/10/bits/vector.tcc: In member function ‘void
std::vector<_Tp, _Alloc>::_M_realloc_insert(std::vector<_Tp,
_Alloc>::iterator, _Args&& ...) [with _Args =
{CServerBrowserHttp::CEntry}; _Tp = CServerBrowserHttp::CEntry; _Alloc =
std::allocator]’:
/usr/include/c++/10/bits/vector.tcc:426:7: note: parameter passing for
argument of type ‘std::vector::iterator’
changed in GCC 7.1
  426 |   vector<_Tp, _Alloc>::
  |   ^~~
/usr/include/c++/10/bits/vector.tcc: In static member function ‘static
bool CServerBrowserHttp::Parse(json_value*,
std::vector*, std::vector*)’: