[Buildroot] [PATCH 1/1] package/tesseract-ocr: override neon check

[Buildroot] [PATCH 1/1] package/tesseract-ocr: override neon check

[Fabrice Fontaine]

Override neon check to avoid the following build failure with uclibc-ng
raised since bump to version 5.0.1 in commit
df3f0c226fccd8d6143b09a1e05a0f5f23587136:

src/arch/simddetect.cpp:61:14: fatal error: sys/auxv.h: No such file or directory
   61 | #    include <sys/auxv.h>
      |              ^~~~~~~~~~~~

It should be noted that a patch to properly fix this build failure was
rejected by upstream as "ARM support without NEON would result in a very
slow Tesseract": https://github.com/tesseract-ocr/tesseract/pull/3717

So, a follow-up patch adding a dependency on !BR2_TOOLCHAIN_USES_UCLIBC
for all CPU architectures or only for ARM without NEON could be
approriate.

Fixes:
 - http://autobuild.buildroot.org/results/0bac6491188b6681df2ca13eb230706e83cbb130

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/tesseract-ocr/tesseract-ocr.mk | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/package/tesseract-ocr/tesseract-ocr.mk b/package/tesseract-ocr/tesseract-ocr.mk
index 7e23e64136..a7498931fb 100644
--- a/package/tesseract-ocr/tesseract-ocr.mk
+++ b/package/tesseract-ocr/tesseract-ocr.mk
@@ -24,6 +24,12 @@ TESSERACT_OCR_CONF_ENV = \
 TESSERACT_OCR_CONF_OPTS = \
 	--disable-opencl
 
+ifeq ($(BR2_ARM_CPU_HAS_NEON),y)
+TESSERACT_OCR_CONF_ENV += ax_cv_check_cxxflags__mfpu_neon=yes
+else
+TESSERACT_OCR_CONF_ENV += ax_cv_check_cxxflags__mfpu_neon=no
+endif
+
 # Language data files download
 ifeq ($(BR2_PACKAGE_TESSERACT_OCR_LANG_ENG),y)
 TESSERACT_OCR_DATA_FILES += eng.traineddata
-- 
2.34.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/1] package/tesseract-ocr: override neon check

[Arnout Vandecappelle]

On 25/01/2022 23:34, Fabrice Fontaine wrote:
> Override neon check to avoid the following build failure with uclibc-ng
> raised since bump to version 5.0.1 in commit
> df3f0c226fccd8d6143b09a1e05a0f5f23587136:
> 
> src/arch/simddetect.cpp:61:14: fatal error: sys/auxv.h: No such file or directory
>     61 | #    include <sys/auxv.h>
>        |              ^~~~~~~~~~~~
> 
> It should be noted that a patch to properly fix this build failure was
> rejected by upstream as "ARM support without NEON would result in a very
> slow Tesseract": https://github.com/tesseract-ocr/tesseract/pull/3717
> 
> So, a follow-up patch adding a dependency on !BR2_TOOLCHAIN_USES_UCLIBC
> for all CPU architectures or only for ARM without NEON could be
> approriate.
> 
> Fixes:
>   - http://autobuild.buildroot.org/results/0bac6491188b6681df2ca13eb230706e83cbb130
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

  Applied to master, thanks.

  Regards,
  Arnout

> ---
>   package/tesseract-ocr/tesseract-ocr.mk | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/package/tesseract-ocr/tesseract-ocr.mk b/package/tesseract-ocr/tesseract-ocr.mk
> index 7e23e64136..a7498931fb 100644
> --- a/package/tesseract-ocr/tesseract-ocr.mk
> +++ b/package/tesseract-ocr/tesseract-ocr.mk
> @@ -24,6 +24,12 @@ TESSERACT_OCR_CONF_ENV = \
>   TESSERACT_OCR_CONF_OPTS = \
>   	--disable-opencl
>   
> +ifeq ($(BR2_ARM_CPU_HAS_NEON),y)
> +TESSERACT_OCR_CONF_ENV += ax_cv_check_cxxflags__mfpu_neon=yes
> +else
> +TESSERACT_OCR_CONF_ENV += ax_cv_check_cxxflags__mfpu_neon=no
> +endif
> +
>   # Language data files download
>   ifeq ($(BR2_PACKAGE_TESSERACT_OCR_LANG_ENG),y)
>   TESSERACT_OCR_DATA_FILES += eng.traineddata
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot