Patchwork D7203: fsmonitor: reapply dd35abc409ee

login
register
mail settings
Submitter phabricator
Date Nov. 2, 2019, 10:04 p.m.
Message ID <differential-rev-PHID-DREV-7j5gdavafce76jwj5lzr-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/42685/
State Superseded
Headers show

Comments

phabricator - Nov. 2, 2019, 10:04 p.m.
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  The recent revendoring of pywatchman undid this bug fix.
  Let's reapply it.

REPOSITORY
  rHG Mercurial

BRANCH
  stable

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

AFFECTED FILES
  hgext/fsmonitor/pywatchman/__init__.py

CHANGE DETAILS




To: indygreg, #hg-reviewers
Cc: mercurial-devel
phabricator - Nov. 20, 2019, 5:51 p.m.
marmoute added a comment.


  Out of curiosity, was the fix submitted upstream too ?

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D7203/new/

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

To: indygreg, #hg-reviewers
Cc: marmoute, mercurial-devel

Patch

diff --git a/hgext/fsmonitor/pywatchman/__init__.py b/hgext/fsmonitor/pywatchman/__init__.py
--- a/hgext/fsmonitor/pywatchman/__init__.py
+++ b/hgext/fsmonitor/pywatchman/__init__.py
@@ -996,7 +996,7 @@ 
             p = subprocess.Popen(cmd, **args)
 
         except OSError as e:
-            raise WatchmanError('"watchman" executable not in PATH (%s)', e)
+            raise WatchmanError('"watchman" executable not in PATH (%s)' % e)
 
         stdout, stderr = p.communicate()
         exitcode = p.poll()