Skip to content

Commit 22cd58b

Browse files
Merge pull request #6163 from MetRonnie/argv
`cylc version --long`: ensure correct path is printed for `cylc` executable
2 parents d715aff + 5b21590 commit 22cd58b

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

cylc/flow/scripts/cylc.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ def get_version(long=False):
8888
from pathlib import Path
8989
version = f"{__version__}"
9090
if long:
91-
version += f" ({Path(sys.executable).parent.parent})"
91+
version += f" ({Path(sys.argv[0])})"
9292
return version
9393

9494

tests/functional/cli/01-help.t

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -79,12 +79,7 @@ run_ok "${TEST_NAME_BASE}-id" cylc help id
7979

8080
# Check "cylc version --long" output is correct.
8181
cylc version --long | head -n 1 > long1
82-
WHICH="$(command -v cylc)"
83-
PARENT1="$(dirname "${WHICH}")"
84-
PARENT2="$(dirname "${PARENT1}")"
85-
echo "$(cylc version) (${PARENT2})" > long2
86-
# the concise version of the above is a bash quoting nightmare:
87-
# echo "$(cylc version) ($(dirname $(dirname $(which cylc))))" > long2
82+
echo "$(cylc version) ($(command -v cylc))" > long2
8883
cmp_ok long1 long2
8984

9085
# --help with no DISPLAY

0 commit comments

Comments
 (0)