Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

WIP: Move cx_Freeze build to GitHub Actions #101

Closed
wants to merge 1 commit into from

Conversation

qstokkink
Copy link
Owner

@qstokkink qstokkink commented Jul 31, 2024

Fixes #97

This PR is a WIP and the second attempt at switching (most of) the build process to GitHub Actions, now using cx_Freeze.

TODO (current focus):

  • Figure out why cx_Freeze insists on adding garbage to the tar.gz.

@qstokkink qstokkink force-pushed the add_ga_builders_v2 branch 8 times, most recently from b82508b to 33dcf6f Compare July 31, 2024 14:18
@qstokkink qstokkink force-pushed the add_ga_builders_v2 branch from 33dcf6f to fdaa5a6 Compare July 31, 2024 14:37
@qstokkink
Copy link
Owner Author

I'll retire this attempt. TIL:

  • Using the declarative format of cx_Freeze to exclude junk packages, like tests, is not easy. Using our current approach, of a dedicated makedist_debian.sh, seems like a fine way to go about this.
  • Using the declarative format of cx_Freeze for cross-platform setup.py, will need a lot of platform-specific workarounds anyway.
  • We can downsize our own code base using the autogeneration of cx_Freeze's bdist_deb, etc.
  • Automatically retrieving the version from git using setuptools_scm is possible but probably more trouble (i.e., hidden caveats) than it's worth.

Most of the issues above can be fixed. However, the result would not be the elegant solution I'm looking for.

@qstokkink qstokkink closed this Aug 1, 2024
@qstokkink qstokkink deleted the add_ga_builders_v2 branch August 8, 2024 13:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Attempt to move building to GitHub Actions
1 participant