search

inflation / jpegxl-rs

GNU General Public License v3.0
56 stars 11 forks source link

Unable to build crates.io `jpegxl-src` #44

Closed joshstoik1 closed 5 months ago

joshstoik1 commented 5 months ago

Hi there,

I'm unable to build the latest jpegxl-src release from crates.io. I suspect it has to do with the excluded assets, as I have no such trouble building it from the cloned git repo.

  -- The C compiler identification is GNU 12.2.0
  -- The CXX compiler identification is GNU 12.2.0
  -- Detecting C compiler ABI info
  -- Detecting C compiler ABI info - done
  -- Check for working C compiler: /usr/bin/cc - skipped
  -- Detecting C compile features
  -- Detecting C compile features - done
  -- Detecting CXX compiler ABI info
  -- Detecting CXX compiler ABI info - done
  -- Check for working CXX compiler: /usr/bin/c++ - skipped
  -- Detecting CXX compile features
  -- Detecting CXX compile features - done
  -- CMAKE_SYSTEM_PROCESSOR is x86_64
  -- Performing Test CXX_FUZZERS_SUPPORTED
  -- Performing Test CXX_FUZZERS_SUPPORTED - Failed
  -- Performing Test CXX_MACRO_PREFIX_MAP
  -- Performing Test CXX_MACRO_PREFIX_MAP - Success
  -- Performing Test CXX_NO_RTTI_SUPPORTED
  -- Performing Test CXX_NO_RTTI_SUPPORTED - Success
  -- Found PkgConfig: /usr/bin/pkg-config (found version "1.8.1") 
  -- tcmalloc version  -- tcmalloc 2.8.0 disabled due to https://github.com/gperftools/gperftools/issues/1204
  -- Performing Test JXL_HWY_DISABLED_TARGETS_FORCED
  -- Performing Test JXL_HWY_DISABLED_TARGETS_FORCED - Failed
  -- Compiled IDs C:GNU, C++:GNU
  -- Disabled AVX512 (set JPEGXL_ENABLE_AVX512 to enable it)
  -- Disabled AVX512_SPR (set JPEGXL_ENABLE_AVX512_SPR to enable it)
  -- Disabled AVX512_ZEN4 (set JPEGXL_ENABLE_AVX512_ZEN4 to enable it)
  -- Performing Test CMAKE_HAVE_LIBC_PTHREAD
  -- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
  -- Found Threads: TRUE  
  -- Performing Test ATOMICS_LOCK_FREE_INSTRUCTIONS
  -- Performing Test ATOMICS_LOCK_FREE_INSTRUCTIONS - Success
  -- Performing Test HWY_EMSCRIPTEN
  -- Performing Test HWY_EMSCRIPTEN - Failed
  -- Performing Test HWY_RISCV
  -- Performing Test HWY_RISCV - Failed
  -- Looking for sys/auxv.h
  -- Looking for sys/auxv.h - found
  -- Looking for asm/hwcap.h
  -- Looking for asm/hwcap.h - not found
  -- Build type is 'Debug'
  -- Performing Test BROTLI_EMSCRIPTEN
  -- Performing Test BROTLI_EMSCRIPTEN - Failed

[snip]

  -- Configuring incomplete, errors occurred!

  --- stderr
  -- Compiler is not EMSCRIPTEN
  CMake Error at CMakeLists.txt:496 (add_subdirectory):
    add_subdirectory given source "tools" which is not an existing directory.

Please let me know if you need any other information!

inflation commented 5 months ago

Hmmm. Good catch. It seems libjxl v0.10 or later changes how it compile complementary tools. In before, it won't include the subdirectory if we turn off the option, but now it needs the directory no matter what. I'll push a new version to fix it asap.

joshstoik1 commented 5 months ago

The build errors have gone away with jpegxl-sys 0.10.2+libjxl-0.10.2/jpegxl-src v0.10.4.

Thanks for the quick turnaround!