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

Replace the FPM-based RPM package building tooling with our own tooling #15143

Open
Tracked by #3708
jkoritzinsky opened this issue Oct 8, 2024 · 2 comments · May be fixed by #15191
Open
Tracked by #3708

Replace the FPM-based RPM package building tooling with our own tooling #15143

jkoritzinsky opened this issue Oct 8, 2024 · 2 comments · May be fixed by #15191
Assignees

Comments

@jkoritzinsky
Copy link
Member

Today, we build RPMs with FPM, a Ruby-based tool that wraps rpmbuild. We have a set of MSBuild targets that produce json that we used to generate the arguments to fpm. This poses a variety of problems:

  • We need to maintain a separate Docker image that has the right version of Ruby installed.
  • Adding a new flag to RPM images requires a significant amount of work.
  • It's possible that we could at some point end up building RPM packages with newer RPM features that don't install on old distros (same problem as the old Deb tooling)

Moving to our own tooling gives us more control over the packages, especially as we're trying to build distro-independent packages (something rpmbuild isn't really designed to do).

This tooling can also be used to enable us to unwrap and rewrap RPM packages, which we need to do for signing.

@ellahathaway
Copy link
Member

@jkoritzinsky - same comment as #14436 (comment). Can you help me give a tshirt size estimate for this work?

@jkoritzinsky
Copy link
Member Author

jkoritzinsky commented Oct 29, 2024

Including the work already done, T-Shirt Size: M or L
Remaining work is T-Shirt Size: XS or S

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

2 participants