stack: make a deep copy of `dependencies` before modifying its items The algorithm later on in this method uses .remove() to remove individual elements from items in dependencies, which before this patch modified the cached property contents. So for further use that dictionary was in the form of {1: set([])}, i.e. all sets were empty. This deep copy block could be way simpler, but the problem is that sometimes we get lists of _succs() from evolvebits.builddependencies(). Note: this happens only in topic's stack version of builddependencies() and it looks like a suboptimal way to handle multiple successors (see evolve's counterpart function). stack.builddependencies method is removed, it has served its purpose (see the previous patch).

     1

import sys

     2

     3

formatted_args = []

     4

     5

UNSAFE_CHARACTERS = [" ", "!", "\"", "#", "$", "&", "'", "(", ")", "*", ",", ";", "<", ">", "?", "[", "\\", "]", "^", "`", "{", "|", "}", ":", "~", "/"]

     6

     7

     8

def find_unsafe(arg):

     9

    for unsafe in UNSAFE_CHARACTERS:

    10

        if unsafe in arg:

    11

            return True

    12

    13

    return False

    14

    15

    16

for arg in sys.argv[1:]:

    17

    if find_unsafe(arg):

    18

        formatted_args.append('"%s"' % arg)

    19

    else:

    20

        formatted_args.append(arg)

    21

    22

print("$ " + " ".join(formatted_args))