D7188: git: force using bytes in command name

Submitter	phabricator
Date	Nov. 1, 2019, 4:39 p.m.
Message ID	<differential-rev-PHID-DREV-fmi4al2ftlzzguhmwaef-req@mercurial-scm.org>
Download	mbox \| patch
Permalink	/patch/42653/
State	New
Headers	show Return-Path: <mercurial-devel-bounces@mercurial-scm.org> Date: Fri, 1 Nov 2019 16:39:21 +0000 To: Phabricator <phabricator@mercurial-scm.org> From: "maho (Maho)" <phabricator@mercurial-scm.org> Subject: D7188: git: force using bytes in command name Message-ID: <differential-rev-PHID-DREV-fmi4al2ftlzzguhmwaef-req@mercurial-scm.org> Precedence: bulk Thread-Topic: PHID-DREV-fmi4al2ftlzzguhmwaef Thread-Index: NTA0YjBhMjk4ZTdlODRjYTU4MDRhODZlN2Uw MIME-Version: 1.0 Cc: mercurial-devel@mercurial-scm.org Reply-To: "maho \(Maho\)" <phabricator+D7188+public+f20c91a02d8b79d7@mercurial-scm.org> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: mercurial-devel-bounces@mercurial-scm.org Sender: "Mercurial-devel" <mercurial-devel-bounces@mercurial-scm.org>

Comments

phabricator - Nov. 1, 2019, 4:39 p.m.

maho created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D7188

AFFECTED FILES
  mercurial/extensions.py

CHANGE DETAILS




To: maho, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/mercurial/extensions.py b/mercurial/extensions.py
--- a/mercurial/extensions.py
+++ b/mercurial/extensions.py
@@ -549,6 +549,8 @@ 
       extensions.wrapcommand(commands.table, 'bookmarks', exbookmarks,
                              synopsis, docstring)
     '''
+    assert command == stringutil.forcebytestr(command), \
+        "`command` should be bytes(python3)/str(python2)"
     assert callable(wrapper)
     aliases, entry = cmdutil.findcmd(command, table)
     for alias, e in pycompat.iteritems(table):

Patchwork D7188: git: force using bytes in command name

Comments

Patch