Expand environment variable in file names for C++ apps #600
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
blakewalters
approved these changes
Jun 1, 2020
rtownson
approved these changes
Aug 12, 2020
ftessier
force-pushed
the
feature-expand-envvars
branch
from
45249b8
to
f210bac
Compare
|
Rebased on |
ftessier
approved these changes
Aug 19, 2020
|
Travis seems down, but I have checked locally that compilation is successful and that there is no conflict when rebasing on |
ftessier
force-pushed
the
feature-expand-envvars
branch
from
f210bac
to
f1240f8
Compare
|
Can't merge until Travis completes. Will do then. |
Add an egs++ egsExpandPath global function to expand the FIRST environment variable found anywhere in its string argument: - Expands Linux or Windows style environment variable - Expands the environment variable - Appends folder separator if missing - Replaces backslashes with slashes - Removes duplicate slashes in the expanded path This function assumes that there is only one environment variable in the absolute file path, and that it is a directory.
ftessier
force-pushed
the
feature-expand-envvars
branch
from
f1240f8
to
c108576
Compare
|
Rebased on |
ftessier
force-pushed
the
feature-expand-envvars
branch
from
c108576
to
438d020
Compare
ftessier
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Introduces the C++ EGSnrc global function
egsExpandPathto search for one environment variable anywhere in the string argument and expand it with its value. It understands Unix ($VAR) and Windows (%VAR%) environment variables, as well as their file separators (backslash or slash). However the file name is output using slashes as file separators. Consequently:$EGS_HOMEor$HEN_HOUSEThis feature matches the functionality already existing for Mortran apps although with added flexibility.