Bug#964044: mrpt: FTBFS: test failure
On Fri, 17 Jul 2020 08:41:13 +0200 Gianfranco Costamagna wrote: > Hello again > > I'm deleting the upload, looks like the new ftdi broke configure script. > > see the attached build log > > G. reuploaded, I gave ftdi a patch, and an upload is coming shortly G.
Bug#964044: mrpt: FTBFS: test failure
Hi, Gianfranco: > > This issue has been fixed upstream: > > https://github.com/MRPT/mrpt/commit/15234dc335c2413e3fd41021f7511f1d36fe915b. > > Could you please apply the fix to the Debian package so that > > ros-geometry2 transition can be completed? Thanks > > > looks like that commit is already part of 2.0.4? Sorry for the confusion on this particular unit test. Yes, the commit you mention was already included into 2.0.4, and it solved all the issues detected by "valgrind helgrind". *But* even with that fixed, it failed as you reported in this bug, so there is this newer commit upstream: https://github.com/MRPT/mrpt/commit/e84511500276d38d3eeff0b220e8d45e0d74fc93 which is not yet released as 2.0.5, and which you can include as a patch if you want to go on with the ros-geometry2 transition. Cheers, JL -- /** * Jose Luis Blanco-Claraco * Universidad de Almería - Departamento de Ingeniería * [Homepage]( https://w3.ual.es/~jlblanco/ ) * [GH profile]( https://github.com/jlblancoc ) */
Bug#964044: mrpt: FTBFS: test failure
Hello, > which is not yet released as 2.0.5, and which you can include as a > patch if you want to go on with the ros-geometry2 transition. > Since you are the maintainer, I'm uploading in deferred/1 your patch, refreshed in some parts to apply to the current 2.0.4 (and removing the rules part, since we can't patch with quilt debian directory) I'm attaching the diff that will go in unstable in ~24h I didn't use NMU as notation, but a "team upload" since the patch comes from you :) let me know if this is good, so I can speed it up and let it go in sid today! G. diff -Nru mrpt-2.0.4/debian/changelog mrpt-2.0.4/debian/changelog --- mrpt-2.0.4/debian/changelog 2020-06-20 17:24:00.0 +0200 +++ mrpt-2.0.4/debian/changelog 2020-07-17 08:14:20.0 +0200 @@ -1,3 +1,11 @@ +mrpt (1:2.0.4-2) unstable; urgency=medium + + * Team upload + * debian/patches/e84511500276d38d3eeff0b220e8d45e0d74fc93.patch: +- cherry-pick upstream "fix" for a failing test (Closes: #964044). + + -- Gianfranco Costamagna Fri, 17 Jul 2020 08:14:20 +0200 + mrpt (1:2.0.4-1) unstable; urgency=medium * New version of upstream sources. diff -Nru mrpt-2.0.4/debian/patches/e84511500276d38d3eeff0b220e8d45e0d74fc93.patch mrpt-2.0.4/debian/patches/e84511500276d38d3eeff0b220e8d45e0d74fc93.patch --- mrpt-2.0.4/debian/patches/e84511500276d38d3eeff0b220e8d45e0d74fc93.patch 1970-01-01 01:00:00.0 +0100 +++ mrpt-2.0.4/debian/patches/e84511500276d38d3eeff0b220e8d45e0d74fc93.patch 2020-07-17 08:14:20.0 +0200 @@ -0,0 +1,55 @@ +From e84511500276d38d3eeff0b220e8d45e0d74fc93 Mon Sep 17 00:00:00 2001 +From: Jose Luis Blanco Claraco +Date: Thu, 2 Jul 2020 23:53:34 +0200 +Subject: [PATCH] give up with the problematic unit test if building a Debian + package + +--- + doc/doxygen-pages/changeLog_doc.h | 4 + libs/apps/src/RawlogGrabberApp_unittest.cpp | 10 ++ + packaging/debian/rules | 2 ++ + 3 files changed, 16 insertions(+) + +Index: mrpt-2.0.4/doc/doxygen-pages/changeLog_doc.h +=== +--- mrpt-2.0.4.orig/doc/doxygen-pages/changeLog_doc.h mrpt-2.0.4/doc/doxygen-pages/changeLog_doc.h +@@ -47,6 +47,10 @@ + - Fix: mrpt::maps::CPointsMapXYZI::setFromPCLPointCloudXYZI() was using a non-existing method. + - Fix: mrpt::nav::PlannerSimple2D did not honored maximum path length correctly. + - Fix race condition in CGenericCamera_AVI unit test. ++- Deprecations: ++ - mrpt::system::TParameters is now deprecated. ++- BUG FIXES: ++ - Avoid crash in camera-calib app when clicking "Close" while capturing a live video. + + -- + # Version 2.0.3: Released May 13, 2020 +Index: mrpt-2.0.4/libs/apps/src/RawlogGrabberApp_unittest.cpp +=== +--- mrpt-2.0.4.orig/libs/apps/src/RawlogGrabberApp_unittest.cpp mrpt-2.0.4/libs/apps/src/RawlogGrabberApp_unittest.cpp +@@ -13,6 +13,7 @@ + #include + #include + #include ++#include + #include + + #if MRPT_HAS_FFMPEG && MRPT_HAS_OPENCV +@@ -21,6 +22,15 @@ + TEST(RawlogGrabberApp, DISABLED_CGenericCamera_AVI) + #endif + { ++ // This particular unit test is REALLY problematic for some reason on build ++ // farms. It's safer to just disable it in this case: ++ if (::getenv("DEB_BUILD_ARCH") || ::getenv("DEB_BUILD_MAINT_OPTIONS")) ++ { ++ std::cerr << "Warning: Disabling test since we are building a Debian " ++ "package.\n"; ++ return; ++ } ++ + using namespace std::string_literals; + + const std::string ini_fil = diff -Nru mrpt-2.0.4/debian/patches/series mrpt-2.0.4/debian/patches/series --- mrpt-2.0.4/debian/patches/series2020-06-20 17:24:00.0 +0200 +++ mrpt-2.0.4/debian/patches/series2020-07-17 08:14:20.0 +0200 @@ -1 +1,2 @@ fix-inconsistent-appstream-metadata-license.diff +e84511500276d38d3eeff0b220e8d45e0d74fc93.patch diff -Nru mrpt-2.0.4/debian/rules mrpt-2.0.4/debian/rules --- mrpt-2.0.4/debian/rules 2020-06-20 17:24:00.0 +0200 +++ mrpt-2.0.4/debian/rules 2020-07-17 08:14:20.0 +0200 @@ -68,6 +68,8 @@ # Show CPU flags, to help debugging unit test crashes related to # illegal instructions, etc. cat /proc/cpuinfo + # Show env vars for debugging: + env # Autoconfigure step: dh_auto_configure -- $(CMAKE_FLAGS)
Bug#964044: mrpt: FTBFS: test failure
Hello, On Mon, 13 Jul 2020 21:20:41 +0200 Sebastian Ramacher wrote: > On 2020-06-30 22:37:57 +0200, Sebastian Ramacher wrote: > > Source: mrpt > > Version: 1:2.0.4-1 > > Severity: serious > > Tags: ftbfs > > Justification: fails to build from source (but built successfully in the > > past) > > > > binNMUs of mrpt failed to build: > > | [--] 1 test from RawlogGrabberApp > > | [ RUN ] RawlogGrabberApp.CGenericCamera_AVI > > | [CCameraSensor::initialize] Opening camera... > > | [CCameraSensor::initialize] FFmpeg stream: > > /<>/share/mrpt/datasets/dummy_video.avi... > > | [CCameraSensor::initialize] Done! > > | Rawlog grabbed objects: 1 > > | /<>/libs/apps/src/RawlogGrabberApp_unittest.cpp:94: Failure > > | Expected: (app.rawlog_saved_objects) >= (REQUIRED_GRAB_OBS), actual: 1 vs > > 3 > > | [ FAILED ] RawlogGrabberApp.CGenericCamera_AVI (1511 ms) > > | [--] 1 test from RawlogGrabberApp (1511 ms total) > > > > See > > https://buildd.debian.org/status/fetch.php?pkg=mrpt&arch=amd64&ver=1%3A2.0.4-1%2Bb1&stamp=1593549279&raw=0 > > for example > > This issue has been fixed upstream: > https://github.com/MRPT/mrpt/commit/15234dc335c2413e3fd41021f7511f1d36fe915b. > Could you please apply the fix to the Debian package so that > ros-geometry2 transition can be completed? Thanks looks like that commit is already part of 2.0.4? G. > > Cheers > -- > Sebastian Ramacher
Bug#964044: mrpt: FTBFS: test failure
On 2020-06-30 22:37:57 +0200, Sebastian Ramacher wrote: > Source: mrpt > Version: 1:2.0.4-1 > Severity: serious > Tags: ftbfs > Justification: fails to build from source (but built successfully in the past) > > binNMUs of mrpt failed to build: > | [--] 1 test from RawlogGrabberApp > | [ RUN ] RawlogGrabberApp.CGenericCamera_AVI > | [CCameraSensor::initialize] Opening camera... > | [CCameraSensor::initialize] FFmpeg stream: > /<>/share/mrpt/datasets/dummy_video.avi... > | [CCameraSensor::initialize] Done! > | Rawlog grabbed objects: 1 > | /<>/libs/apps/src/RawlogGrabberApp_unittest.cpp:94: Failure > | Expected: (app.rawlog_saved_objects) >= (REQUIRED_GRAB_OBS), actual: 1 vs 3 > | [ FAILED ] RawlogGrabberApp.CGenericCamera_AVI (1511 ms) > | [--] 1 test from RawlogGrabberApp (1511 ms total) > > See > https://buildd.debian.org/status/fetch.php?pkg=mrpt&arch=amd64&ver=1%3A2.0.4-1%2Bb1&stamp=1593549279&raw=0 > for example This issue has been fixed upstream: https://github.com/MRPT/mrpt/commit/15234dc335c2413e3fd41021f7511f1d36fe915b. Could you please apply the fix to the Debian package so that ros-geometry2 transition can be completed? Thanks Cheers -- Sebastian Ramacher signature.asc Description: PGP signature
Bug#964044: mrpt: FTBFS: test failure
Source: mrpt Version: 1:2.0.4-1 Severity: serious Tags: ftbfs Justification: fails to build from source (but built successfully in the past) binNMUs of mrpt failed to build: | [--] 1 test from RawlogGrabberApp | [ RUN ] RawlogGrabberApp.CGenericCamera_AVI | [CCameraSensor::initialize] Opening camera... | [CCameraSensor::initialize] FFmpeg stream: /<>/share/mrpt/datasets/dummy_video.avi... | [CCameraSensor::initialize] Done! | Rawlog grabbed objects: 1 | /<>/libs/apps/src/RawlogGrabberApp_unittest.cpp:94: Failure | Expected: (app.rawlog_saved_objects) >= (REQUIRED_GRAB_OBS), actual: 1 vs 3 | [ FAILED ] RawlogGrabberApp.CGenericCamera_AVI (1511 ms) | [--] 1 test from RawlogGrabberApp (1511 ms total) See https://buildd.debian.org/status/fetch.php?pkg=mrpt&arch=amd64&ver=1%3A2.0.4-1%2Bb1&stamp=1593549279&raw=0 for example Cheers -- Sebastian Ramacher signature.asc Description: PGP signature