Hi,

>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].

ok, rebased with current debian/unstable package and build good

>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

ok



I did grab the package from unstable, added the commit above, and did a 
complete build.
It didn't fail on s390x, so I don't know how to trigger that failure.


thanks,
G.

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/

___________________________________________________________

Reply via email to