-
Notifications
You must be signed in to change notification settings - Fork 308
stop using deprecated verbose option in build step of BLAT easyblock
#3858
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
stop using deprecated verbose option in build step of BLAT easyblock
#3858
Conversation
verbose option in build step of BLAT easyblock
|
@boegelbot please test @ jsc-zen3 |
|
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3135106285 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
|
Test report by @boegel Overview of tested easyconfigs (in order)
Build succeeded for 1 out of 1 (1 easyconfigs in total) |
|
Duplicate of #3856? |
Yes, but I think the approach I'm using here with So I would like to see this merged, and then we can update #3856 accordingly for the other easyblocks being touched there (GHC, SCons), and perhaps also tackle the other relevant easyblocks that use the |
laraPPr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
|
Going in, thanks @boegel! |
(created using
eb --new-pr)Fixes installation of BLAT easyconfigs when use of deprecated functionality is not allowed: