paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-03 21:43:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

push-to-pull-request: Fix regex usage in parsing remote.

Browse Source 
This replaces ad4617c95 with a different fix for the same issue:
instead of stripping the `.git` off separately, we can just correct
the regex, using `+?` to fix our stepping in a classic regex pitfall.
This commit is contained in:
Greg Price
2018-06-01 16:36:56 -07:00
parent e891078d94
commit 8e81ca0fb2
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
tools/push-to-pull-request
View File

@@ -28,9 +28,9 @@ fi
remote_url="$(git config remote."$remote".url)" remote_url="$(git config remote."$remote".url)"
repo_fq="$(echo "$remote_url" | perl -lne 'print $1 if ( repo_fq="$(echo "$remote_url" | perl -lne 'print $1 if (
m, ^ git\@github\.com: m, ^ git\@github\.com:
([^/]+ / [^/]+) ([^/]+ / [^/]+?)
(\.git)? (?:\.git)?
$ ,x )' | sed 's/.git$//')" $ ,x )')"
if [ -z "$repo_fq" ]; then if [ -z "$repo_fq" ]; then
# We're pretty specific about what we expect the URL to look like; # We're pretty specific about what we expect the URL to look like;