Skip to content

Commit

Permalink
edksetup.sh: Use bash variable $PWD instead of executing pwd command
Browse files Browse the repository at this point in the history
This is a (very minor) optimization: `pwd` runs the command (even as a
built-in), whereas $PWD simply evaluates the value of the variable.

ALso, modern scripts should generally use $(...) to run commands,
instead of `...`.

Signed-off-by: Rebecca Cran <[email protected]>
Reviewed-by: Laszlo Ersek <[email protected]>
Reviewed-by: Leif Lindholm <[email protected]>
Message-Id: <[email protected]>
  • Loading branch information
bexcran authored and lersek committed Jul 23, 2019
1 parent f6f1e0b commit 80d4f9e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion edksetup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ function SetWorkspace()
#
# Set $WORKSPACE
#
export WORKSPACE=`pwd`
export WORKSPACE=$PWD
return 0
}

Expand Down

0 comments on commit 80d4f9e

Please sign in to comment.