HomePhabricator

(stable) Don't use "--" to separate flags and arguments in "git ls-remote"

Description

(stable) Don't use "--" to separate flags and arguments in "git ls-remote"

Summary: Fixes T12416. See that task for discussion. Slightly older versions of git do not appear to support use of -- to separate flags and arguments.

Test Plan:

  • Ran bin/repository update PHABX.
  • In T12416, had a user with Git 2.1.4 confirm that git ls-remote X worked while git ls-remote -- X failed.
  • Read git help ls-remote to look for any kind of suspicious --destroy-the-world flags, didn't see any that made me uneasy.

Reviewers: chad, avivey

Reviewed By: avivey

Maniphest Tasks: T12416

Differential Revision: https://secure.phabricator.com/D17508

Details

Provenance
epriestleyAuthored on Mar 19 2017, 12:29 AM
epriestleyPushed on Mar 19 2017, 12:54 AM
Reviewer
avivey
Differential Revision
D17508: Don't use "--" to separate flags and arguments in "git ls-remote"
Parents
rPda80ed415fd3: (stable) Promote 2017 Week 11
Branches
Unknown
Tags
Unknown
Tasks
T12416: Repository mirroring fails due to new `git ls-remote` usage
Build Status
Buildable 16030
Build 21253: Run Core Tests