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

Patch FPM Build Script #302

Merged
merged 4 commits into from
Feb 21, 2024
Merged

Patch FPM Build Script #302

merged 4 commits into from
Feb 21, 2024

Conversation

RedProkofiev
Copy link
Contributor

Resolves #301

Renames the buildscript, updates dependencies, dependency orders and also adds a 2x2 matrix for el7/el8 vs debian builds for python 2 and 3.

…tags

A mixture of inconsistent testing and dependencies led to some weird errors that were hard to reproduce.
I cordoned it down to:
- Ordering of --depends calls, where calling python3-pip before python3-stomppy led to an almost instant
fail
- Bad module names
It was not in fact the ifs that were at fault.  This script has now been tested with Rocky 8 and CentOS 8
from start to finish, and Debian previously.  It will now be tested with EL7.
Fixes incompatibility with EL7 by changing some module names.
@RedProkofiev RedProkofiev requested a review from a team as a code owner February 19, 2024 17:18
scripts/ssm-build.sh Outdated Show resolved Hide resolved
scripts/ssm-build.sh Outdated Show resolved Hide resolved
@tofu-rocketry tofu-rocketry self-assigned this Feb 19, 2024
@tofu-rocketry tofu-rocketry added this to the 3.4.0 milestone Feb 19, 2024
RedProkofiev and others added 2 commits February 19, 2024 18:21
Co-authored-by: Adrian Coveney <[email protected]>
Co-authored-by: Adrian Coveney <[email protected]>
@tofu-rocketry
Copy link
Member

Packages building and installing on EL 7 and 8

@tofu-rocketry tofu-rocketry merged commit 62a37c0 into apel:dev Feb 21, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

New build script deps
2 participants