merge: use constants for actions We finish up establishing named constants in this file with actions. I remember scratching my head trying to figure out what this code was doing as part of addressing a recent security issue with subrepos. Having the named constants in place definitely makes things easier to read. I'm not convinced the new constants have the best names (I'm not an expert in this code). But they can be changed easily enough. Also, since these constants are internal only, we might want to change their values to something more human readable to facilitate debugging. Or maybe we could employ an enum type some day... Differential Revision: https://phab.mercurial-scm.org/D2701

     1

from __future__ import (

     2

    absolute_import,

     3

    print_function,

     4

)

     5

     6

import argparse

     7

import os

     8

     9

ap = argparse.ArgumentParser()

    10

ap.add_argument('path', nargs='+')

    11

opts = ap.parse_args()

    12

    13

def gather():

    14

    for p in opts.path:

    15

        if not os.path.exists(p):

    16

            return

    17

        if os.path.isdir(p):

    18

            yield p + os.path.sep

    19

            for dirpath, dirs, files in os.walk(p):

    20

                for d in dirs:

    21

                    yield os.path.join(dirpath, d) + os.path.sep

    22

                for f in files:

    23

                    yield os.path.join(dirpath, f)

    24

        else:

    25

            yield p

    26

    27

print('\n'.join(sorted(gather(), key=lambda x: x.replace(os.path.sep, '/'))))