Patchwork D3404: scripts: remove code for special handling of generaldelta; r?sheehan

login
register
mail settings
Submitter phabricator
Date April 17, 2018, 9:58 p.m.
Message ID <differential-rev-PHID-DREV-vqy6ebckqqmpi37thym5-req@phab.mercurial-scm.org>
Download mbox | patch
Permalink /patch/31172/
State Superseded
Headers show

Comments

phabricator - April 17, 2018, 9:58 p.m.
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  generaldelta has been the default for new Mercurial repos for
  probably 2 years now. Setting format.generaldelta=true had no effect
  because it was redundant with the default. So stop doing that.

REPOSITORY
  rHG Mercurial

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

AFFECTED FILES
  scripts/sync-hg-repos-from-manifest

CHANGE DETAILS




To: indygreg, #hg-reviewers
Cc: mercurial-devel
phabricator - April 17, 2018, 9:59 p.m.
indygreg abandoned this revision.
indygreg added a comment.


  Sent this to the wrong Phabricator instance :)

REPOSITORY
  rHG Mercurial

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

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

Patch

diff --git a/scripts/sync-hg-repos-from-manifest b/scripts/sync-hg-repos-from-manifest
--- a/scripts/sync-hg-repos-from-manifest
+++ b/scripts/sync-hg-repos-from-manifest
@@ -49,8 +49,6 @@ 
         if not os.path.exists(path):
             is_clone = True
             args = [HG, 'init', path]
-            if 'generaldelta' in requires:
-                args.extend(['--config', 'format.generaldelta=true'])
 
             print('creating empty repo: %s' % path)
             subprocess.check_output(args, cwd='/', stderr=subprocess.STDOUT)