Skip to content

[libcxx] [ci] Don't install wget in the Windows jobs #95215

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

mstorsjo
Copy link
Member

Nothing uses wget - only curl is used, and that's available out of the box.

Nothing uses wget - only curl is used, and that's available out
of the box.
@mstorsjo mstorsjo requested a review from philnik777 June 12, 2024 09:42
@llvmbot llvmbot added libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. github:workflow labels Jun 12, 2024
@llvmbot
Copy link
Member

llvmbot commented Jun 12, 2024

@llvm/pr-subscribers-github-workflow

@llvm/pr-subscribers-libcxx

Author: Martin Storsjö (mstorsjo)

Changes

Nothing uses wget - only curl is used, and that's available out of the box.


Full diff: https://github.com/llvm/llvm-project/pull/95215.diff

1 Files Affected:

  • (modified) .github/workflows/libcxx-build-and-test.yaml (+1-1)
diff --git a/.github/workflows/libcxx-build-and-test.yaml b/.github/workflows/libcxx-build-and-test.yaml
index 44a3d79c72c0a..9a3ad9357c733 100644
--- a/.github/workflows/libcxx-build-and-test.yaml
+++ b/.github/workflows/libcxx-build-and-test.yaml
@@ -209,7 +209,7 @@ jobs:
       - uses: actions/checkout@v4
       - name: Install dependencies
         run: |
-          choco install -y ninja wget
+          choco install -y ninja
           pip install psutil
       - name: Install a current LLVM
         if: ${{ matrix.mingw != true }}

@ldionne ldionne merged commit 27ac46e into llvm:main Jun 12, 2024
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github:workflow libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants