Skip to content

define alias for the modulefile in your easyconfigs#952

Merged
boegel merged 8 commits intoeasybuilders:developfrom
pescobar:add-alias
Jul 8, 2014
Merged

define alias for the modulefile in your easyconfigs#952
boegel merged 8 commits intoeasybuilders:developfrom
pescobar:add-alias

Conversation

@pescobar
Copy link
Copy Markdown
Member

first working version. It needs more testing

with this you can add this to your easyconfigs

modextraalias = {
    'gatk':               'java -jar $root/gatk.jar',
}

and the alias will be added in the generated modulefile

set-alias gatk "java -jar $root/gatk.jar"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

hmm, maybe modextra_aliases is better? plural, but also avoid the double a to make it more readable (although that doesn't match the modextrafoo pattern that was being used for the others

@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 21, 2014

@pescobar: you'll need to update the SHA1's in the easyconfig.py test module, see https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/643/console

The best way to do that is to run it, see what the correct sha1 is, replace the old one, run again. Until you're replaced all of them.

This happens every time a new easyconfig parameter is added, because of the rather weird unit test for custom module naming schemes.

@pescobar
Copy link
Copy Markdown
Member Author

pescobar commented Jul 3, 2014

test this please

rename modextraalias to modaliases
@boegel boegel added this to the v1.14 milestone Jul 5, 2014
@boegel
Copy link
Copy Markdown
Member

boegel commented Jul 8, 2014

@pescobar: looks great, merging in, thanks!

boegel added a commit that referenced this pull request Jul 8, 2014
define alias for the modulefile in your easyconfigs
@boegel boegel merged commit 4dd4fe7 into easybuilders:develop Jul 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants