Skip to content

{bio}[foss/2018b] MaxQuant v1.6.10.43#9281

Merged
smoors merged 2 commits intoeasybuilders:developfrom
SethosII:20191115082303_new_pr_MaxQuant161043
Nov 15, 2019
Merged

{bio}[foss/2018b] MaxQuant v1.6.10.43#9281
smoors merged 2 commits intoeasybuilders:developfrom
SethosII:20191115082303_new_pr_MaxQuant161043

Conversation

@SethosII
Copy link
Copy Markdown
Contributor

@SethosII SethosII commented Nov 15, 2019

This PR uses #9102

I could need some guidance on this one. Is modaliases the way to go here? Is a sanity_check_commands with maxquantcmd --help helpful? (I tried this but the alias doesn't seem to be known during the sanity check) Should source_urls be removed because you need to register and get a link with some hash and an expiration date so a normal download will fail due to error 403?

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Nov 15, 2019

@SethosII modaliases is indeed fine here.
modaliases are only added to the module file, so if you want sanity_check_commands you need to specify the full command (you could define a local_* variable to avoid having to type it twice).
source_urls should indeed be removed.

Copy link
Copy Markdown
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

(see comment above)

@SethosII
Copy link
Copy Markdown
Contributor Author

@smoors I removed source_urls. --help and --version don't return 0 but 1 so there is no easy sanity_check_commands.

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Nov 15, 2019

@SethosII as a workaround, you could grep the output of the --help output, like is done in ACT-18.0.3-Java-11.eb (not saying that you should).

@smoors
Copy link
Copy Markdown
Contributor

smoors commented Nov 15, 2019

Test report by @smoors
SUCCESS
Build succeeded for 2 out of 2 (1 easyconfigs in this PR)
node353.hydra.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/d29da23d61aef2902eee4a71fcc6ff0a for a full test report.

Copy link
Copy Markdown
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

lgtm

@smoors smoors added the new label Nov 15, 2019
@smoors smoors added this to the next release (4.1.0) milestone Nov 15, 2019
@smoors
Copy link
Copy Markdown
Contributor

smoors commented Nov 15, 2019

Going in, thanks @SethosII!

@smoors smoors merged commit f39dcb9 into easybuilders:develop Nov 15, 2019
@SethosII SethosII deleted the 20191115082303_new_pr_MaxQuant161043 branch November 18, 2019 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants