On Wed, 2013-05-22 at 04:34 -0700, Luis R. Rodriguez wrote:

> +def put(tar_bz2=None, signed_tar=None, path=None):
> +    cmd = ['kup', 'put', tar_bz2, signed_tar, path]

Without even looking, the =None here can't be right since if you don't
pass any of these args, it'll just get completely confused and probably
crash trying to pass None as an argument? Seems you should just require
all these args.

johannes

--
To unsubscribe from this list: send the line "unsubscribe backports" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to