Skip to content

Commit 34d6db9

Browse files
Merge pull request #55 from OpenAstroTech/task/js/drop-3.7-support
Drop 3.7, 3.8 support, enable 3.12,3 .13 support for Linux
2 parents df7b947 + 41f4984 commit 34d6db9

File tree

5 files changed

+16
-15
lines changed

5 files changed

+16
-15
lines changed

.github/workflows/build-and-release.yaml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -118,16 +118,16 @@ jobs:
118118
env:
119119
ARTIFACT_ZIP_NAME: ${{ env.ARTIFACT_ZIP_NAME }}
120120
- name: Upload artifact
121-
uses: actions/upload-artifact@v3
121+
uses: actions/upload-artifact@v4
122122
with:
123123
name: ${{ env.ARTIFACT_ZIP_NAME }}
124124
path: ${{ env.ARTIFACT_ZIP_NAME }}.zip
125125

126126
build-virtualenv:
127-
runs-on: 'ubuntu-20.04'
127+
runs-on: 'ubuntu-22.04'
128128
strategy:
129129
matrix:
130-
py_version: [ "3.7", "3.8", "3.9", "3.10", "3.11" ]
130+
py_version: [ "3.9", "3.10", "3.11", "3.12", "3.13" ]
131131

132132
steps:
133133
- uses: actions/checkout@v3
@@ -171,8 +171,8 @@ jobs:
171171
env:
172172
ARTIFACT_ZIP_NAME: ${{ env.ARTIFACT_ZIP_NAME }}
173173
- name: Upload artifact
174-
if: ${{ matrix.py_version }} == "3.10" # Only publish one artifact
175-
uses: actions/upload-artifact@v3
174+
if: ${{ matrix.py_version == 3.10 }} # Only publish one artifact
175+
uses: actions/upload-artifact@v4
176176
with:
177177
name: ${{ env.ARTIFACT_ZIP_NAME }}
178178
path: ${{ env.ARTIFACT_ZIP_NAME }}.zip
@@ -181,14 +181,14 @@ jobs:
181181
# only publish release on tags
182182
if: ${{ startsWith(github.ref, 'refs/tags/') == true }}
183183
needs: [build-windows, build-virtualenv]
184-
runs-on: 'ubuntu-20.04'
184+
runs-on: 'ubuntu-22.04'
185185
permissions:
186186
# See https://github.com/softprops/action-gh-release/issues/236#issuecomment-1150530128
187187
contents: write
188188

189189
steps:
190190
- name: Download all artifacts
191-
uses: actions/download-artifact@v3
191+
uses: actions/download-artifact@v4
192192
with:
193193
path: artifacts
194194
- name: Show artifact contents

OATFWGUI/gui_logic.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,8 @@ def __init__(self, main_app: QWidget):
120120
def spawn_worker_thread(self, fn):
121121
@Slot()
122122
def worker_thread_slot():
123-
all_threads_removed = self.threadpool.waitForDone(msecs=5000)
123+
msecs = 5000
124+
all_threads_removed = self.threadpool.waitForDone(msecs)
124125
if not all_threads_removed:
125126
log.fatal(f'Waited too long for threads to sys.exit! {self.threadpool.activeThreadCount()}')
126127
sys.exit(1)

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ OpenAstroTech FirmWare Graphical User Interface -- A graphical way to build and
66
## Supported platforms
77
- Windows 64 bit
88
- Linux 64 bit
9-
- Requires Python 3.7..3.11, git, libc >= 2.28 (check with `ldd --version`)
9+
- Requires Python 3.9..3.13, git, libc >= 2.28 (check with `ldd --version`)
1010

1111
MacOS support [is in progress](https://github.com/OpenAstroTech/OATFWGUI/commits/feature/js/official-mac-support/), but isn't reliable yet.
1212

requirements.txt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
platformio==6.1.13
2-
PySide6-Essentials==6.5.3 # Last version that supports python 3.7
3-
requests~=2.28.1
1+
platformio==6.1.16 # Hard-pin version so that issues can be reproduced
2+
PySide6-Essentials~=6.8.1
3+
requests~=2.32.3
44
semver~=2.13.0
5-
pygments~=2.13.0
5+
pygments~=2.18.0

scripts/OATFWGUI_Linux.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
set -e
33
# This is the entry point for the "compiled" Linux app
44

5-
# list_include_item "10 11 12" "2"
65
function list_include_item {
76
local list="$1"
87
local item="$2"
@@ -32,6 +31,7 @@ function check_ldd_version {
3231
return 1
3332
fi
3433
# Only support >= 28
34+
# 2.28 from: https://doc.qt.io/qt-6/supported-platforms.html#availability-of-packages
3535
if [ "$LIBC_VER_MIN" -lt 28 ]; then
3636
echo "LIBC minor version $LIBC_VER_MIN ($LIBC_VER_ALL) is not supported"
3737
return 1
@@ -58,7 +58,7 @@ function check_py_version {
5858
return 1
5959
fi
6060
# Only support 3.7+
61-
if ! list_include_item '7 8 9 10 11' "$PY_VER_MIN"; then
61+
if ! list_include_item '9 10 11 12 13' "$PY_VER_MIN"; then
6262
echo "Python minor version $PY_VER_MIN ($PY_VER_ALL) is not supported"
6363
return 1
6464
fi

0 commit comments

Comments
 (0)