Skip to content
Snippets Groups Projects
  1. Dec 23, 2019
  2. Dec 22, 2019
    • Matt Harbison's avatar
      revlog: split the content verification of a node into a separate method · 7c3118b9
      Matt Harbison authored
      This will be used by LFS to tune what is skipped.
      
      In the future, this could also be used by LFS to indicate which nodes tagged
      with `skipread` are simply in need of a blob fetch, so that they can be done in
      a batch later.  (Currently, `skipread` also indicates censored data and errors.)
      Additionally, it could be used to cache the sha1 hash value for each blob so
      that large blobs don't need to be re-read and re-hashed if they are used by
      multiple nodes.
      
      Differential Revision: https://phab.mercurial-scm.org/D7710
      7c3118b9
    • Matt Harbison's avatar
      verify: update comment to say that lfs doesn't need fulltext to check renames · a447efd9
      Matt Harbison authored
      The reason is that `filelog.renamed()` indirectly calls `filelog.revision()`,
      which is what accesses the full text.  However, LFS wraps `filelog.renamed()`
      and completely handles the case where an LFS blob is in play by using rawdata.
      
      I've got a test to demonstrate that this is the case, and prevent regressions.
      But the `skipread` flag is set on all lfs revisions when using `--no-lfs`,
      regardless of whether or not the blobs are local.  Just above this, that flag is
      consulted, causing the rename checks to be skipped.  That will need to be
      loosened up first.
      
      Differential Revision: https://phab.mercurial-scm.org/D7709
      a447efd9
  3. Dec 18, 2019
  4. Dec 13, 2019
  5. Dec 07, 2019
  6. Dec 20, 2019
  7. Dec 13, 2019
    • Pierre-Yves David's avatar
      rust-index: add a function to convert PyObject index for hg-core · f98f0e3d
      Pierre-Yves David authored
      Function in hg-core need something implementing the `Graph` trait. Right now,
      the `hg-cpython` entry points directly turn the PyObject passed as argument
      into a `cindex::Index`. However, if we start having the option to use an Index
      in Rust, we need to dispatch between the different possible PyObject we could
      receive.
      
      So move the "duplicate" call into a unified function. When time come. It will be
      easy to update the logic of all interface when the time come.
      
      Differential Revision: https://phab.mercurial-scm.org/D7653
      f98f0e3d
  8. Dec 11, 2019
  9. Dec 18, 2019
  10. Dec 19, 2019
  11. Dec 17, 2019
  12. Dec 18, 2019
    • Martin von Zweigbergk's avatar
      rebase: use rewriteutil.precheck() instead of reimplementing it · 71fee456
      Martin von Zweigbergk authored
      After this patch, there's still another place in `rebase.py`, in the
      `--stop` code path, that reimplements `rewriteutil.precheck()`. I
      couldn't fix that place because it `rewriteutil.precheck()` checks
      that there is only one dirstate parent, which fails because we have
      two parents at that point. I think it's incorrect that rebase leaves
      the user with two parents during conflicts, but changing that is way
      out of scope for this series.
      
      Differential Revision: https://phab.mercurial-scm.org/D7685
      71fee456
  13. Dec 17, 2019
  14. Dec 13, 2019
  15. Dec 18, 2019
  16. Dec 17, 2019
  17. Dec 16, 2019
    • Matt Harbison's avatar
      util: move common proxyobserver attributes to the base class · 4222b9d5
      Matt Harbison authored
      Fixes the following pytype warnings:
      
          line 791, in _writedata: No attribute 'logdata' on baseproxyobserver [attribute-error]
          line 792, in _writedata: No attribute 'logdataapis' on baseproxyobserver [attribute-error]
          line 793, in _writedata: No attribute 'fh' on baseproxyobserver [attribute-error]
          line 794, in _writedata: No attribute 'fh' on baseproxyobserver [attribute-error]
          line 799, in _writedata: No attribute 'logdataapis' on baseproxyobserver [attribute-error]
          line 800, in _writedata: No attribute 'fh' on baseproxyobserver [attribute-error]
          line 802, in _writedata: No attribute 'fh' on baseproxyobserver [attribute-error]
          line 803, in _writedata: No attribute 'name' on baseproxyobserver [attribute-error]
          line 805, in _writedata: No attribute 'fh' on baseproxyobserver [attribute-error]
          line 809, in _writedata: No attribute 'logdataapis' on baseproxyobserver [attribute-error]
          line 810, in _writedata: No attribute 'fh' on baseproxyobserver [attribute-error]
          line 814, in _writedata: No attribute 'fh' on baseproxyobserver [attribute-error]
          line 815, in _writedata: No attribute 'name' on baseproxyobserver [attribute-error]
          line 817, in _writedata: No attribute 'fh' on baseproxyobserver [attribute-error]
      
      Differential Revision: https://phab.mercurial-scm.org/D7675
      4222b9d5
  18. Dec 12, 2019
    • Martin von Zweigbergk's avatar
      config: drop debug messages saying where config was read from · 6b687282
      Martin von Zweigbergk authored
      `hg config --debug` includes lines like this:
      
        set config by: $EDITOR
      
      but also lines like this:
      
        $EDITOR: ui.editor=emacs -nw
      
      The `set config by` messages don't seem to provide much additional
      information over what we get from the `$EDITOR:`-type message. I could
      imagine wanting to see which values got overriden by a later entry,
      but that information is already not present. So let's just remove the
      first type of output. My next patch would otherwise amplify the
      redundant output (there would be one `set config by` for each line in
      `mergetools.rc`).
      
      Differential Revision: https://phab.mercurial-scm.org/D7627
      6b687282
  19. Dec 11, 2019
Loading