Skip to content
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

Repackage using newer tools #9

Merged
merged 13 commits into from
Nov 29, 2023
Merged

Repackage using newer tools #9

merged 13 commits into from
Nov 29, 2023

Conversation

peterfpeterson
Copy link
Member

@peterfpeterson peterfpeterson commented Nov 22, 2023

This is a major rework of how the packaging metadata is described and the rpm spec file. The setup.py has been removed and all information is specified in the pyproject.toml.

The rpmbuild.sh script has been changed to build a sdist first rather than have git create the tarball. Creating the sdist is the only thing CI attempts.

The specfile has been, essentially, rewritten. It no longer depends on python to install the package and uses the rpm macros for various commands (e.g. %{__install} rather than install).

A thing to consider doing later is using the systemd scriptlets.

tip: to see what the rpm macros actually do, run rpm --showrc and grep the results for things

@peterfpeterson peterfpeterson marked this pull request as ready for review November 22, 2023 22:04
@idigs
Copy link

idigs commented Nov 27, 2023

It would be nice to reproduce this build https://github.com/mantidproject/livereduce/actions/runs/6963139456/workflow on a separate machine

@idigs
Copy link

idigs commented Nov 27, 2023

In an effort to eliminate ambiguity with the /usr/bin/rpmbuild command, rename the 'rpmbuild' shell script as 'rpmbuild.sh'

Copy link

@idigs idigs left a comment

Choose a reason for hiding this comment

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

Approved

@peterfpeterson peterfpeterson merged commit 97618c0 into main Nov 29, 2023
2 checks passed
@peterfpeterson peterfpeterson deleted the repackage branch November 29, 2023 21:50
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