[Commented On] D8771: localrepo: only use BOOKMARKS_IN_STORE_REQUIRMENT is we have a store

[martinvonz (Martin von Zweigbergk)]

martinvonz added a comment.


  The commit message was hard to read.
  
  > localrepo: only use BOOKMARKS_IN_STORE_REQUIRMENT is we have a store
  
  My suggestion: localrepo: only use 'bookmarksinstore' requirement if we have 'store'
  
  > This adds check that if we are using store and if we are not, we skip adding the
  > BOOKMARKS_IN_STORE_REQUIREMENT.
  
  My suggestion: This adds a check that we are have the 'store' requirement and if we are not, we skip adding the 'bookmarksinstore'.
  
  Regarding the patch itself, when will it be useful? Is the point to detect cases where we have created such repos? Or to prevent the user from creating new repos? Why don't we update the read path to ignore 'bookmarksinstore' if 'store' is not enabled? Can we have a test case or two (just manually add 'bookmarksinstore' to `.hg/requires`)?

REPOSITORY
  rHG Mercurial

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

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

To: pulkit, #hg-reviewers
Cc: martinvonz, marmoute, mercurial-patches
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-patches/attachments/20200801/e6a0c286/attachment-0002.html>