Skip to content

enhance Rpm generic easyblock to support installing tarball of RPMs (by unpacking it first) + take into account (pre)installopts#2580

Merged
akesandgren merged 2 commits intoeasybuilders:developfrom
boegel:rpm_extract
Nov 24, 2021
Merged

enhance Rpm generic easyblock to support installing tarball of RPMs (by unpacking it first) + take into account (pre)installopts#2580
akesandgren merged 2 commits intoeasybuilders:developfrom
boegel:rpm_extract

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Sep 21, 2021

I have a use case for this (commercial software that comes as a tarball of RPMs...), changes the behavior of using Rpm with extract_sources = True.

@boegel boegel added this to the next release (4.5.0?) milestone Sep 21, 2021
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 21, 2021

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS Mono-2.10.6-intel-2016b.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node2691.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/404015475160f962677271da8715bc2e for a full test report.

@boegel boegel requested review from Micket and ocaisa and removed request for Micket September 22, 2021 13:22
@boegel boegel removed the request for review from ocaisa October 2, 2021 10:05
@boegel boegel requested a review from bartoldeman October 13, 2021 14:24
Copy link
Copy Markdown
Contributor

@akesandgren akesandgren left a comment

Choose a reason for hiding this comment

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

LGTM

@akesandgren
Copy link
Copy Markdown
Contributor

Going in, thanks @boegel!

@akesandgren akesandgren merged commit 582f0fe into easybuilders:develop Nov 24, 2021
@boegel boegel deleted the rpm_extract branch November 24, 2021 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants