Skip to content

Commit

Permalink
Use system QScintilla and QuaZIP for Ubuntu native builds (#1884)
Browse files Browse the repository at this point in the history
* Use system QScintilla

* Use system QuaZIP

* Add a very dirty fix (with unexpected behavior when using CONFIG-=) to eliminate attempts to use non-system QuaZIP or QScinitilla when using the corresponding use_system options._*
---------

Co-authored-by: iakov <[email protected]>
  • Loading branch information
MinyazevR and iakov authored Jan 10, 2025
1 parent b86d55e commit 8c56cf9
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/common_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ jobs:
os: ubuntu-22.04
build: true
build_installer: false
qmake_extra: "CONFIG+=tests CONFIG+=noPch CONFIG+=ccache CONFIG+=no-sanitizers"
qmake_extra: "CONFIG+=tests CONFIG+=noPch CONFIG+=ccache CONFIG+=no-sanitizers CONFIG+=use_system_qscintilla2 CONFIG+=use_system_quazip"
trik_python3_version_minor: 10

build-ubuntu-debug-tests:
Expand All @@ -102,7 +102,7 @@ jobs:
build: true
build_installer: false
config: debug
qmake_extra: "CONFIG+=tests CONFIG+=noPch CONFIG+=ccache CONFIG+=small_debug_info CONFIG+=sanitizer CONFIG+=sanitize_undefined CONFIG+=sanitize_address"
qmake_extra: "CONFIG+=tests CONFIG+=noPch CONFIG+=ccache CONFIG+=small_debug_info CONFIG+=use_system_qscintilla2 CONFIG+=use_system_quazip CONFIG+=sanitizer CONFIG+=sanitize_undefined CONFIG+=sanitize_address"
trik_python3_version_minor: 10

build-altlinux-release-tests:
Expand Down
11 changes: 11 additions & 0 deletions .github/workflows/setup_environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,17 @@ jobs:
submodules: recursive
fetch-depth: 0

- name: Remove thirdparty submodules (use from libs from distro)
run: |
set -xeu
if [[ "${{ inputs.qmake_extra }}" =~ "+=use_system_quazip" ]]; then
rm -rf thirdparty/quazip
fi
if [[ "${{ inputs.qmake_extra }}" =~ "+=use_system_qscintilla2" ]]; then
rm -rf thirdparty/qscintilla
fi
ls thirdparty
- name: Set up environment
run: buildScripts/github/install.sh
env:
Expand Down
2 changes: 1 addition & 1 deletion buildScripts/github/install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ case "$(uname)" in
elif [ "$ID" = "ubuntu" ]; then
sudo apt-get update && sudo apt-get install -y --no-install-recommends ccache curl libusb-1.0-0-dev \
make qtscript5-dev qttools5-dev-tools qtmultimedia5-dev libqt5serialport5-dev libqt5svg5-dev \
libudev-dev "$TRIK_PYTHON"-dev qtbase5-private-dev qtwayland5
libudev-dev "$TRIK_PYTHON"-dev qtbase5-private-dev qtwayland5 libqscintilla2-qt5-dev libquazip5-dev
elif [[ "$ID" = "rocky" || "$ID" = '"rocky"' ]]; then
GCC_VERSION=${GCC_VERSION:-13}
sudo yum update -y && sudo yum install -y --setopt=install_weak_deps=False epel-release
Expand Down

0 comments on commit 8c56cf9

Please sign in to comment.