[Yum-devel] [PATCH 2/2] Return exitcode for success in finding matching files.

Zdenek Pavlas zpavlas at redhat.com
Wed Jul 24 09:20:53 UTC 2013


> +    if len(outputpackages) > 0:
> +        return 0
> +    else:
> +        return 1

> -        main(sys.argv[1:])
> +        ret = main(sys.argv[1:])
> +        sys.exit(ret)

ACK, but why didn't you just:

+    if not outputpackages:
+        sys.exit(1)

> While adding that patch, I have also made the script (almost)
> flake8-clean.

What's wrong with trailing whitespaces?


More information about the Yum-devel mailing list