Browse Source

refactor(CI): Rename Windows cross compile builds to match cmake types

Debug and Release with capital letters are consistent with all our other CI
script build type names as well as cmake build types.
pull/6610/head
Anthony Bilinski 3 years ago
parent
commit
46ef31293c
No known key found for this signature in database
GPG Key ID: 2AA8E0DA1B31FB3C
  1. 4
      .github/workflows/build-test-deploy.yaml
  2. 2
      windows/cross-compile/README.md
  3. 16
      windows/cross-compile/build.sh

4
.github/workflows/build-test-deploy.yaml

@ -353,7 +353,7 @@ jobs: @@ -353,7 +353,7 @@ jobs:
contents: write
strategy:
matrix:
build_type: [debug, release]
build_type: [Debug, Release]
steps:
- uses: actions/checkout@v3
- uses: ./.github/actions/load-docker-image
@ -433,7 +433,7 @@ jobs: @@ -433,7 +433,7 @@ jobs:
contents: write
strategy:
matrix:
build_type: [debug, release]
build_type: [Debug, Release]
steps:
- uses: actions/checkout@v3
- uses: ./.github/actions/load-docker-image

2
windows/cross-compile/README.md

@ -10,5 +10,5 @@ $ docker compose run windows_builder @@ -10,5 +10,5 @@ $ docker compose run windows_builder
# mkdir build-windows
# cd build-windows
# # Example is using --arch x86_64, --arch i686 should be used if you are in the i686 docker image
# /qtox/windows/cross-compile/build.sh --src-dir /qtox --arch x86_64 --build-type release
# /qtox/windows/cross-compile/build.sh --src-dir /qtox --arch x86_64 --build-type Release
```

16
windows/cross-compile/build.sh

@ -68,18 +68,18 @@ fi @@ -68,18 +68,18 @@ fi
if [ -z "$BUILD_TYPE" ]
then
echo "Error: No build type was specified. Please specify either 'release' or 'debug', case sensitive, as the second argument to the script."
echo "Error: No build type was specified. Please specify either 'Release' or 'Debug', case sensitive, as the second argument to the script."
exit 1
fi
if [[ "$BUILD_TYPE" != "release" ]] && [[ "$BUILD_TYPE" != "debug" ]]
if [[ "$BUILD_TYPE" != "Release" ]] && [[ "$BUILD_TYPE" != "Debug" ]]
then
echo "Error: Incorrect build type was specified. Please specify either 'release' or 'debug', case sensitive, as the second argument to the script."
echo "Error: Incorrect build type was specified. Please specify either 'Release' or 'Debug', case sensitive, as the second argument to the script."
exit 1
fi
# Spell check on windows currently not supported, disable
if [[ "$BUILD_TYPE" == "release" ]]
if [[ "$BUILD_TYPE" == "Release" ]]
then
cmake -DCMAKE_TOOLCHAIN_FILE=/build/windows-toolchain.cmake \
-DCMAKE_PREFIX_PATH=/windows \
@ -90,7 +90,7 @@ then @@ -90,7 +90,7 @@ then
-DSTRICT_OPTIONS=ON \
-DTEST_CROSSCOMPILING_EMULATOR=wine \
"$QTOX_SRC_DIR"
elif [[ "$BUILD_TYPE" == "debug" ]]
elif [[ "$BUILD_TYPE" == "Debug" ]]
then
cmake -DCMAKE_TOOLCHAIN_FILE=/build/windows-toolchain.cmake \
-DCMAKE_PREFIX_PATH=/windows \
@ -122,7 +122,7 @@ set -u @@ -122,7 +122,7 @@ set -u
# Strip
set +e
if [[ "$BUILD_TYPE" == "release" ]]
if [[ "$BUILD_TYPE" == "Release" ]]
then
$ARCH-w64-mingw32-strip -s $QTOX_PREFIX_DIR/*.exe
fi
@ -130,7 +130,7 @@ $ARCH-w64-mingw32-strip -s $QTOX_PREFIX_DIR/*.dll @@ -130,7 +130,7 @@ $ARCH-w64-mingw32-strip -s $QTOX_PREFIX_DIR/*.dll
$ARCH-w64-mingw32-strip -s $QTOX_PREFIX_DIR/*/*.dll
set -e
if [[ "$BUILD_TYPE" == "debug" ]]
if [[ "$BUILD_TYPE" == "Debug" ]]
then
cp -r /debug_export/* ${QTOX_PREFIX_DIR}
fi
@ -141,7 +141,7 @@ zip qtox-"$ARCH"-"$BUILD_TYPE".zip -r * @@ -141,7 +141,7 @@ zip qtox-"$ARCH"-"$BUILD_TYPE".zip -r *
popd
# Create installer
if [[ "$BUILD_TYPE" == "release" ]]
if [[ "$BUILD_TYPE" == "Release" ]]
then
mkdir -p $QTOX_PACKAGE_DIR
pushd $QTOX_PACKAGE_DIR

Loading…
Cancel
Save