FPM build script for RPM and DEB #279
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Author
|
Build script is here! Finally working on EL8 and Debian systems, with service setup. |
Contributor
Author
|
For some reason there's still a merge conflict - was the patch reverted? or is this due to my dev branch not being fast forwarded? |
Member
That'll be because we merged the change to the CHANGELOG into the Could you have a look at the code quality checks and see if you can improve that? |
nick-whyatt
changed the title
nick-whyatt
changed the title
nick-whyatt
force-pushed
the
nick-123
branch
3 times, most recently
from
4b5260b to
fb8fd84
Compare
tofu-rocketry
requested changes
Jan 18, 2024
tofu-rocketry
approved these changes
Jan 26, 2024
tofu-rocketry
force-pushed
the
nick-123
branch
2 times, most recently
from
0d2ef47 to
0da5063
Compare
tofu-rocketry
approved these changes
Feb 6, 2024
- Adds cryptography as a dependency before pyopenssl - Modified pyopenssl to lower case to satisfy pip call
tofu-rocketry
force-pushed
the
nick-123
branch
from
0da5063 to
ee07f16
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #123
Resolves GT-253
This PR adds a build script that supersedes
ssm-build-rpm.shandssm-build-deb.shby adapting the FPM calls made in the original Debian script and adding configurable parameters. It works with Python 2.7 and >= 3.6, and requires FPM 1.14.2 and a version of Ruby at or above 2.7, as well as specifically Setuptools==51.3.3. It allows a version to be specified to pull from the repository and be setup.Usage:
./ssm-build-dual.sh [options] (deb | rpm) <version> <iteration> <python_root_dir>Currently, this should work for all future versions and all the past RPM versions: Debian, it doesn't work for before the coming 3.4.0 update due to CHANGELOG having a title.