Skip to content

Commit a385603

Browse files
kevin-219designpestophagous
authored andcommitted
Windows Installation Fixes
- added .user files to gitignore, .user files are used by qt creator and we don't want those in the repo - fixed an error in the readme for windows builds - changed provision_win.sh to use the requirements.txt that linux uses, so they stay matched - changed update_version_string.sh to execute both commands together instead of appending. The problem with -i (appending) is it creates a temp file somewhere and on windows if you call bash update_version_string.sh, the temp file is created in system32 which causes permissions errors. In general -i should be avoided because the file permissions don't always hold
1 parent d8e56f5 commit a385603

File tree

4 files changed

+8
-5
lines changed

4 files changed

+8
-5
lines changed

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,6 @@ aqtinstall.log
99

1010
# mac:
1111
**/.DS_Store
12+
13+
# Qt Creator User File
14+
*.user

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ After that, please proceed to "How to build" (below) before continuing further:
121121
4. In a terminal, run `build_app.sh`
122122

123123
5. Assuming step (4) was successful, you can launch the app at
124-
`./build/src/app/app` (or on Windows: `./build/windeployfolder/app.exe`)
124+
`./build/src/app/app` (or on Windows: `start build/windeployfolder/app.exe`)
125125

126126
For the iOS version of the app, one of the outputs of `build_app.sh` will be
127127
the xcodeproj. Open the xcodeproj in Xcode and from there you can choose a

tools/ci/provision_win.sh

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,16 +5,17 @@ set -Eeuxo pipefail # https://vaneyckt.io/posts/safer_bash_scripts_with_set_euxo
55
CUR_GIT_ROOT=$(git rev-parse --show-toplevel)
66

77
DL_FOLDER=$CUR_GIT_ROOT/dl_third_party
8+
DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )"
89

9-
pip3 install aqtinstall # https://github.com/miurahr/aqtinstall
10+
pip3 install -r ${DIR}/for_pip/requirements.txt # https://github.com/miurahr/aqtinstall
1011
#
1112
# NOTE: as of Nov 23, 2020, it is not clear whether 'win64_msvc2019_64' is the right ARCH
1213
# argument to pass to aqtinstall. We may need to try other options:
1314
# win64_msvc2017_64, win64_msvc2019_winrt_x64 ?
1415
# Refer to: https://github.com/miurahr/aqtinstall#usage
1516

1617
# https://github.com/miurahr/aqtinstall/issues/126 "Installing smaller subset of the libraries"
17-
python -m aqt install-qt windows desktop 5.15.0 win64_msvc2019_64 --outputdir $DL_FOLDER/Qt_desktop --archives \
18+
python -m aqt install --outputdir $DL_FOLDER/Qt_desktop 5.15.0 windows desktop win64_msvc2019_64 --archives \
1819
icu \
1920
qtbase \
2021
qtconnectivity \

tools/update_version_strings.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,5 @@ if [[ "$OSTYPE" == "darwin"* ]]; then
2727
gsed -i "/GIT_HASH_WHEN_BUILT/c ${NEW_HASHLINE}" "$GENFILE" # 2. then operate in-place on genfile
2828
else
2929
# find matching lines and replace WHOLE line with new strings
30-
sed "/BUILD_ON_DATE/c ${NEW_DATELINE}" "$STRINGSFILE" > "$GENFILE" # 1. create genfile
31-
sed -i "/GIT_HASH_WHEN_BUILT/c ${NEW_HASHLINE}" "$GENFILE" # 2. then operate in-place on genfile
30+
sed -e "/BUILD_ON_DATE/c ${NEW_DATELINE}" "$STRINGSFILE" -e "/GIT_HASH_WHEN_BUILT/c ${NEW_HASHLINE}" > "$GENFILE" # 1. create genfile
3231
fi

0 commit comments

Comments
 (0)