-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Python 3.12 update. #58
Changes from 84 commits
111024a
ff41dd8
a102aee
d5240fa
3a2291c
4a5cb66
12b79af
543d85f
f6e873c
40291f7
87713dd
e7e0fb6
8aaeaa3
e515cfc
c562a70
bf771c3
1ee3fd8
f5e176a
e0675ce
e61df38
041ee81
2724f91
238dda3
abdeddf
0feae9d
1018d97
04a4a72
9844b39
c164875
676caf9
dcabbcb
eb0ab77
8ce0751
42d6e8f
176a8ad
c936949
9d08ba7
f1f1615
ec8a45e
3a41156
f7a73bf
1f4ce2a
36616a8
8e372ef
4f22e7e
35f4b1c
7fe9200
0f0c21e
5bb35ca
e7cc6e0
5766411
797af65
f3a26fe
6246aae
1648d3b
7921672
e43aaa7
253a5d7
21a69bb
3cb102c
9545364
879667f
e4c1428
b46e772
bb559dd
d65c50f
3ec71bf
253e629
e783ef9
ddd58ee
955703b
1535503
2ae496a
6187a23
e4868b4
c1d74ac
c87952a
bf5f05b
ffac61a
7548eaa
ced2651
8a2f99d
a5de779
2303a25
aada621
ac6595f
e45dcd7
77ba463
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,5 +4,5 @@ | |
/src/*/*.tar.gz | ||
/src/*/*.tgz | ||
/src/*/*.zip | ||
/build-pythia/ | ||
/build-py3/ | ||
/cache/ |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,22 +9,24 @@ DIST_DIR="dist" | |
# Setting this as 0 disables tests during building (skips chevahbs_test phase). | ||
PYTHIA_BUILD_TESTS="${PYTHIA_BUILD_TESTS:-1}" | ||
|
||
# Python and lib versions. | ||
PYTHON_BUILD_VERSION="3.11.9" | ||
# Set versions for Python and its libraries. | ||
# For major Python updates, e.g. 3.11->3.12, also update pythia.conf and | ||
# src/Python-Windows/chevahbs (e.g. "python311._pth"->"python312._pth"). | ||
PYTHON_BUILD_VERSION="3.12.7" | ||
LIBFFI_VERSION="3.4.6" | ||
ZLIB_VERSION="1.3.1" | ||
BZIP2_VERSION="1.0.8" | ||
# To check the signature of the XZ download: | ||
# "gpg --keyserver-options auto-key-retrieve --verify xz-*.sig xz-*.gz". | ||
XZ_VERSION="5.6.2" | ||
# Statically build the BSD libedit on selected platforms to get the | ||
# readline module available without linking to the GPL-only readline libs. | ||
# readline module available without linking to the GPL-only readline libraries. | ||
# If there's a need to reenable this, our libedit patch for Python 3.9 was | ||
# https://github.com/chevah/pythia/pull/5/commits/09c128154d23feb6b1a7cb5a8d79. | ||
# A newer patch is available at https://github.com/python/cpython/issues/57710. | ||
LIBEDIT_VERSION="20170329-3.1" | ||
# Our OpenSSL libs are only used for Python's "ssl" module lately. | ||
OPENSSL_VERSION="3.0.14" | ||
# Our OpenSSL libraries are only used for Python's "ssl" module lately. | ||
OPENSSL_VERSION="3.0.15" | ||
# Use the version of the "sqlite-autoconf-VERSION.tar.gz" upstream download. | ||
# To get its SHA3-256 signature: "openssl dgst -sha3-256 sqlite-autoconf-*". | ||
# When updating this, also update the year in src/sqlite/chevahbs, if needed. | ||
|
@@ -36,19 +38,16 @@ SQLITE_VERSION="3460000" | |
BOOTSTRAP_GET_PIP="https://bootstrap.pypa.io/get-pip.py" | ||
|
||
# Python modules installed after bootstraping pip. | ||
PIP_VERSION="24.0" | ||
SETUPTOOLS_VERSION="70.0.0" | ||
PIP_VERSION="24.2" | ||
SETUPTOOLS_VERSION="70.3.0" | ||
# pycparser is explicitly installed to work around setuptools auto dependencies. | ||
PYCPARSER_VERSION="2.22" | ||
|
||
# Python modules that have to be built and/or installed in Pythia. | ||
PSUTIL_VERSION="5.9.8" | ||
PSUTIL_VERSION="6.0.0" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we still need this variabl here. I think that we can install all the packageis via then the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hhmmm… Now that you put it this way, it's obvious There's also Actually, lately only So, I guess no? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Maintaining There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Left things as is for now, as To make sure it's not left behind, I've used the above GitHub action on Windows, where What do you think? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also filled #62 for improving current situation. |
||
PYWIN32_VERSION="306" | ||
|
||
# Pin safety to keep the deps of the test environment under control. | ||
SAFETY_VERSION="3.2.0" | ||
|
||
# Global flags for building required libs. | ||
# Global flags for building required libraries. | ||
BUILD_LIBFFI="no" | ||
BUILD_ZLIB="no" | ||
BUILD_BZIP2="yes" | ||
|
@@ -72,7 +71,3 @@ PIP_ARGS=(\ | |
--index-url="$PIP_INDEX_URL" \ | ||
--no-warn-script-location \ | ||
) | ||
# Array of safety IDs to ignore. | ||
# 67599: pip 24.0, https://data.safetycli.com/v/67599/f17/ (disputed). | ||
# 70612: jinja2 3.1.4, https://data.safetycli.com/v/70612/97c (disputed). | ||
SAFETY_IGNORED_IDS=(67599 70612) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,11 +1,14 @@ | ||
PYTHON_CONFIGURATION="default@3.11.3.f9d9434" | ||
# When building a new major Python version, e.g. 3.11->3.12, | ||
# update this in advance (e.g. use "default@3.12.0.deadbeef"), | ||
# and remove BUILD_ENV_* files (e.g. with `./build.sh clean -a`). | ||
PYTHON_CONFIGURATION="default@3.12.6.b46e772" | ||
# This is defined as a Bash array of options to be passed to commands. | ||
BASE_REQUIREMENTS=("chevah-brink==1.0.13" "paver==1.3.4") | ||
BASE_REQUIREMENTS=("chevah-brink==1.0.15" "paver==1.3.4") | ||
# Use our private PyPi server instead of the default one set in pythia.sh. | ||
PIP_INDEX_URL="https://bin.chevah.com:20443/pypi/simple" | ||
# Use our production server instead of the GitHub releases set by default. | ||
BINARY_DIST_URI="https://bin.chevah.com:20443/production" | ||
# For testing packages, make sure this one is the last uncommented instance: | ||
#BINARY_DIST_URI="https://bin.chevah.com:20443/testing" | ||
# Also overwrite the default pypi.org site set by default in pythia.sh. | ||
PIP_INDEX_URL="https://bin.chevah.com:20443/pypi/simple" | ||
# This is used by the Python runtime. | ||
BINARY_DIST_URI="https://bin.chevah.com:20443/testing" | ||
# This directory is used by the Python runtime. | ||
CHEVAH_BUILD_DIR="build-py3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not happy with this ... but we can look to fix this in a separate PR
The CI run should not modify the source code
Can we have the "requirements.txt" manually updated before a push ?
I guess that this is a question for #62
What we can do as part of the CI, is make sure that "requirements.txt" doesn't need an update during a CI run
We have something similar for chevah/server
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure what that would be. I know it's used for cache updates, but what do you mean by "making sure it doesn't need an update"?