Re: [Bug 403509] New: hal-kernel-includes patch does not apply cleany

[Yiannis Mavroukakis <yiannis jaguarfreight com>]

Hi Guenther,

Sorry for the brevity I'll provide a little more detail.
Slackware ships with a choice of two kernels 2.4.33.3 or 2.6.17.13.
With 2.4.33.3 the headers are installed as part of the setup process.
If the 2.6.17.13 kernel is chosen however, the headers need to be
installed as a separate action after the setup has completed. So with
this in mind, the user needs to be mindful to have installed the latest
kernel headers (or symlink or whatever) to get hal to compile. 
However, there may be plenty of installs that don't grab the new headers
so the patch is needed, and against the hal version included in
2.17.latest it fails. Granted, those who choose Slack as a distro are
probably more experienced than most IMHO (no flames please...) and will
take the necessary steps to fix the issue, but the lowest common
denominator should also be catered for..

Yiannis

On Sat, 2007-02-03 at 00:41 +0100, guenther wrote:
> On Fri, 2007-02-02 at 17:16 +0000, Yiannis Mavroukakis wrote:
> > On Fri, 2007-02-02 at 09:47 +0000, Peter wrote:
> 
> > > Patch is not required running slack 11. 
> 
> > Only if the 2.6.x headers have been added, after the end of the
> > installation and many may not be inclined to do so :)
> 
> Not sure if I can follow you here. :)  In which cases is this patch
> still necessary, if any? When would it not be necessary, what needs to
> be done, and what impact does this have?
> 
> Since neither Joseph nor me got any Slackware system, we need your guys
> help and opinion. We sure prefer not to ship any distro-specific hack,
> if possible. Worse, shipping a broken patch...
> 
> 
> Any opinion welcome, discussion much appreciated. ;)  Thanks...
> 
>   guenther
> 
> 
> -- 
> char *t="\10pse\0r\0dtu\0  ghno\x4e\xc8\x79\xf4\xab\x51\x8a\x10\xf4\xf4\xc4";
> main(){ char h,m=h=*t++,*x=t+2*h,c,i,l=*x,s=0; for (i=0;i<l;i++){ i%8? c<<=1:
> (c=*++x); c&128 && (s+=h); if (!(h>>=1)||!t[s+h]){ putchar(t[s]);h=m;s=0; }}}
> 

Note:__________________________________________________________________
This message is for the named person's use only. It may contain
confidential, proprietary or legally privileged information. No
confidentiality or privilege is waived or lost by any mistransmission.
If you receive this message in error, please immediately delete it and
all copies of it from your system, destroy any hard copies of it and
notify the sender. You must not, directly or indirectly, use, disclose,
distribute, print, or copy any part of this message if you are not the
intended recipient. Jaguar Freight Services and any of its subsidiaries
each reserve the right to monitor all e-mail communications through its
networks.
Any views expressed in this message are those of the individual sender,
except where the message states otherwise and the sender is authorized
to state them to be the views of any such entity.
________________________________________________________________________
This e-mail has been scanned for all known viruses.