public inbox for development@lists.ipfire.org
 help / color / mirror / Atom feed
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	[thread overview]
Message-ID: <0753C71D-F748-479F-9883-6F85E0C564DC@ipfire.org> (raw)
In-Reply-To: <35e9cc23-d37f-c45f-dfbe-acedd98d1a41@ipfire.org>

[-- Attachment #1: Type: text/plain, Size: 4012 bytes --]

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

> On 4 Jan 2022, at 18:29, Peter Müller <peter.mueller(a)ipfire.org> wrote:
> 
> Full changelog as given in the NEWS file:
> 
> kmod 29
> =======
> 
> - Improvements
> 	- Add support to use /usr/local as a place for configuration files. This makes it easier
> 	  to install locally without overriding distro files.
> 
> - Bug fixes
> 	- Fix `modinfo -F` when module is builtin: when we asked by a specific field from modinfo,
> 	  it was not working correctly if the module was builtin
> 
> 	- Documentation fixes on precedence order of /etc and /run: the correct order is
> 	  /etc/modprobe.d, /run/modprobe.d, /lib/modprobe.d
> 
> 	- 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
> 
> 	- Fix kernel command line parsing when there are quotes present. Grub
> 	  mangles the command line and changes it from 'module.option="val with
> 	  spaces"' to '"module.option=val 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.
> 
> 	- Fix a memory leak, overflow and double free on error path
> 
> 	- Fix documentation for return value from kmod_module_get_info(): we
> 	  return the number of entries we added to the list
> 
> 	- Fix output of modules.builtin.alias.bin index: we were writing an empty file due to
> 	  the misuse of kmod_module_get_info()
> 
> - Infra/internal
> 	- Retire integration with semaphoreci
> 
> 	- Declare the github mirror also as an official upstream source: now besides accepting
> 	  patches via mailing list, PRs on github are also acceptable
> 
> 	- 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)
> 
> Signed-off-by: Peter Müller <peter.mueller(a)ipfire.org>
> ---
> config/rootfiles/common/kmod | 2 +-
> lfs/kmod                     | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
> 
> 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 @@
> 
> include Config
> 
> -VER        = 28
> +VER        = 29
> 
> THISAPP    = kmod-$(VER)
> DL_FILE    = $(THISAPP).tar.xz
> @@ -40,7 +40,7 @@ objects = $(DL_FILE)
> 
> $(DL_FILE) = $(DL_FROM)/$(DL_FILE)
> 
> -$(DL_FILE)_MD5 = 0a2b887b1b3dfb8c0b3f41f598203e56
> +$(DL_FILE)_MD5 = e81e63acd80697d001c8d85c1acb38a0
> 
> install : $(TARGET)
> 
> -- 
> 2.26.2


      reply	other threads:[~2022-01-04 22:10 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-04 18:29 Peter Müller
2022-01-04 22:10 ` Michael Tremer [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=0753C71D-F748-479F-9883-6F85E0C564DC@ipfire.org \
    --to=michael.tremer@ipfire.org \
    --cc=development@lists.ipfire.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox