Paths

Table of Contentst

Differential D18616

Swap "-R" and "serve" argument order for Mercurial
ClosedPublic
Actions

Authored by epriestley on Sep 15 2017, 7:18 PM.

Tags

None

Referenced Files

	F22748447: D18616.diff
	Sun, Oct 4, 6:53 PM

	Unknown Object (File)
	Jul 5 2026, 9:21 AM

	Unknown Object (File)
	Jul 5 2026, 9:21 AM

	Unknown Object (File)
	Jul 5 2026, 5:27 AM

	Unknown Object (File)
	Jul 4 2026, 6:39 AM

	Unknown Object (File)
	Jul 4 2026, 6:39 AM

	Unknown Object (File)
	Jul 4 2026, 3:11 AM

	Unknown Object (File)
	Jul 1 2026, 10:59 PM

Subscribers

None

Details

Reviewers

Commits

rP76cdaa4f2d10: (stable) Swap "-R" and "serve" argument order for Mercurial
rPb352cacdd917: Swap "-R" and "serve" argument order for Mercurial

Summary

See https://discourse.phabricator-community.org/t/unable-to-use-current-mercurial-on-debian-stretch/391/13. I missed that -R is order-sensitive.

Test Plan

Verified both orders work on 3.5.2.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Sep 15 2017, 7:18 PM

Harbormaster completed remote builds in B18478: Diff 44701.Sep 15 2017, 7:20 PM

amckinley accepted this revision.Sep 15 2017, 7:42 PM

This revision is now accepted and ready to land.Sep 15 2017, 7:42 PM

Closed by commit rPb352cacdd917: Swap "-R" and "serve" argument order for Mercurial (authored by epriestley). · Explain WhySep 15 2017, 8:07 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T13012: Mercurial "--config" and "--debugger" command injection vulnerability.Nov 10 2017, 3:42 PM

epriestley mentioned this in T12521: "(Exception) Expected nonempty 'cmds' specification!" when trying to clone or pull mercurial repository.Jan 4 2018, 7:11 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

controller/

DiffusionServeController.php

2 lines

Diff 44702

src/applications/diffusion/controller/DiffusionServeController.php

Loading...