Skip to content

Commit

Permalink
Fix missing Release Notes and bump config->mock requirement
Browse files Browse the repository at this point in the history
  • Loading branch information
praiskup committed Feb 14, 2024
1 parent dcc80f8 commit 5bd8ece
Show file tree
Hide file tree
Showing 9 changed files with 39 additions and 3 deletions.
2 changes: 1 addition & 1 deletion mock-core-configs/mock-core-configs.spec
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ Provides: mock-configs
# distribution-gpg-keys contains GPG keys used by mock configs
Requires: distribution-gpg-keys >= 1.98
# specify minimal compatible version of mock
Requires: mock >= 5.0
Requires: mock >= 5.4.post1
Requires: mock-filesystem

Requires(post): coreutils
Expand Down
17 changes: 17 additions & 0 deletions releng/release-notes-next/init-and-ownership-problems.bugfix
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
Mock [has been][PR#1322] [fixed][commit#27dde5da] so that it no longer
inadvertently changes the group ownership of the in-chroot $HOME directory
to the wrong group ID. In previous versions of Mock, the group ownership
was changed to the effective group ID of the user on the host that
executed Mock. This could confuse some tools during the build process, as
they might want to create similarly owned files at the `rpmbuild` time
(but that assumption would be incorrect, such GID doesn't exist
in-chroot). Now, Mock changes the files to the `mockbuild:mock`
ownership, where the `mock` group ID is always 135 (the same GID on host
and in-chroot). This matches the effective GID for the `rpmbuild`
process, ensuring that the tools executed during the build process have
full control over such files.

While on this, Mock was also [optimized][commit#db64d468202] to do this
ownership change before, and only if, the `rpmbuild` process is started
(so e.g. plain `mock --chroot` commands do not touch the file ownership at
all).
1 change: 1 addition & 0 deletions releng/release-notes-next/mageia-7-eol.config
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Mageia 7 configs [marked as end-of-live][PR#1316].
2 changes: 2 additions & 0 deletions releng/release-notes-next/mageia-cross-arch.config
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Mageia config files started using the `{{ repo_arch }}` option to fix the
[cross-arch builds][issue#1317].
4 changes: 4 additions & 0 deletions releng/release-notes-next/mageia-i686-eol.config
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
The OpenMandriva i686 chroots [have been marked as end-of-life][PR#1315], fixing
[issue#987][] and [issue#1012][].


3 changes: 3 additions & 0 deletions releng/release-notes-next/mock-filesystem-deps.bugfix
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
The `mock` package has been fixed to depend on precisely the same version
of `mock-filesystem` sub-package (product of the same rpm build). This is
to protect against incompatible Mock sub-package installations.
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
Mock parses the DNF configuration specified in `config_opts["dnf.conf"]` itself
to perform some post-processing tasks, such as bind-mounting the on-host repo
directories into the bootstrap chroot based on this DNF config. In previous
versions of Mock, there was an issue where it failed to parse the DNF config if
some of the DNF options contained the '%' symbol. This was due to Python's
ConfigParser raising an %-interpolation exception that was ignored but Mock, but
resulting in Mock ignoring the rest of the config file and finishing without
performing the mentioned bind-mounts. This bug has been fixed, and the '%' sign
is no longer considered to be a special Python ConfigParser character.
2 changes: 1 addition & 1 deletion releng/release-notes-next/report-file-system-type.feature
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ Filesystem Size Used Avail Use% Mou
/dev/mapper/luks-3aa4fbe3-5a19-4025-b70c-1d3038b76bd4 399G 9.1G 373G 3% /
~~~

Newly, also file system type is reported:
Newly, [also file system type is reported][issue#1263]:

~~~
Filesystem Type Size Used Avail Use% Mounted on
Expand Down
2 changes: 1 addition & 1 deletion releng/release-notes-next/use_host_shadow_utils.config
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Added a config option called "use_host_shadow_utils", to account for situations where
[Added][PR#1283] a config option called "use_host_shadow_utils", to account for situations where
users have host shadow-utils configurations that cannot provision or destroy users and
groups in the buildroot; one example of this kind of configuration is using
FreeIPA-provided subids on the buildhost. The option defaults to True since mock has made a conscious
Expand Down

0 comments on commit 5bd8ece

Please sign in to comment.