when buildInfo is a dict with nvr like

{'name': 'N',
  'version': 'V',
  'release': 'R'}

it is still a valid input to trigger get_build().
Maybe checking if extra is present when it is a dict is better?

once get_build_type() get optimized, this line could be

btypeinfo = get_build_type(kws['info'])

to save one get_build() call

@julian8628 fixed

... or not 'extra' in buildInfo - most extra could be there and could be empty.

Shouldn't we return all btypes? @julian8628?

Shouldn't we return all btypes? @julian8628?

oh, yes. it should be all btypes

@tkopecek @julian8628 fixed

maybe 'build_types' or 'build types' to indicate it is a dict?

otherwise, :thumbsup:

Can we just name this "btypes" instead of "build types"? The term "btype" means something specifically in Koji.

EDIT: Ok, I read get_build_type() and I see that can return multiple types, so plural is right.

EDIT2: I'm wondering if all messaging systems support a space character in a header (like "build type")

ok, let's use btypes :)

@ktdreyer fixed

LGTM, thanks!

Commit 9b45805 fixes this pull-request