D2754: phabricator: print deprecation warning only once

joerg.sonnenberger (Joerg Sonnenberger) phabricator at mercurial-scm.org
Fri Mar 9 15:10:39 UTC 2018


joerg.sonnenberger 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/D2754

AFFECTED FILES
  contrib/phabricator.py

CHANGE DETAILS

diff --git a/contrib/phabricator.py b/contrib/phabricator.py
--- a/contrib/phabricator.py
+++ b/contrib/phabricator.py
@@ -99,13 +99,17 @@
     process('', params)
     return util.urlreq.urlencode(flatparams)
 
+printed_token_warning = False
+
 def readlegacytoken(repo):
     """Transitional support for old phabricator tokens.
 
     Remove before the 4.6 release.
     """
+    global printed_token_warning
     token = repo.ui.config('phabricator', 'token')
-    if token:
+    if token and not printed_token_warning:
+        printed_token_warning = True
         repo.ui.warn(_('phabricator.token is deprecated - please '
                        'migrate to the phabricator.auth section.\n'))
     return token



To: joerg.sonnenberger, #hg-reviewers
Cc: mercurial-devel


More information about the Mercurial-devel mailing list