Skip to content

[libc][workflows] fix pre-commit fullbuild dependency problem #119441

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 5 commits into from
Dec 10, 2024

Conversation

SchrodingerZhu
Copy link
Contributor

@SchrodingerZhu SchrodingerZhu commented Dec 10, 2024

  • fix the issue where linux-headers-generic cannot be installed

@SchrodingerZhu SchrodingerZhu changed the title [libc][workflows] try fix build [libc][workflows] fix pre-commit fullbuild and re-enable gcc Dec 10, 2024
@llvmbot
Copy link
Member

llvmbot commented Dec 10, 2024

@llvm/pr-subscribers-libc

@llvm/pr-subscribers-github-workflow

Author: Schrodinger ZHU Yifan (SchrodingerZhu)

Changes

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

1 Files Affected:

  • (modified) .github/workflows/libc-fullbuild-tests.yml (+3-4)
diff --git a/.github/workflows/libc-fullbuild-tests.yml b/.github/workflows/libc-fullbuild-tests.yml
index b07e94244c2e8c..357e9064e90100 100644
--- a/.github/workflows/libc-fullbuild-tests.yml
+++ b/.github/workflows/libc-fullbuild-tests.yml
@@ -18,9 +18,8 @@ jobs:
         include:
           - c_compiler: clang
             cpp_compiler: clang++
-          # TODO: add back gcc build when it is fixed
-          # - c_compiler: gcc
-          #   cpp_compiler: g++
+          - c_compiler: gcc
+            cpp_compiler: g++
     steps:
     - uses: actions/checkout@v4
     
@@ -44,7 +43,7 @@ jobs:
     - name: Prepare dependencies (Ubuntu)
       run: |
         sudo apt-get update
-        sudo apt-get install -y libmpfr-dev libgmp-dev libmpc-dev ninja-build linux-headers-generic linux-libc-dev
+        sudo apt-get install -y libmpfr-dev libgmp-dev libmpc-dev ninja-build linux-libc-dev
         sudo ln -sf /usr/include/$(uname -p)-linux-gnu/asm /usr/include/asm
 
     - name: Set reusable strings

@llvmbot llvmbot added the libc label Dec 10, 2024
@SchrodingerZhu SchrodingerZhu changed the title [libc][workflows] fix pre-commit fullbuild and re-enable gcc [libc][workflows] fix pre-commit fullbuild dependency problem Dec 10, 2024
@SchrodingerZhu
Copy link
Contributor Author

merge this as it is a minor fix

@SchrodingerZhu SchrodingerZhu merged commit 24162bd into llvm:main Dec 10, 2024
7 checks passed
@SchrodingerZhu SchrodingerZhu deleted the libc/fix-ci branch December 10, 2024 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants