{tools}[foss/2017b] udocker 1.1.1 w/ Python 2.7.14#5770
{tools}[foss/2017b] udocker 1.1.1 w/ Python 2.7.14#5770boegel merged 3 commits intoeasybuilders:developfrom
Conversation
| dependencies = [ | ||
| ('Python', '2.7.14'), | ||
| ] | ||
|
|
There was a problem hiding this comment.
just to report that this easyconfig is about ready to work, modulo its sanitycheck; use via eb --from-pr 5770 -r;
I only had to resort to this trick to let it pass (I can feel the hot breath of @boegel on my neck over this hack :)
options = {'modulename': 'string'}
then we are able to obtain (@A1ve5 : could you implement that --version? it's a nice to have):
[fgeorgatos@somewhere ~]$ ml !$;!!
ml udocker;which udocker
~/.local/easybuild/software/udocker/1.1.1-foss-2017b-Python-2.7.14/bin/udocker
[fgeorgatos@somewhere ~]$ udocker --version
Error: parsing command line, use: udocker help
seriously, this requires minimal wrap-up effort to get it into next release... thanks @A1ve5
There was a problem hiding this comment.
thanks for this @fgeorgatos
On udocker we try to mimic docker command syntax, so "udocker version" should work.
Nevertheless I'll try to cover the "--version" option as well.
There was a problem hiding this comment.
On devel branch now: indigo-dc/udocker@083b727
There was a problem hiding this comment.
@fgeorgatos We're not savages, you can bypass the import check using this:
options = {'modulename': False};-)
|
@A1ve5 @fgeorgatos Merging A1ve5#1 should make this good to go (installation works using that updated easyconfig). For future PRs: please target the |
(style) fixes for udocker easyconfig file
|
Test report by @boegel |
|
Test report by @boegel |
|
Going in, thanks @A1ve5! |
Still not working but a basis for an udocker easyconfig