evolve: remove meaningless transaction nesting Before this patch, functions below nest transaction scope, even though they are invoked only inside a transaction scope created at _solveone(). - _solvebumped() - _solvedivergent() - relocate() via _solveunstable() or _solvebumped() Transaction nesting is useful for localizing "success" (e.g. one scope per commit inside wider scope for multiple committing). But such nesting is redundant for _solveone(), because there is no code path, which causes failure after successfully closing inner transaction(s). In addition to it, this nesting makes it complicated to close current transaction successfully with exception raising inside inner scope, like "hg shelve" at detection of conflicts. "tr.close()" is required at each outer scopes for such case. To remove meaningless transaction nesting, this patch replaces repo.transaction() in functions above by repo.currenttransaction(). This reuses transaction created at _solveone(). This patch also adds 'assert tr' after getting current running transaction, to avoid invocation of functions above without transaction.

     1

recursive-include docs/figures *.svg

     2

include docs/figures/hgview-example.png

     3

include docs/*.rst

     4

include docs/*.py

     5

include docs/tutorials/*.t

     6

include docs/makefile

     7

include docs/static/*.svg

     8

include hgext/__init__.py

     9

include hgext/evolve.py

    10

include hgext/pushexperiment.py

    11

include hgext/simple4server.py

    12

include setup.py

    13

include README

    14

include COPYING

    15

include tests/*.t

    16

include tests/*.py

    17

include tests/_exc-util.sh

    18

include tests/dummyssh

    19

exclude tests/test-oldconvert.t

    20

exclude tests/test-qsync.t

    21

exclude tests/test-drop.t