Try to fix CI (#2918)

Fixes a few issues that have been cropping up: * Update `rustup` on Windows to latest to skip over the 1.24.1 installed on GitHub Actions which can fail to install. * Remove the no-longer-needed `define-llvm-env` action * Install generic llvm/lldb packges instead of specific ones that may migrate in versions over time.
veracruz-project · May 21, 2021 · 7d20368 · 7d20368
1 parent 8b9057a
commit 7d20368
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 40 deletions.
diff --git a/.github/actions/define-llvm-env/README.md b/.github/actions/define-llvm-env/README.md
diff --git a/.github/actions/define-llvm-env/action.yml b/.github/actions/define-llvm-env/action.yml
diff --git a/.github/actions/define-llvm-env/main.js b/.github/actions/define-llvm-env/main.js
diff --git a/.github/actions/install-rust/main.js b/.github/actions/install-rust/main.js
@@ -6,11 +6,10 @@ function set_env(name, val) {
   fs.appendFileSync(process.env['GITHUB_ENV'], `${name}=${val}\n`)
 }
 
-if (process.platform === 'darwin') {
-  child_process.execSync(`curl https://sh.rustup.rs | sh -s -- -y --default-toolchain=none --profile=minimal`);
-  const bindir = `${process.env.HOME}/.cargo/bin`;
-  fs.appendFileSync(process.env['GITHUB_PATH'], `${bindir}\n`);
-  process.env.PATH = `${process.env.PATH}:${bindir}`;
+// Needed for now to get 1.24.2 which fixes a bug in 1.24.1 that causes issues
+// on Windows.
+if (process.platform === 'win32') {
+  child_process.execFileSync('rustup', ['self', 'update']);
 }
 
 child_process.execFileSync('rustup', ['set', 'profile', 'minimal']);

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -234,7 +234,6 @@ jobs:
     - uses: ./.github/actions/install-rust
       with:
         toolchain: ${{ matrix.rust }}
-    - uses: ./.github/actions/define-llvm-env
 
     - name: Install libclang
       # Note: libclang is pre-installed on the macOS and linux images.
@@ -277,21 +276,18 @@ jobs:
 
     # Test debug (DWARF) related functionality.
     - run: |
-        sudo apt-get update && sudo apt-get install -y gdb lldb
+        sudo apt-get update && sudo apt-get install -y gdb lldb llvm
         cargo test test_debug_dwarf -- --ignored --test-threads 1
       if: matrix.os == 'ubuntu-latest'
       env:
         RUST_BACKTRACE: 1
-        LLDB: /usr/bin/lldb
 
     # Test debug (DWARF) related functionality on new backend.
     - run: |
-        sudo apt-get update && sudo apt-get install -y gdb lldb
         cargo test test_debug_dwarf -- --ignored --test-threads 1 --test debug::
       if: matrix.os == 'ubuntu-latest'
       env:
         RUST_BACKTRACE: 1
-        LLDB: /usr/bin/lldb
 
     # Test uffd functionality on Linux
     - run: |
@@ -311,7 +307,7 @@ jobs:
         RUST_BACKTRACE: 1
 
   # Perform all tests (debug mode) for `wasmtime` with the old x86 backend.
-  test_x64:
+  test_x86:
     name: Test old x86 backend
     runs-on: ubuntu-latest
     steps:
@@ -321,7 +317,6 @@ jobs:
     - uses: ./.github/actions/install-rust
       with:
         toolchain: stable
-    - uses: ./.github/actions/define-llvm-env
 
     # Install wasm32 targets in order to build various tests throughout the
     # repo.