This needs clearer handling of the separators. In particular, the builder code expects space (repo['arches'].split()), but the cli suggests commas (metavar="ARCH1,ARCH2, ...").

Elsewhere in Koji, we tend to use spaces to separate arches in a list since this makes splitting easier. However, passing a space separated list to an --arch option is awkward because of the quoting needed. I suggest accepting either one in the cli and passing a space separated list to the hub call. I thought we had code to do this for archlists somewhere else, but I can't seem to find it right now.

ah, yes, we've that koji.parse_arches - added

ah, yes, we've that koji.parse_arches - added

I don't see this in the current change set. Was it lost somewhere in the rebase?

Without it, my concerns above still apply.

I think the best place to handle this is in the hub call. I'd like to be sure that the arch list is sanitized to space separated before it goes into the db.

Hmm, bad rebase. Updated.

Commit 5e3e7c4b fixes this pull-request