Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

sed fix #23

Merged
merged 3 commits into from

2 participants

@sickill
Owner

I've changed sed regexp to be (hopefully) more cross-platform compatible.

@digitaljhelms

Same error...

@sickill
Owner

@digitaljhelms Should work now, I tested on FreeBSD and it worked. As far as I know OSX has BSD-like tools.

@digitaljhelms

The force is strong with this one.

@sickill
Owner

I have just tested parsing-fix branch on OSX and sed regexp works properly. I think I'll merge it into master.

@digitaljhelms

Yes, it works, guess I could have been clearer in my previous comment -- I've been using the parsing-fix branch locally since you pushed it.

@sickill
Owner

Ok. I didn't get your last comment indeed. Merging.

@sickill sickill merged commit 5dab62c into master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 3 deletions.
  1. +3 −3 git-dude
View
6 git-dude
@@ -68,7 +68,7 @@ while true; do
repo_name=$(basename "$dir_name" .git)
cd "$dir_name"
- changes=$(git fetch 2>&1 | grep -F -- '->' | sed -E 's/^\s+\+\s+//')
+ changes=$(git fetch 2>&1 | grep -F -- '->' | sed 's/^ [+*=!-] //')
icon_path=$(git config dude.icon || true)
icon_path=${icon_path:-`pwd`/icon.png}
@@ -83,11 +83,11 @@ while true; do
notify $icon_path "New commits in $repo_name/$branch_name" "$commit_messages"
;;
*new\ branch*)
- branch_name=$(echo "$line" | awk '{ print $4 }')
+ branch_name=$(echo "$line" | awk '{ print $3 }')
notify $icon_path "New branch $repo_name/$branch_name" ""
;;
*new\ tag*)
- tag_name=$(echo "$line" | awk '{ print $4 }')
+ tag_name=$(echo "$line" | awk '{ print $3 }')
notify $icon_path "New tag $repo_name/$tag_name" ""
;;
esac
Something went wrong with that request. Please try again.