Control: tag -1 moreinfo

Julian Andres Klode <j...@debian.org> (2013-07-21):
> But it might make sense to include the patch:
> 
>   * Patch userland for 3.X kernels
> 
> While there is no bug report for that, it fixes the ndiswrapper
> tool to use modprobe.d and modprobe.conf on 3.X kernels, instead of
> modutils/ and modules.conf.

Speaking of which, that only works for 3.x.y kernels:

> --- /dev/null
> +++ b/debian/patches/Add-support-for-3.x-kernel-versions.patch
> @@ -0,0 +1,28 @@
> +From 71ec9e8a80b74652d660232b560bbe1dfa7915ba Mon Sep 17 00:00:00 2001
> +From: Tim Gardner <tim.gard...@canonical.com>
> +Date: Mon, 13 Feb 2012 14:19:58 -0700
> +Subject: [PATCH] Add support for 3.x kernel versions
> +
> +As suggested by Kano on Freenode #ubuntu-kernel
> +
> +Signed-off-by: Tim Gardner <tim.gard...@canonical.com>
> +---
> + utils/ndiswrapper | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/utils/ndiswrapper b/utils/ndiswrapper
> +index 4bb91f2..ba082a3 100755
> +--- a/utils/ndiswrapper
> ++++ b/utils/ndiswrapper
> +@@ -55,7 +55,7 @@ if (@ARGV < 1) {
> + 
> + my $modconf;
> + if (`uname -r` =~ /(\d+)\.(\d+)\.(\d+)/) {
> +-    if ($2 > 4) {
> ++    if (($2 > 4) || ($1 > 2)) {

The regex isn't anchored (^) and wants 3 components. The third one was
dropped a while ago, but maybe in a version higher than what this module
supports anyway. Just thought I'd mention it…

Mraw,
KiBi.

Attachment: signature.asc
Description: Digital signature

Reply via email to