Skip to content

packaging: Fix %files in spec #240

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

packaging: Fix %files in spec #240

wants to merge 3 commits into from

Conversation

martinpitt
Copy link
Owner

@martinpitt
Copy link
Owner Author

/packit build

@martinpitt
Copy link
Owner Author

@lachmanfrantisek @lbarcziova Can you please help me with packit being AWOL here? https://dashboard.packit.dev/jobs/srpm doesn't have any trace of it. I just rebased and landed #153 and that didn't have it either. Thanks!

@martinpitt
Copy link
Owner Author

@lachmanfrantisek @lbarcziova whatever you did, thank you! 🤗

@hroncok
Copy link

hroncok commented Jul 14, 2025

The module name seems to be dbusmock, not python_dbusmock. You also need to package the dist-info folder as well.

The easiest way to do this is to use %pyproject_save_files:

%install
%pyproject_install
%pyproject_save_files -l dbusmock

...

%files -n python3-dbusmock -f %{pyproject_files}

@martinpitt
Copy link
Owner Author

@hroncok Right, thanks -- I sent this in a hurry yesterday. Now I devoted some actual time to it. Fixed, thanks for the save_files tip!

That was missing from commit c98793c.
There is a Python `packaging` module as well, and ruff's "I001 Import
block is un-sorted or un-formatted" rule mistook the packaging/
directory as local module.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants