On Tuesday, December 21, 2021 5:49:34 PM CET Maxwell G wrote:
On Tuesday, December 21, 2021 8:22:55 AM CST Miro Hrončok wrote:
On 19. 12. 21 22:39, Pavel Raiskup wrote:
On Sunday, December 19, 2021 10:22:57 PM CET Pavel Raiskup wrote:
So it seems that fedpkg doesn't (yet) know there's ~/.config/mock* at all.
Proposed fix: https://pagure.io/rpkg/pull-request/595
It seems that this would only fix the issue if the symbolic link has already been created. But before that, `fedpkg --release epel8 mockbuild` would still fail, wouldn't it? Can we fix that as well, even if it's fixed in some documentation only?
Yes, I think `fedpkg --release epel8 mockbuild` should print the same error message mock does when no default has been set, instead of defaulting to a non-functional koji config.
Fedpkg _automatically_ downloads the relevant Koji config ('koji mock-config' variant, which is broken for normal users) when mock doesn't ship the appropriate config file (it doesn't by default).
@onosek, would it be too bad if we made the Koji config download feature opt-in in fedpkg? Something like 'fedpkg mockbuild --use-koji-mock-config'? - when used, all local mock configuration is ignored - when not used, 'mock -r epel-8-x86_64' even if the config doesn't exist, which would automatically trigger the new Mock error message
Pavel