Skip to content

fix set-alias statements for MaxQuant v2.0.3.0#15743

Merged
boegel merged 1 commit intoeasybuilders:developfrom
jkwmoore:patch-1
Sep 9, 2022
Merged

fix set-alias statements for MaxQuant v2.0.3.0#15743
boegel merged 1 commit intoeasybuilders:developfrom
jkwmoore:patch-1

Conversation

@jkwmoore
Copy link
Copy Markdown
Contributor

Amends $EBROOTMAXQUANT to %(installdir)s as this variable would be undefined (as a TCL variable).

Amends ``$EBROOTMAXQUANT`` to ``%(installdir)s`` as this variable would be undefined (as a TCL variable).
@verdurin
Copy link
Copy Markdown
Member

Hi @jkwmoore welcome to EasyBuild and thanks for the contribution.

Was looking at testing this, and the MaxQuant download is for a newer version.

Do you have a link for this older version? (apologies for my laziness...)

@jkwmoore
Copy link
Copy Markdown
Contributor Author

Had a look in my email but the link they gave me has expired (after 7 days) I'm afraid.

@verdurin
Copy link
Copy Markdown
Member

Sure, I'm wondering though whether there's a different version-specific link we can use, other than https://www.maxquant.org/download_asset/maxquant/latest

@jkwmoore
Copy link
Copy Markdown
Contributor Author

Suspect the best approach will be to register for the download and get the latest copy then pivot and fling them an email asking for the older copy desired. I can't imagine they would say no.

@boegel boegel changed the title Amend to correct set-alias statements. fix set-alias statements for MaxQuant v2.0.3.0 Aug 3, 2022
@boegel boegel added the bug fix label Aug 3, 2022
@boegel boegel added this to the next release (4.6.1?) milestone Aug 3, 2022
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Sep 9, 2022

Going in, thanks @jkwmoore!

@boegel boegel merged commit c99ebde into easybuilders:develop Sep 9, 2022
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.

3 participants