Bug#895469: openjfx FTBFS on armel: invalid instructions

2018-06-11 Thread Emmanuel Bourg
Le 11/06/2018 à 15:10, peter green a écrit :

> Debdiff for what I uploaded to raspbian is available at
> http://debdiffs.raspbian.org/main/o/openjfx/openjfx_8u141-b14-3%2brpi1.debdiff

Thanks a lot for the help Peter!

It looks like JSStringRef.cpp was eventually fixed upstream (at least in
OpenJFX 9.0.1+11 [1], but not on the 8u-dev branch yet).

I'll try to upload OpenJFX 8u171 with this patch before upgrading to
OpenJFX 11.

[1]
https://salsa.debian.org/java-team/openjfx9/blob/master/modules/javafx.web/src/main/native/Source/JavaScriptCore/API/JSStringRef.cpp#L65



Bug#895469: openjfx FTBFS on armel: invalid instructions

2018-06-11 Thread peter green

movw and movt are armv7 only, so this is also affecting Raspbian. To fix both 
armel and raspbian the conditional should be <7

I whipped up a patch for that. I then ran into another issue, the compiler was 
bitching about uint16_t vs char16_t issues so I whipped up a patch for that too.

Debdiff for what I uploaded to raspbian is available at 
http://debdiffs.raspbian.org/main/o/openjfx/openjfx_8u141-b14-3%2brpi1.debdiff 
. No intent to NMU in Debian.



Bug#895469: openjfx FTBFS on armel: invalid instructions

2018-04-11 Thread Adrian Bunk
Source: openjfx
Version: 8u141-b14-3
Severity: serious
Tags: buster sid patch

https://buildd.debian.org/status/fetch.php?pkg=openjfx=armel=8u161-b12-1=1523051663=0

...
/tmp/ccxxsDUf.s: Assembler messages:
/tmp/ccxxsDUf.s:311: Error: selected processor does not support `movw 
r8,#:lower16:.Lllint_op_enter-.LrelativePCBase' in ARM mode
/tmp/ccxxsDUf.s:312: Error: selected processor does not support `movt 
r8,#:upper16:.Lllint_op_enter-.LrelativePCBase' in ARM mode
/tmp/ccxxsDUf.s:316: Error: selected processor does not support `movw 
r8,#:lower16:.Lllint_op_get_scope-.LrelativePCBase' in ARM mode
/tmp/ccxxsDUf.s:317: Error: selected processor does not support `movt 
r8,#:upper16:.Lllint_op_get_scope-.LrelativePCBase' in ARM mode
...


The baseline of the armel port was raised a few months ago,
this can be fixed with:

debian/patches/26-disable-webkit-jit-for-armv4.patch
-+#if CPU(ARM) && WTF_ARM_ARCH_VERSION < 5
++#if CPU(ARM) && WTF_ARM_ARCH_VERSION <= 5