Patchwork [3,of,4,V2] bookmark: deprecate 'bmstore.write' method

mail settings
Submitter Pierre-Yves David
Date Dec. 7, 2015, 7:33 p.m.
Message ID <>
Download mbox | patch
Permalink /patch/11907/
State Accepted
Headers show


Pierre-Yves David - Dec. 7, 2015, 7:33 p.m.
# HG changeset patch
# User Pierre-Yves David <>
# Date 1449387247 28800
#      Sat Dec 05 23:34:07 2015 -0800
# Node ID 4469287024fbad9588cb3a75daa1aa76d80d4a76
# Parent  abc6aeab55afb3158a6b34b9ba1f8917835a3843
# EXP-Topic deprecationwarning
# Available At
#              hg pull -r 4469287024fb
bookmark: deprecate 'bmstore.write' method

This function does not collaborate with the transaction and must disappear. As
we have likely a lot of third party users, we make it deprecated to let them some
time to upgrade their code.

Thanks goes to Laurent Charignon for cleanup the last remains of the 'write'


diff --git a/mercurial/ b/mercurial/
--- a/mercurial/
+++ b/mercurial/
@@ -103,10 +103,12 @@  class bmstore(dict):
         in a format equal to those of localtags.
         We also store a backup of the previous state in undo.bookmarks that
         can be copied back on rollback.
+        msg = 'bm.write() is deprecated, use bm.recordchange(transaction)'
+        self._repo.ui.deprecwarn(msg, '3.7')
         if self._clean:
         repo = self._repo
         if (repo.ui.configbool('devel', 'all-warnings')
                 or repo.ui.configbool('devel', 'check-locks')):