diff --git a/planex/cmd/pin.py b/planex/cmd/pin.py index cc469813..695fce24 100644 --- a/planex/cmd/pin.py +++ b/planex/cmd/pin.py @@ -51,13 +51,11 @@ def repo_or_path(arg): (URL, None) """ if arg.startswith("ssh://"): - split = arg.split("#") - if len(split) > 2 or not split: + if arg.count('#') > 1: raise ValueError( "Expected URL or ssh://URL#commitish but got {}".format(arg)) - if len(split) == 1: - return (arg, None) - return tuple(split) + if '#' in arg: + return tuple(arg.split("#")) return (arg, None) @@ -145,8 +143,8 @@ def get_pin_content(args, spec): pinfile["PatchQueue0"]["prefix"] = resources["PatchQueue0"].prefix # When both a PQ0 and an Archive0 are present, and point to the same - # repository, we assume that they are pointint to the same tarball. - # This, by default, planex-pin will overwrite the Archive0 with + # repository, we assume that they are pointing to the same tarball. + # Thus, by default, planex-pin will overwrite the Archive0 with # the same content as PatchQueue0. This could fail when multiple # archives are present and the one matching the PQ is not the first # one, but for now I value the simplicity of the code over covering