Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Deal with spaces in $HOME #86

Merged
merged 1 commit into from

2 participants

@matthiasgoergens

Hello,

Thanks for writing z. I have been using it on Linux for a while, and recently tried it out on Windows / Cygwin. I noticed an oversight that prevented it from working properly when your $HOME path contained spaces. As always with bash, you can fix all problems by more quoting.

Cheers,
Matthias.

@rupa rupa merged commit a41f9c6 into from
@rupa
Owner

Thanks,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 17, 2013
  1. @matthiasgoergens
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  z.sh
View
2  z.sh
@@ -49,7 +49,7 @@ _z() {
# maintain the file
local tempfile
- tempfile="$(mktemp $datafile.XXXXXX)" || return
+ tempfile="$(mktemp "$datafile.XXXXXX")" || return
while read line; do
[ -d "${line%%\|*}" ] && echo $line
done < "$datafile" | awk -v path="$*" -v now="$(date +%s)" -F"|" '
Something went wrong with that request. Please try again.