comparison mercurial/upgrade.py @ 31898:9db45228da52

upgrade: directly iterate over optimisations

Since we already have the list of optimisations independent from the
deficiencies, we can use it directly.

(we make a dual assignement in this changeset to simplify the next one)

31897:d0067250542d

        raise error.Abort(_('cannot upgrade repository; do not support '
                            'destination requirement: %s') %
                          _(', ').join(sorted(unsupportedreqs)))

    # Find and validate all improvements that can be made.
    optimizations = findoptimizations(repo)

    # Validate arguments.
    unknownoptimize = optimize - set(i.name for i in optimizations)
    if unknownoptimize:
        raise error.Abort(_('unknown optimization action requested: %s') %

                   'changes:\n\n'))

        printrequirements()
        printupgradeactions()

        unusedoptimize = [i for i in improvements
                          if i.name not in actions and i.type == optimisation]
        if unusedoptimize:
            ui.write(_('additional optimizations are available by specifying '
                     '"--optimize <name>":\n\n'))
            for i in unusedoptimize:
                ui.write(_('%s\n   %s\n\n') % (i.name, i.description))

31898:9db45228da52

        raise error.Abort(_('cannot upgrade repository; do not support '
                            'destination requirement: %s') %
                          _(', ').join(sorted(unsupportedreqs)))

    # Find and validate all improvements that can be made.
    alloptimizations = optimizations = findoptimizations(repo)

    # Validate arguments.
    unknownoptimize = optimize - set(i.name for i in optimizations)
    if unknownoptimize:
        raise error.Abort(_('unknown optimization action requested: %s') %

                   'changes:\n\n'))

        printrequirements()
        printupgradeactions()

        unusedoptimize = [i for i in alloptimizations
                          if i.name not in actions]
        if unusedoptimize:
            ui.write(_('additional optimizations are available by specifying '
                     '"--optimize <name>":\n\n'))
            for i in unusedoptimize:
                ui.write(_('%s\n   %s\n\n') % (i.name, i.description))