Note this includes a change which can potentially break existing configs. If you have a config that inherits from a Fedora 31, 32 or Rawhide config, or a RHEL 8 config, and adds to the yum.conf section - something like this, from openQA, which does exactly this:
it will suddenly stop working with this version of mock, with an error KeyError: 'yum.conf'. This is because of this commit, which changes these configs from having a yum.conf section to having a dnf.conf section.
These updates do seem to be somewhat against the Fedora and EPEL update policies, which discourage shipping major releases with incompatible changes as updates for stable releases.
There's actually commit from november 2019 which flipped the default for
Fedora to 'dnf.conf', the cited one is just "sync" to make everything
behave the same.
These updates do seem to be somewhat against the Fedora and EPEL update
policies, which discourage shipping major releases with incompatible
changes as updates for stable releases.
Well, mock has to adapt a lot to new environments outside from EPEL (you
need to adapt epel7 mock for building e.g. fedora rawhide). What you
suggest would make mock mostly unmaintainable (we would have to maintain
two versions, help with this would be welcome).
But the yum.conf -> dnf.conf was probably rather cosmetic change, and
should happen only in newly added configs. I realized this too late after
taking over the maintenance. If you have some good proposal (fixing mock
somehow so we make it compatible with both variants or so) please suggest.
Btw., the include() feature itself is pretty new thing, definitely
brought by updates to el7 (maybe even epel8). I didn't know it is so
widely adopted already. Sorry for inconvenience.
This update has been submitted for testing by praiskup.
This update's test gating status has been changed to 'waiting'.
This update's test gating status has been changed to 'ignored'.
praiskup edited this update.
This update has been pushed to testing.
praiskup edited this update.
New build(s):
Removed build(s):
Karma has been reset.
This update has been submitted for testing by praiskup.
praiskup edited this update.
praiskup edited this update.
This update has been pushed to testing.
praiskup edited this update.
New build(s):
Removed build(s):
Karma has been reset.
This update has been submitted for testing by praiskup.
praiskup edited this update.
Note this includes a change which can potentially break existing configs. If you have a config that inherits from a Fedora 31, 32 or Rawhide config, or a RHEL 8 config, and adds to the
yum.conf
section - something like this, from openQA, which does exactly this:it will suddenly stop working with this version of mock, with an error
KeyError: 'yum.conf'
. This is because of this commit, which changes these configs from having ayum.conf
section to having adnf.conf
section.These updates do seem to be somewhat against the Fedora and EPEL update policies, which discourage shipping major releases with incompatible changes as updates for stable releases.
This update has been pushed to testing.
praiskup edited this update.
New build(s):
Removed build(s):
Karma has been reset.
This update has been submitted for testing by praiskup.
There's actually commit from november 2019 which flipped the default for Fedora to 'dnf.conf', the cited one is just "sync" to make everything behave the same.
Well, mock has to adapt a lot to new environments outside from EPEL (you need to adapt epel7 mock for building e.g. fedora rawhide). What you suggest would make mock mostly unmaintainable (we would have to maintain two versions, help with this would be welcome).
But the
yum.conf
->dnf.conf
was probably rather cosmetic change, and should happen only in newly added configs. I realized this too late after taking over the maintenance. If you have some good proposal (fixing mock somehow so we make it compatible with both variants or so) please suggest.Btw., the
include()
feature itself is pretty new thing, definitely brought by updates to el7 (maybe even epel8). I didn't know it is so widely adopted already. Sorry for inconvenience.This update has been pushed to testing.
This update can be pushed to stable now if the maintainer wishes
This update has been obsoleted by mock-2.1-1.el8.