D110: sparse: add a requirement when a repository uses sparse (BC)

indygreg (Gregory Szorc) phabricator at mercurial-scm.org
Mon Jul 17 18:26:12 EDT 2017


indygreg added inline comments.

INLINE COMMENTS

> martinvonz wrote in sparse.py:526
> It feels like this would be clearer as two method: the old one and a new one that just turns sparse off. That would ideally unlink .hg/sparse, no? The new method would not accept includes, excludes, or profiles, and force also doesn't seem relevant as far as I can tell.

Yes, it would be cleaner to have a dedicated function to turn off sparse. Yes, it would unlink .hg/sparse. I was being a bit lazy when I wrote this patch because I wanted to get something in before the freeze :)

REPOSITORY
  rHG Mercurial

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

EMAIL PREFERENCES
  https://phab.mercurial-scm.org/settings/panel/emailpreferences/

To: indygreg, #hg-reviewers, durin42
Cc: martinvonz, mercurial-devel


More information about the Mercurial-devel mailing list