diff options
| author | Tom Stellard <tstellar@redhat.com> | 2022-08-02 02:02:15 -0700 |
|---|---|---|
| committer | Tom Stellard <tstellar@redhat.com> | 2022-08-02 02:02:15 -0700 |
| commit | 0214d986fb64f89cc75289199100475af2e73870 (patch) | |
| tree | db74edc53cb13d6aebb4ca27c681cdf467d5b117 | |
| parent | 08c239e29fb4ebc4970fbb041a027f8fffcc1a21 (diff) | |
workflows: Remove symbol versions from libclang.so in the libclang ABI test
Now that the symbol version for libclang.so changes for each release again,
we need to remove the symbol versions from the shared library in order
for the ABI checker to be able to compare with an older version of the
shared library.
| -rw-r--r-- | .github/workflows/libclang-abi-tests.yml | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/.github/workflows/libclang-abi-tests.yml b/.github/workflows/libclang-abi-tests.yml index 90e816b0eaed..1b063f6430a1 100644 --- a/.github/workflows/libclang-abi-tests.yml +++ b/.github/workflows/libclang-abi-tests.yml @@ -124,12 +124,10 @@ jobs: - name: Dump ABI run: | parallel abi-dumper -lver ${{ matrix.ref }} -skip-cxx -public-headers ./install/include/${{ needs.abi-dump-setup.outputs.ABI_HEADERS }} -o {}-${{ matrix.ref }}.abi ./build/lib/{} ::: ${{ needs.abi-dump-setup.outputs.ABI_LIBS }} - # Remove symbol versioning from dumps, so we can compare across major - # versions. We don't need to do this for libclang.so since its ABI - # is stable across major releases and the symbol versions don't change. - if [ -e libclang-cpp.so-${{ matrix.ref }}.abi ]; then - sed -i 's/LLVM_[0-9]\+/LLVM_NOVERSION/' libclang-cpp.so-${{ matrix.ref }}.abi - fi + for lib in ${{ needs.abi-dump-setup.outputs.ABI_LIBS }}; do + # Remove symbol versioning from dumps, so we can compare across major versions. + sed -i 's/LLVM_${{ matrix.llvm_version_major }}/LLVM_NOVERSION/' $lib-${{ matrix.ref }}.abi + done - name: Upload ABI file uses: actions/upload-artifact@v2 with: |