You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "lidavidm (via GitHub)" <gi...@apache.org> on 2025/02/03 05:18:13 UTC

[I] ci: update clang version in nightly tests [arrow-adbc]

lidavidm opened a new issue, #2497:
URL: https://github.com/apache/arrow-adbc/issues/2497

   ### What feature or improvement would you like to see?
   
   https://github.com/apache/arrow-adbc/actions/runs/13103925294/job/36555718277


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] ci: update clang version in nightly tests [arrow-adbc]

Posted by "kou (via GitHub)" <gi...@apache.org>.
kou commented on issue #2497:
URL: https://github.com/apache/arrow-adbc/issues/2497#issuecomment-2632789910

   I agree with this is an upstream problem.
   
   How about specifying the latest LLVM major version explicitly as a workaround? 
   
   ```diff
   diff --git a/ci/docker/cpp-clang-latest.dockerfile b/ci/docker/cpp-clang-latest.dockerfile
   index a1b76a552..731c60343 100644
   --- a/ci/docker/cpp-clang-latest.dockerfile
   +++ b/ci/docker/cpp-clang-latest.dockerfile
   @@ -20,14 +20,28 @@ ARG GO
    
    RUN export DEBIAN_FRONTEND=noninteractive && \
        apt-get update -y && \
   -    apt-get install -y curl gnupg && \
   -    echo "deb http://apt.llvm.org/bookworm/ llvm-toolchain-bookworm main" \
   +    apt-get install -y curl gnupg jq && \
   +    # llvmorg-19.1.7 ->
   +    # 19.1.7 ->
   +    # 19
   +    latest_llvm_major_version=$( \
   +      curl https://api.github.com/repos/llvm/llvm-project/releases/latest | \
   +        jq -r .tag_name | \
   +        cut -d- -f2 | \
   +        cut -d. -f1) && \
   +    echo "deb http://apt.llvm.org/bookworm/ llvm-toolchain-bookworm-${latest_llvm_major_version} main" \
             > /etc/apt/sources.list.d/llvm.list && \
   -    echo "deb-src http://apt.llvm.org/bookworm/ llvm-toolchain-bookworm main" \
   +    echo "deb-src http://apt.llvm.org/bookworm/ llvm-toolchain-bookworm-${latest_llvm_major_version} main" \
             >> /etc/apt/sources.list.d/llvm.list && \
        curl -L https://apt.llvm.org/llvm-snapshot.gpg.key | apt-key add - && \
        apt-get update -y && \
   -    apt-get install -y clang libc++abi-dev libc++-dev libomp-dev && \
   +    apt-get install -y \
   +      clang-${latest_llvm_major_version} \
   +      libc++abi-${latest_llvm_major_version}-dev \
   +      libc++-${latest_llvm_major_version}-dev \
   +      libomp-${latest_llvm_major_version}-dev && \
   +    ln -s clang clang-${latest_llvm_major_version} && \
   +    ln -s clang++ clang++-${latest_llvm_major_version} && \
        apt-get clean
    
    RUN export DEBIAN_FRONTEND=noninteractive && \
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] ci: update clang version in nightly tests [arrow-adbc]

Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #2497:
URL: https://github.com/apache/arrow-adbc/issues/2497#issuecomment-2630885789

   Hmm, odd, it seems it's because `clang` has been aliased to clang-20 (wrongly? since LLVM 20 isn't yet out)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] ci: update clang version in nightly tests [arrow-adbc]

Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #2497:
URL: https://github.com/apache/arrow-adbc/issues/2497#issuecomment-2632922395

   Oh I see, you have a clever cURL command :)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] ci: update clang version in nightly tests [arrow-adbc]

Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #2497:
URL: https://github.com/apache/arrow-adbc/issues/2497#issuecomment-2632609500

   @kou do you have any idea what's going on here? It seems LLVM's deb repository has `clang` pointing to `clang-20` which doesn't actually exist yet. (Is this affecting Arrow's version of this test?)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] ci: update clang version in nightly tests [arrow-adbc]

Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #2497:
URL: https://github.com/apache/arrow-adbc/issues/2497#issuecomment-2632917953

   Yeah, I thought about that...that means we have to remember to update it though which is unfortunate


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] ci: update clang version in nightly tests [arrow-adbc]

Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm closed issue #2497: ci: update clang version in nightly tests
URL: https://github.com/apache/arrow-adbc/issues/2497


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] ci: update clang version in nightly tests [arrow-adbc]

Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #2497:
URL: https://github.com/apache/arrow-adbc/issues/2497#issuecomment-2630944051

   > N: Can't select versions from package 'clang-20' as it is purely virtual
   
   Probably an upstream problem, let's wait and see for now...


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org