From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH] kmod: Update to 29
Date: Tue, 04 Jan 2022 22:10:07 +0000
Message-ID: <0753C71D-F748-479F-9883-6F85E0C564DC@ipfire.org>
In-Reply-To: <35e9cc23-d37f-c45f-dfbe-acedd98d1a41@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0139262676488509009=="
List-Id: <development.lists.ipfire.org>

--===============0139262676488509009==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Reviewed-by: Michael Tremer <michael.tremer(a)ipfire.org>

> On 4 Jan 2022, at 18:29, Peter M=C3=BCller <peter.mueller(a)ipfire.org> wro=
te:
>=20
> Full changelog as given in the NEWS file:
>=20
> kmod 29
> =3D=3D=3D=3D=3D=3D=3D
>=20
> - Improvements
> 	- Add support to use /usr/local as a place for configuration files. This m=
akes it easier
> 	  to install locally without overriding distro files.
>=20
> - Bug fixes
> 	- Fix `modinfo -F` when module is builtin: when we asked by a specific fie=
ld from modinfo,
> 	  it was not working correctly if the module was builtin
>=20
> 	- Documentation fixes on precedence order of /etc and /run: the correct or=
der is
> 	  /etc/modprobe.d, /run/modprobe.d, /lib/modprobe.d
>=20
> 	- Fix the priority order that we use for searching configuration files. The
> 	  correct one is /etc, /run, /usr/local/lib, /lib, for both modprobe.d
> 	  and depmo.d
>=20
> 	- Fix kernel command line parsing when there are quotes present. Grub
> 	  mangles the command line and changes it from 'module.option=3D"val with
> 	  spaces"' to '"module.option=3Dval with spaces"'. Although this is weird
> 	  behavior and grub could have been fixed, the kernel understands it
> 	  correctly for builtin modules. So change libkmod to also parse it
> 	  correctly. This also brings another hidden behavior from the kernel:
> 	  newline in the kernel command line is also allowed and can be used to
> 	  separate options.
>=20
> 	- Fix a memory leak, overflow and double free on error path
>=20
> 	- Fix documentation for return value from kmod_module_get_info(): we
> 	  return the number of entries we added to the list
>=20
> 	- Fix output of modules.builtin.alias.bin index: we were writing an empty =
file due to
> 	  the misuse of kmod_module_get_info()
>=20
> - Infra/internal
> 	- Retire integration with semaphoreci
>=20
> 	- Declare the github mirror also as an official upstream source: now besid=
es accepting
> 	  patches via mailing list, PRs on github are also acceptable
>=20
> 	- Misc improvements to testsuite, so we can use it reliably regardless
> 	  of the configuration used: now tests will skip if we don't have the
> 	  build dependencies)
>=20
> Signed-off-by: Peter M=C3=BCller <peter.mueller(a)ipfire.org>
> ---
> config/rootfiles/common/kmod | 2 +-
> lfs/kmod                     | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
>=20
> diff --git a/config/rootfiles/common/kmod b/config/rootfiles/common/kmod
> index 56806fd1f..f46db860b 100644
> --- a/config/rootfiles/common/kmod
> +++ b/config/rootfiles/common/kmod
> @@ -9,6 +9,6 @@ sbin/rmmod
> #usr/lib/libkmod.la
> #usr/lib/libkmod.so
> usr/lib/libkmod.so.2
> -usr/lib/libkmod.so.2.3.6
> +usr/lib/libkmod.so.2.3.7
> #usr/lib/pkgconfig/libkmod.pc
> #usr/share/bash-completion/completions/kmod
> diff --git a/lfs/kmod b/lfs/kmod
> index 3d9acd899..5499002b6 100644
> --- a/lfs/kmod
> +++ b/lfs/kmod
> @@ -1,7 +1,7 @@
> ###########################################################################=
####
> #                                                                          =
   #
> # IPFire.org - A linux based firewall                                      =
   #
> -# Copyright (C) 2007-2018  IPFire Team  <info(a)ipfire.org>               =
      #
> +# Copyright (C) 2007-2022  IPFire Team  <info(a)ipfire.org>               =
      #
> #                                                                          =
   #
> # This program is free software: you can redistribute it and/or modify     =
   #
> # it under the terms of the GNU General Public License as published by     =
   #
> @@ -24,7 +24,7 @@
>=20
> include Config
>=20
> -VER        =3D 28
> +VER        =3D 29
>=20
> THISAPP    =3D kmod-$(VER)
> DL_FILE    =3D $(THISAPP).tar.xz
> @@ -40,7 +40,7 @@ objects =3D $(DL_FILE)
>=20
> $(DL_FILE) =3D $(DL_FROM)/$(DL_FILE)
>=20
> -$(DL_FILE)_MD5 =3D 0a2b887b1b3dfb8c0b3f41f598203e56
> +$(DL_FILE)_MD5 =3D e81e63acd80697d001c8d85c1acb38a0
>=20
> install : $(TARGET)
>=20
> --=20
> 2.26.2


--===============0139262676488509009==--