Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Feb 14, 2016

@boegel boegel added this to the v2.7.0 milestone Feb 14, 2016
@boegel
Copy link
Member Author

boegel commented Feb 14, 2016

@wpoely86: please review?

optarch (E) = ''
repositorypath (D) = /Users/example/.local/easybuild/ebfiles_repo
robot-paths (D) = /Users/example/easybuild-easyconfigs/easybuild/easyconfigs
sourcepath (D) = /Users/example/.local/easybuild/sources
Copy link
Contributor

Choose a reason for hiding this comment

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

all good, useful feature.

an idea that comes to mind: it would be nice, if the output corresponded directly with shell variables, fi.:
EASYBUILD_MODULES_TOOL=Lmod etc; OTOH, this just one of the many ways to tune things...

Copy link
Member Author

Choose a reason for hiding this comment

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

I think the output should be uniform in any case, and clearly readable.

The way it is printed now is very readable imho, a one-to-one mapping with entries in configuration and very close to the corresponding command line arguments (just prefix with --).

I feel the mapping to the corresponding environment variable is trivial enough (upper case, prefix with EASYBUILD_ and translate - to _) not to clutter the output with EASYBUILD_ everywhere...

@wpoely86
Copy link
Member

lgtm

@boegel
Copy link
Member Author

boegel commented Feb 15, 2016

easybuilders/easybuild-framework#1611 is merged, so good to go, thanks for the feedback @fgeorgatos and @wpoely86!

boegel added a commit that referenced this pull request Feb 15, 2016
add new docs section on --show-config
@boegel boegel merged commit 7acfa2a into easybuilders:develop Feb 15, 2016
@boegel boegel deleted the show_config branch February 15, 2016 16:21
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.

3 participants