Skip to contentSkip to conversation

libjpeg-turbo: fix build on aarch64 or non-intel build hosts

PATCH In review Tristan Van Berkom 2 participants 2 messages Mar 29, 2017

lore
Share & tools
lore
#0 lore raw

PATCH v3 1/1 libjpeg-turbo: fix build on aarch64 or non-intel build hosts

From f54c719dc0ca3444bc579663b7326d016bf2e1e7 Mon Sep 17 00:00:00 2001
From: Tristan Van Berkom <[email protected]>
Date: Fri, 24 Mar 2017 15:37:36 +0000
Subject: [PATCH] libjpeg-turbo: Support building on non intel targets.

nasm is intel specific, however libjpeg-turbo is still desirable on other architectures.

Without this patch, it would seem that nasm is built on an intel host and then ignored when compiling libjpeg-turbo for an arm target, however it has libjpeg-turbo requiring nasm be built when building on an arm/aarch64 host.

This fixes the issue so that nasm-native is not required on the host when building for an aarch64 target.

---
 meta/recipes-graphics/jpeg/libjpeg-turbo_1.5.1.bb | 3 ++-

 1 file changed, 2 insertions(+), 1 deletion(-)
meta/recipes-graphics/jpeg/libjpeg-turbo_1.5.1.bb +0 -0
index 70d38f4a1a..a650d6af2d 100644
@@ -7,7 +7,8 @@ LIC_FILES_CHKSUM = "file://cdjpeg.h;endline=13;md5=05bab7c7ad899d85bfba60da1a127

                     file://jpeglib.h;endline=16;md5=f67d70e547a2662c079781c72f877f72 \
                     file://djpeg.c;endline=11;md5=b90b6d2b4119f9e5807cd273f525d2af \
 "
-DEPENDS = "nasm-native"
+DEPENDS_append_x86-64 = " nasm-native"
+DEPENDS_append_x86    = " nasm-native"

 SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz \
            file://fix-mips.patch"
--
2.11.0

#1 lore raw

✗ patchtest: failure for libjpeg-turbo: fix build on aarch64 or non-intel build hosts (rev2)

== Series Details ==

Series: libjpeg-turbo: fix build on aarch64 or non-intel build hosts (rev2)
Revision: 2
URL : https://patchwork.openembedded.org/series/6049/
State : failure

== Summary ==

Thank you for submitting this patch series to OpenEmbedded Core. This is an automated response. Several tests have been executed on the proposed series by patchtest resulting in the following failures:

* Issue Series cannot be parsed correctly due to malformed diff lines [test_mbox_format]
Suggested fix Create the series again using git-format-patch and ensure it can be applied using git am
Diff line //jpeglib.h;endline=16;md5=f67d70e547a2662c079781c72f877f72 \

* Issue Series does not apply on top of target branch [test_series_merge_on_head]
Suggested fix Rebase your series on top of targeted branch
Targeted branch master (currently at d68a86d87a)

If you believe any of these test results are incorrect, please reply to the mailing list ([email protected]) raising your concerns. Otherwise we would appreciate you correcting the issues and submitting a new version of the patchset if applicable. Please ensure you add/increment the version number when sending the new version (i.e. [PATCH] -> [PATCH v2] -> [PATCH v3] -> ...).

---

Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite:     http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe