Skip to content

fix verbose output for 'eb' command, need to use "$@" to ensure arguments are shown correctly#2992

Merged
wpoely86 merged 1 commit intoeasybuilders:developfrom
boegel:fix_eb_debug
Sep 5, 2019
Merged

fix verbose output for 'eb' command, need to use "$@" to ensure arguments are shown correctly#2992
wpoely86 merged 1 commit intoeasybuilders:developfrom
boegel:fix_eb_debug

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Sep 4, 2019

Without this fix, extra arguments are not included in the debug output when $EB_VERBOSE is defined:

$ EB_VERBOSE=1 eb bzip2-1.0.6.eb --force
...
>> Selected Python command: python (/usr/bin/python)
>> python -m easybuild.main bzip2-1.0.6.eb
...

With this change, that's fixed:

$ EB_VERBOSE=1 eb bzip2-1.0.6.eb --force
...
>> Selected Python command: python (/usr/bin/python)
>> python -m easybuild.main bzip2-1.0.6.eb --force
...

@boegel boegel added the bug fix label Sep 4, 2019
@boegel boegel added this to the 4.0.0 milestone Sep 4, 2019
@boegel boegel changed the title fix verbose output for 'eb' command, need to use "" to ensure arguments are shown correctly fix verbose output for 'eb' command, need to use "$@" to ensure arguments are shown correctly Sep 4, 2019
@wpoely86 wpoely86 merged commit 8be7c1b into easybuilders:develop Sep 5, 2019
@boegel boegel deleted the fix_eb_debug branch September 5, 2019 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wpoely86 wpoely86 wpoely86 approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.0.0

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @wpoely86