Skip to content

Convert to hatch build system and enable packit - #35

Open
psss wants to merge 3 commits into
mainfrom
enable-packit
Open

Convert to hatch build system and enable packit#35
psss wants to merge 3 commits into
mainfrom
enable-packit

Conversation

@psss

@psss psss commented Aug 12, 2026

Copy link
Copy Markdown
Member

Replace setup.py with hatchling and hatch-vcs for build and version management. Add the spec file and packit configuration to enable copr builds for pull requests and automate the Fedora release pipeline. Add a release workflow for publishing to pypi.

Fix #31.

Replace `setup.py` with `hatchling` and `hatch-vcs` for build and
version management. Add the spec file and packit configuration to
enable copr builds for pull requests and automate the Fedora
release pipeline. Add a release workflow for publishing to pypi.

Fix #31.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@psss psss added the packaging label Aug 12, 2026
@psss psss added this to planning Aug 12, 2026
@psss psss moved this to implement in planning Aug 12, 2026
@psss psss moved this from implement to review in planning Aug 12, 2026
Comment thread packaging/rpm/testcloud.spec
Comment thread packaging/rpm/testcloud.spec Outdated
Comment thread packaging/rpm/testcloud.spec Outdated

# Remove compiled .py files from /etc after os_install_post
rm -f %{buildroot}%{_sysconfdir}/testcloud/*.py{c,o}
rm -rf %{buildroot}%{_sysconfdir}/testcloud/__pycache__

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird that we need to do this

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No better idea here. Would be probably best to not include .py files under /etc, but using a different name could break things? I would rather keep it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, let's just track it somewhere. It is also relevant to the new release of mock that would lock the files right after %install

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh actually are the files even there? I ran it without those lines and did not see any __pycache__ in there

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, let's try to remove the removal: 4360212

Replace the manual groupadd scriptlet with the proper
%sysusers_create_compat macro for pre-F42 compatibility.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@psss
psss requested a review from LecrisUT August 13, 2026 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: review

Development

Successfully merging this pull request may close these issues.

Enable propose_downstream to automate new releases

3 participants