Merge pull request #528 from libexpat/actions-upgrade-clang

Upgrade Clang from 11 to 13 (and fix a related warning)
This commit is contained in:
Sebastian Pipping 2021-12-26 23:52:48 +01:00 committed by GitHub
commit 58d1f13d31
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 12 additions and 10 deletions

View File

@ -46,16 +46,16 @@ jobs:
- name: Add Clang/LLVM repositories - name: Add Clang/LLVM repositories
run: |- run: |-
wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add - wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add -
sudo add-apt-repository 'deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-11 main' sudo add-apt-repository 'deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-13 main'
- name: Install build dependencies - name: Install build dependencies
# NOTE: Please note the version-specific ${PATH} extension for Clang in .travis.sh # NOTE: Please note the version-specific ${PATH} extension for Clang in .travis.sh
run: |- run: |-
sudo dpkg --add-architecture i386 # for wine32 sudo dpkg --add-architecture i386 # for wine32
sudo apt-get update sudo apt-get update
sudo apt-get install --yes --no-install-recommends -V \ sudo apt-get install --yes --no-install-recommends -V \
clang-11 \ clang-13 \
clang-format-11 \ clang-format-13 \
llvm-11 \ llvm-13 \
cmake \ cmake \
docbook2x \ docbook2x \
dos2unix \ dos2unix \

View File

@ -80,16 +80,16 @@ jobs:
- name: Add Clang/LLVM repositories - name: Add Clang/LLVM repositories
run: |- run: |-
wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add - wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add -
sudo add-apt-repository 'deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-11 main' sudo add-apt-repository 'deb http://apt.llvm.org/bionic/ llvm-toolchain-bionic-13 main'
- name: Install build dependencies - name: Install build dependencies
# NOTE: Please note the version-specific ${PATH} extension for Clang in .travis.sh # NOTE: Please note the version-specific ${PATH} extension for Clang in .travis.sh
run: |- run: |-
sudo dpkg --add-architecture i386 # for wine32 sudo dpkg --add-architecture i386 # for wine32
sudo apt-get update sudo apt-get update
sudo apt-get install --yes --no-install-recommends -V \ sudo apt-get install --yes --no-install-recommends -V \
clang-11 \ clang-13 \
clang-format-11 \ clang-format-13 \
llvm-11 \ llvm-13 \
cmake \ cmake \
docbook2x \ docbook2x \
dos2unix \ dos2unix \

View File

@ -39,7 +39,7 @@ if [[ ${TRAVIS_OS_NAME} = osx ]]; then
export PATH="/usr/local/opt/coreutils/libexec/gnubin${PATH:+:}${PATH}" export PATH="/usr/local/opt/coreutils/libexec/gnubin${PATH:+:}${PATH}"
export PATH="/usr/local/opt/findutils/libexec/gnubin${PATH:+:}${PATH}" export PATH="/usr/local/opt/findutils/libexec/gnubin${PATH:+:}${PATH}"
elif [[ ${TRAVIS_OS_NAME} = linux ]]; then elif [[ ${TRAVIS_OS_NAME} = linux ]]; then
export PATH="/usr/lib/llvm-11/bin:${PATH}" export PATH="/usr/lib/llvm-13/bin:${PATH}"
fi fi
echo "New \${PATH}:" echo "New \${PATH}:"

View File

@ -4,7 +4,7 @@ NOTE: We are looking for help with a few things:
Release x.x.x xxx xxxxxxxx xx xxxx Release x.x.x xxx xxxxxxxx xx xxxx
Other changes: Other changes:
#527 Address compiler warnings #527 #528 Address compiler warnings
Release 2.4.2 Sun December 19 2021 Release 2.4.2 Sun December 19 2021
Other changes: Other changes:

View File

@ -1128,6 +1128,8 @@ tmain(int argc, XML_Char **argv) {
#ifdef XML_DTD #ifdef XML_DTD
XML_SetBillionLaughsAttackProtectionActivationThreshold( XML_SetBillionLaughsAttackProtectionActivationThreshold(
parser, attackThresholdBytes); parser, attackThresholdBytes);
#else
(void)attackThresholdBytes; // silence -Wunused-but-set-variable
#endif #endif
} }