PR#2792: cli: mock-config check arch -

koji

#2792 cli: mock-config check arch

Merged Apr 19, 2021 by tkopecek. Opened Mar 30, 2021 by jcupova.

jcupova/koji issue2640 into master

jcupova commented Mar 30, 2021

Fixes: https://pagure.io/koji/issue/2640

tkopecek commented Mar 30, 2021

All these tests should happen in final part of command (on buildcfg). Otherwise you'll call getBuildConfig more times than needed. It should raise different warning if there are no archs in the tag than if arch is not in that list. It would be confusing otherwise.

rebased onto 7d58284b4e860c591a4baa1524fb14a02cdde0e4

Mar 31, 2021

rebased onto 694d7ef8df5459892b41ccef0a790da49c6285ac

Mar 31, 2021

jcupova commented Mar 31, 2021

@tkopecek fixed

rebased onto fd53b5581cf1ad9a1c6e76b03725f6a919b9f13a

Mar 31, 2021

tkopecek commented Mar 31, 2021

:thumbsup:

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

Mar 31, 2021

rebased onto 8d435821c0124a8f6ae51222c2fd7fb966872f61

Apr 1, 2021

Metadata Update from @mfilip:
- Pull-request tagged with: testing-done

Apr 13, 2021

tkopecek commented Apr 19, 2021

Commit 7665ea83 fixes this pull-request

Pull-Request has been merged by tkopecek

Apr 19, 2021

Metadata

Assignee

None

Tags

koji

#2792 cli: mock-config check arch Merged Apr 19, 2021 by tkopecek. Opened Mar 30, 2021 by jcupova. jcupova/koji issue2640 into master

Metadata

testing-done testing-ready

#2792 cli: mock-config check arch

Merged Apr 19, 2021 by tkopecek. Opened Mar 30, 2021 by jcupova.

jcupova/koji issue2640 into master