Page MenuHomePhabricator

Don't use "--" to separate flags and arguments in "git ls-remote"
ClosedPublic

Authored by epriestley on Mar 19 2017, 12:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 10:42 AM
Unknown Object (File)
Mon, Apr 8, 1:27 AM
Unknown Object (File)
Mon, Apr 1, 3:57 AM
Unknown Object (File)
Sat, Mar 30, 9:43 AM
Unknown Object (File)
Fri, Mar 29, 6:10 AM
Unknown Object (File)
Thu, Mar 28, 11:22 PM
Unknown Object (File)
Mar 7 2024, 1:57 AM
Unknown Object (File)
Feb 3 2024, 3:09 PM
Subscribers
None

Details

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.

Diff Detail

Repository
rP Phabricator
Branch
dash1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16028
Build 21251: Run Core Tests
Build 21250: arc lint + arc unit