mbox series

[0/4] package/ccache: bump version

Message ID 20240713122331.298786-1-heiko.thiery@gmail.com
Headers show
Series package/ccache: bump version | expand

Message

Heiko Thiery July 13, 2024, 12:23 p.m. UTC 
---- 8< ----
/tmp/ccmQFw5y.s: Assembler messages:
/tmp/ccmQFw5y.s:89386: Error: unsupported instruction `vmovdqu'
/tmp/ccmQFw5y.s:89400: Error: unsupported instruction `vmovdqu'
/tmp/ccmQFw5y.s:89464: Error: unsupported instruction `vmovdqu'
/tmp/ccmQFw5y.s:89518: Error: unsupported instruction `vmovdqu'
/tmp/ccmQFw5y.s:89525: Error: unsupported instruction `vmovdqu'
/tmp/ccmQFw5y.s:123938: Error: unsupported instruction `vmovdqu'
/tmp/ccmQFw5y.s:123953: Error: unsupported instruction `vmovdqu'
make[4]: *** [src/third_party/blake3/CMakeFiles/blake3.dir/build.make:160: src/third_party/blake3/CMakeFiles/blake3.dir/blake3_avx512.c.o] Error 1
make[3]: *** [CMakeFiles/Makefile2:406: src/third_party/blake3/CMakeFiles/blake3.dir/all] Error 2
---- 8< ----

To solve the problem above I started bumping the ccache package to
4.10.1. While doing this I observed the problem that ccache tries to
download xxhash by its own and fails.

---- 8< ----
-- Downloading...
   dst='/srv/storage/hthiery/buildroot-2/output/build/host-ccache-4.10.1/_deps/xxhash-subbuild/xxhash-populate-prefix/src/v0.8.2.tar.gz'
   timeout='none'
   inactivity timeout='none'
-- Using src='https://github.com/Cyan4973/xxhash/archive/refs/tags/v0.8.2.tar.gz'
CMake Error at xxhash-subbuild/xxhash-populate-prefix/src/xxhash-populate-stamp/download-xxhash-populate.cmake:170 (message):
  Each download failed!

    error: downloading 'https://github.com/Cyan4973/xxhash/archive/refs/tags/v0.8.2.tar.gz' failed
          status_code: 1
          status_string: "Unsupported protocol"
          log:
          --- LOG BEGIN ---
          Protocol "https" not supported or disabled in libcurl
  Closing connection
          --- LOG END ---
---- 8< ----

So I added support for the host package of xxhash and added this to
the ccache dependencies.

In addition the bundled blake3 version seems not to build correctly on
all machines. See the first error about 'unsupported instruction'.
Therefore the new host package for blake3 is introduced.


v2:
    - change subject (fixed naming from ccmake to ccache)

v3:
    - fix legal-info for ccache and blake3

v4
    - requested changes by Arnout, see history of patches for details

Heiko Thiery (4):
  package/blake3: add new host package
  package/xxhash: add host support
  package/ccache: bump version to 4.10.1
  package/pkg-generic.mk: add variable to skip ccache dependency

 DEVELOPERS                   |  1 +
 package/blake3/blake3.hash   |  3 +++
 package/blake3/blake3.mk     | 22 ++++++++++++++++++++++
 package/ccache/ccache.hash   |  4 ++--
 package/ccache/ccache.mk     | 31 +++++++++++++++++++++----------
 package/cmake/cmake.mk       |  3 +++
 package/fakedate/fakedate.mk |  3 +++
 package/hiredis/hiredis.mk   |  3 +++
 package/lzip/lzip.mk         |  3 +++
 package/pkg-generic.mk       | 13 +++++++------
 package/pkgconf/pkgconf.mk   |  3 +++
 package/skeleton/skeleton.mk |  3 +++
 package/tar/tar.mk           |  3 +++
 package/xxhash/xxhash.mk     | 16 ++++++++++++++++
 package/xz/xz.mk             |  3 +++
 package/zstd/zstd.mk         |  3 +++
 16 files changed, 99 insertions(+), 18 deletions(-)
 create mode 100644 package/blake3/blake3.hash
 create mode 100644 package/blake3/blake3.mk

Comments

Arnout Vandecappelle July 13, 2024, 8:27 p.m. UTC | #1 
On 13/07/2024 14:23, Heiko Thiery wrote:
> ---- 8< ----
> /tmp/ccmQFw5y.s: Assembler messages:
> /tmp/ccmQFw5y.s:89386: Error: unsupported instruction `vmovdqu'
> /tmp/ccmQFw5y.s:89400: Error: unsupported instruction `vmovdqu'
> /tmp/ccmQFw5y.s:89464: Error: unsupported instruction `vmovdqu'
> /tmp/ccmQFw5y.s:89518: Error: unsupported instruction `vmovdqu'
> /tmp/ccmQFw5y.s:89525: Error: unsupported instruction `vmovdqu'
> /tmp/ccmQFw5y.s:123938: Error: unsupported instruction `vmovdqu'
> /tmp/ccmQFw5y.s:123953: Error: unsupported instruction `vmovdqu'
> make[4]: *** [src/third_party/blake3/CMakeFiles/blake3.dir/build.make:160: src/third_party/blake3/CMakeFiles/blake3.dir/blake3_avx512.c.o] Error 1
> make[3]: *** [CMakeFiles/Makefile2:406: src/third_party/blake3/CMakeFiles/blake3.dir/all] Error 2
> ---- 8< ----
> 
> To solve the problem above I started bumping the ccache package to
> 4.10.1. While doing this I observed the problem that ccache tries to
> download xxhash by its own and fails.
> 
> ---- 8< ----
> -- Downloading...
>     dst='/srv/storage/hthiery/buildroot-2/output/build/host-ccache-4.10.1/_deps/xxhash-subbuild/xxhash-populate-prefix/src/v0.8.2.tar.gz'
>     timeout='none'
>     inactivity timeout='none'
> -- Using src='https://github.com/Cyan4973/xxhash/archive/refs/tags/v0.8.2.tar.gz'
> CMake Error at xxhash-subbuild/xxhash-populate-prefix/src/xxhash-populate-stamp/download-xxhash-populate.cmake:170 (message):
>    Each download failed!
> 
>      error: downloading 'https://github.com/Cyan4973/xxhash/archive/refs/tags/v0.8.2.tar.gz' failed
>            status_code: 1
>            status_string: "Unsupported protocol"
>            log:
>            --- LOG BEGIN ---
>            Protocol "https" not supported or disabled in libcurl
>    Closing connection
>            --- LOG END ---
> ---- 8< ----
> 
> So I added support for the host package of xxhash and added this to
> the ccache dependencies.
> 
> In addition the bundled blake3 version seems not to build correctly on
> all machines. See the first error about 'unsupported instruction'.

  Maybe that was also due to the CMAKE_ASM_COMPILER mistake? BTW you should have 
included that fix as part of this series, otherwise the blake3 package wouldn't 
have built. Fortunately I remembered (this time) :-)

  Applied to master, thanks. One change that I'll comment separately.

  Regards,
  Arnout


> Therefore the new host package for blake3 is introduced.
> 
> 
> v2:
>      - change subject (fixed naming from ccmake to ccache)
> 
> v3:
>      - fix legal-info for ccache and blake3
> 
> v4
>      - requested changes by Arnout, see history of patches for details
> 
> Heiko Thiery (4):
>    package/blake3: add new host package
>    package/xxhash: add host support
>    package/ccache: bump version to 4.10.1
>    package/pkg-generic.mk: add variable to skip ccache dependency
> 
>   DEVELOPERS                   |  1 +
>   package/blake3/blake3.hash   |  3 +++
>   package/blake3/blake3.mk     | 22 ++++++++++++++++++++++
>   package/ccache/ccache.hash   |  4 ++--
>   package/ccache/ccache.mk     | 31 +++++++++++++++++++++----------
>   package/cmake/cmake.mk       |  3 +++
>   package/fakedate/fakedate.mk |  3 +++
>   package/hiredis/hiredis.mk   |  3 +++
>   package/lzip/lzip.mk         |  3 +++
>   package/pkg-generic.mk       | 13 +++++++------
>   package/pkgconf/pkgconf.mk   |  3 +++
>   package/skeleton/skeleton.mk |  3 +++
>   package/tar/tar.mk           |  3 +++
>   package/xxhash/xxhash.mk     | 16 ++++++++++++++++
>   package/xz/xz.mk             |  3 +++
>   package/zstd/zstd.mk         |  3 +++
>   16 files changed, 99 insertions(+), 18 deletions(-)
>   create mode 100644 package/blake3/blake3.hash
>   create mode 100644 package/blake3/blake3.mk
>