Skip to contentSkip to conversation

autobuild-run: Add an exception for ctng toolchain armv5 with Guile package

PATCH New Romain Naour 2 participants 2 messages Nov 11, 2016

lore
Share & tools
lore

Romain Naour

author
#0 lore raw

PATCH 1/1 autobuild-run: Add an exception for ctng toolchain armv5 with Guile package

Like for CS 2014.05 ARM toolchain, the ctng toolchain trigger an assembler error when Guile package is compiled with -Os.

[1] https://git.buildroot.net/buildroot/commit/?id=401c47e44b26e97db1c12cce0a42c62cfb72a9b9

Signed-off-by Romain Naour <[email protected]>
---
 scripts/autobuild-run | 4 ++++
 1 file changed, 4 insertions(+)
scripts/autobuild-run +4 -0
index 2583098..65276bd 100755
@@ -424,6 +424,10 @@ def fixup_config(**kwargs):
     if 'BR2_PACKAGE_LTTNG_TOOLS=y\n' in configlines and \
        'BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/armv5-ctng-linux-gnueabi.tar.xz"\n' in configlines:
         return False
+    # The ctng toolchain tigger an assembler error with guile package when compiled with -Os (same issue as for CS ARM 2014.05-29)
+    if 'BR2_PACKAGE_GUILE=y\n' in configlines and 'BR2_OPTIMIZE_S=y\n' in configlines and \
+       'BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/armv5-ctng-linux-gnueabi.tar.xz"\n' in configlines:
+        return False
     # The ctng toolchain is affected by PR58854
     if 'BR2_PACKAGE_LTTNG_TOOLS=y\n' in configlines and \
        'BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/armv6-ctng-linux-uclibcgnueabi.tar.xz"\n' in configlines:
Signature · 1 line
2.5.5
Applied
#1 lore raw

PATCH 1/1 autobuild-run: Add an exception for ctng toolchain armv5 with Guile package

Hello,

Quoted 9 lines · Romain Naour

Like for CS 2014.05 ARM toolchain, the ctng toolchain trigger an assembler error when Guile package is compiled with -Os.

[1] https://git.buildroot.net/buildroot/commit/?id=401c47e44b26e97db1c12cce0a42c62cfb72a9b9

Signed-off-by Romain Naour <[email protected]>
---
 scripts/autobuild-run | 4 ++++
 1 file changed, 4 insertions(+)

Applied, thanks! However, it will take some time to get everyone to update their copy of the autobuild-run script.

Thanks,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com