aboutsummaryrefslogtreecommitdiff
path: root/sh/shrc.d
diff options
context:
space:
mode:
authorTom Ryder <tom@sanctum.geek.nz>2017-05-26 20:23:54 +1200
committerTom Ryder <tom@sanctum.geek.nz>2017-05-26 20:23:54 +1200
commit520f9e174aa0569a95469a7d2b746582bdf0c18c (patch)
treeae62b1008b1bfa6c6369d17bcd48baae6be829d2 /sh/shrc.d
parentCorrect default behaviour for bd() with no args (diff)
downloaddotfiles-520f9e174aa0569a95469a7d2b746582bdf0c18c.tar.gz
dotfiles-520f9e174aa0569a95469a7d2b746582bdf0c18c.zip
More bd() improvements
Including rigorous trailing-slash handling
Diffstat (limited to 'sh/shrc.d')
-rw-r--r--sh/shrc.d/bd.sh41
1 files changed, 29 insertions, 12 deletions
diff --git a/sh/shrc.d/bd.sh b/sh/shrc.d/bd.sh
index 7901e115..9b877faf 100644
--- a/sh/shrc.d/bd.sh
+++ b/sh/shrc.d/bd.sh
@@ -1,31 +1,48 @@
# Move back up the directory tree to the first directory matching the name
bd() {
- # Check argument count; default to ".."
- case $# in
- 0) set -- .. ;;
- 1) ;;
- *)
- printf >&2 'bd(): Too many arguments\n'
- return 2
- esac
+ # Check arguments; default to ".."
+ if [ "$#" -gt 1 ] ; then
+ printf >&2 'bd(): Too many arguments\n'
+ return 2
+ fi
+ set -- "${1:-..}"
# Look at argument given; default to going up one level
case $1 in
- # If it has a leading slash or is . or .., don't touch the arguments
- /*|.|..) ;;
+ # If it's slash, dot, or dot-dot, we'll just go there, like `cd` would
+ /|.|..) ;;
+
+ # Anything else with a slash anywhere is an error
+ */*)
+ printf >&2 'bd(): Illegal slash\n'
+ return 2
+ ;;
# Otherwise, we'll try to find a matching ancestor and then shift the
# initial request off the argument list
*)
# Push the current directory onto the stack
- set -- "${1%/}" "$PWD"
+ set -- "$1" "$PWD"
# Keep chopping at the current directory until it's empty or it
# matches the request
- while set -- "$1" "${2%/*}" ; do
+ while : ; do
+
+ # Make certain there are no trailing slashes to foul us up
+ while : ; do
+ case $2 in
+ */) set -- "$1" "${2%/}" ;;
+ *) break ;;
+ esac
+ done
+
+ # Strip a path element
+ set -- "$1" "${2%/*}"
+
+ # Check whether we're arrived
case $2 in
*/"$1") break ;;
*/*) ;;