Skip to content

Conversation

@nat-goodspeed
Copy link
Contributor

No description provided.

bennettgoble and others added 18 commits October 1, 2022 05:39
build-cmd.py used to run an internal 'pip install' to pull in Python package
dependencies. Because of the persistent side effect beyond the current build,
it aborted unless the build was running in a virtualenv, presumed to be
transient for the current build.

Since every GitHub workflow is run in a transient environment, this check is
no longer needed. Moreover, since the build.yaml file can set up the
environment, we simply add a 'pip install' step before the action-autobuild
step. Remove the related code from build-cmd.py.

Don't run on 64-bit Windows: we explicitly want only a 32-bit Windows build.

Defend against eventual removal of the deprecated cgitb package.

Now that there's only one call of the build-cmd.py run() convenience function,
usually unused, the convenience is no longer required.
Specify `python-architecture` as appropriate in build.yaml. This should fix win32 builds, which require a python interpreter of matching architecture.
Specify `python-architecture` as appropriate in build.yaml. This should fix win32 builds, which require a python interpreter of matching architecture.
Workaround for zst compression failing on 32-bit
Specify `python-architecture` as appropriate in build.yaml. This should fix win32 builds, which require a python interpreter of matching architecture.
It seems that when directed to produce a TarFile(mode='w:bz2'), the Python
tarfile module running on GitHub produces a bzip2 compression that's valid
(can be decompressed with bunzip2) but is unknown to Python 3.10's tarfile
module.
@nat-goodspeed nat-goodspeed added the enhancement New feature or request label May 17, 2023
@nat-goodspeed nat-goodspeed merged commit 0eb90e2 into master Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants