Hi again Gianfranco, I think I've (properly) fixed the issues in big-endian architectures for one set of tests. It would be great if you could launch a build in a test machine to confirm it...
The patch is in [1]. You could test it over release 1.4.0 (*without* the latest patch which, if you recall it, just put a #if 0 around the failing tests!), or just grab the whole thing from git master [2]. >From Debian logs, I see that there is one more test that fails in *some* big endian architectures. I'm almost sure it could be debugged by running it with gdb. In these last weeks I managed to create a system to run unit tests under gdb as part of the Debian build, but it's disabled by default because it caused problems in armhf. You could also uncomment it (see lines [3] of debian/rules) to see if we can get more useful info about potential failures... Just replace MRPT_TEST_TARGET = test with: MRPT_TEST_TARGET = test_gdb Thanks for your support!! [1] https://github.com/MRPT/mrpt/commit/e791599a30a0b60b551ee3e31225609b1a798a39 [2] https://github.com/MRPT/mrpt [3] https://github.com/MRPT/mrpt/blob/b346bbbadbcf0f582c078c9e975f0ad4e0125565/packaging/debian/rules#L23 On Mon, Jul 4, 2016 at 12:13 PM, Gianfranco Costamagna <locutusofb...@debian.org> wrote: > Hi, > > >>lease, find the workaround (not solution!) commit in [1]. Please, if > >>possible, apply it directly over the current v1.4.0 Debian package to >>unblock building in big endian platforms. It would be great if you >>could sponsor the update in Debian, not only in Ubuntu. >> >>If I find spare time to work in a real solution, I'll contact you just >>in case you could help me testing the patches in porter machines... > > > I can sponsor whatever you give me, a dsc, a tarball of debian packaging > directory, > whatever (a git snapshot) > > > Right now, I applied the two commits as patches, and the fix for > breaks+replaces > fields, and I uploaded it in Ubuntu (to check if everything is correct) > > I called it ~build2 [1], so on the Debian upload it will be overridden > automatically > by the auto import robot > > > here [2] > > [1] https://launchpad.net/ubuntu/+source/mrpt/1:1.4.0-1build2 > > [2] > http://launchpadlibrarian.net/270793330/mrpt_1%3A1.4.0-1build1_1%3A1.4.0-1build2.diff.gz > > I'm looking the build logs, if you can give me a dsc file I'll sponsor it in > a matter of minutes. > > If you don't change the version, just send me a tarball of the debian > directory, it should be enough for me! > > thanks for "fixing" :) > > Gianfranco -- ___________________________________________________________ Jose Luis Blanco-Claraco CITE-IV 1.05 Universidad de Almería, Departamento de Ingeniería 04120 Almería (Spain) http://www.ual.es/~jlblanco/ ___________________________________________________________
