Hi Micah,
On Tue, 14 Jun 2011 00:29:49 -0500, Micah Gersten wrote:
> I filed this bug [1] in Launchpad for Ubuntu and attached the
> following patch:
>
>
> +--- lm-sensors-3-3.2.0.orig/prog/detect/sensors-detect
> ++++ lm-sensors-3-3.2.0/prog/detect/sensors-detect
> +@@ -2258,7 +2258,8 @@ use vars qw(@kernel_version $kernel_arch
> +
> + sub initialize_kernel_version
> + {
> +- `uname -r` =~ /(\d+)\.(\d+)\.(\d+)(.*)/;
> ++ #Third decimal only for point releases starting with 3.0
> ++ `uname -r` =~ /(\d+)\.(\d+)(\.(\d+))?(.*)/;
> + @kernel_version = ($1, $2, $3, $4);
> + chomp($kernel_arch = `uname -m`);
> +
Did you actually test your patch? I fail to see how it could work.
@kernel_version would contain (2, 6, .39, 39) for kernel 2.6.39.1. I
have the following instead:
@@ -2462,8 +2462,8 @@
sub initialize_kernel_version
{
- `uname -r` =~ /(\d+)\.(\d+)\.(\d+)(.*)/;
- @kernel_version = ($1, $2, $3, $4);
+ `uname -r` =~ /(\d+)\.(\d+)(?:\.(\d+))?(.*)/;
+ @kernel_version = ($1, $2, $3 || 0, $4);
chomp($kernel_arch = `uname -m`);
# We only support kernels >= 2.6.5
which properly sets @kernel_version to (2, 6, 39, .1).
That being said, I would much prefer if Linus could just get a life and
stick with 3-number versions, i.e. 3.0.0. Updating user-space at wide
to cope with this change is just wasting everyone's time at the moment,
for no benefit.
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
