comparison mercurial/localrepo.py @ 15584:9df9444e96ec

addchangegroup: use a postrelease callback to call changegroup hook

This is will allow addchangegroup to drop lock related code.

15583:926a06f7a353

                phases.advanceboundary(self, 0, added)
            # make changelog see real files again
            cl.finalize(trp)

            tr.close()
        finally:
            tr.release()
            if lock:
                lock.release()

        if changesets > 0:
            # forcefully update the on-disk branch cache
            self.ui.debug("updating the branch cache\n")
            self.updatebranchcache()
            self.hook("changegroup", node=hex(cl.node(clstart)),
                      source=srctype, url=url)

            for n in added:
                self.hook("incoming", node=hex(n), source=srctype, url=url)

        # never return 0 here:
        if dh < 0:
            return dh - 1
        else:
            return dh + 1

15584:9df9444e96ec

                phases.advanceboundary(self, 0, added)
            # make changelog see real files again
            cl.finalize(trp)

            tr.close()

            def postaddchangegroup():
                if changesets > 0:
                    # forcefully update the on-disk branch cache
                    self.ui.debug("updating the branch cache\n")
                    self.updatebranchcache()
                    self.hook("changegroup", node=hex(cl.node(clstart)),
                              source=srctype, url=url)

                    for n in added:
                        self.hook("incoming", node=hex(n), source=srctype,
                                  url=url)
            self._postrelease(postaddchangegroup)

        finally:
            tr.release()
            if lock:
                lock.release()
        # never return 0 here:
        if dh < 0:
            return dh - 1
        else:
            return dh + 1