Skip to content

monkey patch shell_quote in vsc.utils.generaloption to fix #1438#1915

Merged
boegel merged 4 commits intoeasybuilders:developfrom
boegel:monkey_patch_shell_quote
Sep 19, 2016
Merged

monkey patch shell_quote in vsc.utils.generaloption to fix #1438#1915
boegel merged 4 commits intoeasybuilders:developfrom
boegel:monkey_patch_shell_quote

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Sep 16, 2016

(temporary) workaround for #1438, until hpcugent/vsc-base#151 can be fixed properly

cc @geimer

@boegel boegel added this to the v2.9.0 milestone Sep 16, 2016
Copy link
Copy Markdown
Member

@wpoely86 wpoely86 left a comment

Choose a reason for hiding this comment

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

ugly, but I haven't got time to do better

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 16, 2016

@geimer can you test whether this fixes your problem? It should...

@geimer
Copy link
Copy Markdown
Contributor

geimer commented Sep 19, 2016

@boegel This PR solves my issues reported in #1438. I just robot-built foss-2016b.eb from scratch with EASYBUILD_SUFFIX_MODULES_PATH= and EASYBUILD_TEST_REPORT_ENV_FILTER="^SSH|GPG|USER|HOSTNAME|UID|KDE|XDG|.*(COOKIE|SESSION).*". Worked like a charm.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Sep 19, 2016

@geimer great, thanks for confirming the workaround! Going in...

@boegel boegel merged commit ea4f7b2 into easybuilders:develop Sep 19, 2016
@boegel boegel deleted the monkey_patch_shell_quote branch September 19, 2016 12:10
@JensTimmerman
Copy link
Copy Markdown

should be fixed by hpcugent/vsc-base#257

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.

4 participants