Fix escaping of bookmarknames properly for `arc patch`
Fix escaping of bookmarknames properly for `arc patch`

Summary:
Fixes T4596. I misunderstood this issue and D8512 was not correct. Specifically:

  - The `hg log` needs to be escaped, since otherwise "arcpatch-x" is interpreted as a revset.
  - The `hg update` does not need to be escaped, since updating to a revset doesn't make sense and the command never treats its argument as a revset.
  - The `hg bookmark` does not need to be escaped, for similar reasons.

Test Plan:
  - Ran these commands in isolation and got sensible, consistent results.
  - Ran `arc patch` several times in a row and got proper bookmark names.

Reviewers: btrahan, durham, rvanvelzen

Reviewed By: rvanvelzen

Subscribers: epriestley

Maniphest Tasks: T4596

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

zhihu/arcanist,sneakpeeq/arcanist,uber/arcanist,sneakpeeq/arcanist,haskell-infra/arcanist,calblueprint/arcanist,PageUpPeopleOrg/arcanist-patches,wikimedia/phabricator-arcanist,fivestars/arcanist,calblueprint/arcanist,memsql/arcanist,phacility/arcanist,kevinseelbach/arcanist,memsql/arcanist,codelabs-ch/arcanist,r4nt/arcanist,dguo-coursera/arcanist,Khan/arcanist,KieranHayes/arcanist,Symplicity/arcanist,aik099/arcanist,chundongwang/arcanist,jra3/arcanist,zhihu/arcanist,hach-que/arcanist,kevinseelbach/arcanist,Khan/arcanist,calblueprint/arcanist,KieranHayes/arcanist,chundongwang/arcanist,PageUpPeopleOrg/arcanist-patches,phacility/arcanist,codelabs-ch/arcanist,wikimedia/phabricator-arcanist,eSpark/arcanist,wikimedia/phabricator-arcanist,sneakpeeq/arcanist,eSpark/arcanist,alexmv/arcanist,codelabs-ch/arcanist,wikimedia/phabricator-arcanist,KieranHayes/arcanist,uber/arcanist,PageUpPeopleOrg/arcanist-patches,wxstars/arcanist,alexmv/arcanist,phacility/arcanist,dguo-coursera/arcanist,wharris/arcanist,haskell-infra/arcanist,jra3/arcanist,zhihu/arcanist,memsql/arcanist,calblueprint/arcanist,aik099/arcanist,kevinseelbach/arcanist,wharris/arcanist,phacility/arcanist,Symplicity/arcanist,PageUpPeopleOrg/arcanist-patches,kevinseelbach/arcanist,aik099/arcanist,r4nt/arcanist,alexmv/arcanist,uber/arcanist,folsom-labs/arcanist,wharris/arcanist,folsom-labs/arcanist,sneakpeeq/arcanist,haskell-infra/arcanist,hach-que/arcanist,aik099/arcanist,codelabs-ch/arcanist,wharris/arcanist,dguo-coursera/arcanist,jra3/arcanist,memsql/arcanist,fivestars/arcanist,alexmv/arcanist,chundongwang/arcanist,coursera/arcanist,wxstars/arcanist,fivestars/arcanist,coursera/arcanist,dguo-coursera/arcanist,r4nt/arcanist,hach-que/arcanist,coursera/arcanist,uber/arcanist,coursera/arcanist,haskell-infra/arcanist,hach-que/arcanist,folsom-labs/arcanist,zhihu/arcanist,eSpark/arcanist,phacility/arcanist,wxstars/arcanist,folsom-labs/arcanist,chundongwang/arcanist,codelabs-ch/arcanist,Khan/arcanist,r4nt/arcanist,Symplicity/arcanist,KieranHayes/arcanist,fivestars/arcanist,Khan/arcanist,wxstars/arcanist,eSpark/arcanist,Symplicity/arcanist,jra3/arcanist