From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: alsa-drivers-HEAD does not compile against recent RHE5.1 kernels Date: Wed, 04 Jun 2008 17:16:12 +0200 Message-ID: References: <1211506900.3811.105.camel@localhost> <1211542742.3811.139.camel@localhost> <1211545221.3811.142.camel@localhost> <1212104263.17410.114.camel@localhost> <1212592348.28032.111.camel@localhost> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx1.suse.de (ns.suse.de [195.135.220.2]) by alsa0.perex.cz (Postfix) with ESMTP id 0A17A2447D for ; Wed, 4 Jun 2008 17:16:14 +0200 (CEST) In-Reply-To: <1212592348.28032.111.camel@localhost> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Ben Stanley Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org At Thu, 05 Jun 2008 01:12:28 +1000, Ben Stanley wrote: > > So, if config.h is not needed in 'newer' kernels (and causes an error?), > and is needed in 'older' kernels, it is then an error to universally > include it in a configure test program where it causes an error if used > inappropriately... Well, apparently it's only RH kernels that cause the problem. So I haven't cared about it at all. > I think the error is not noticed in newer newer kernels because > init_utsname is defined in newer newer kernels. (I should look into this > statement some more.) The problem is, as usual, RH kernels backported init_utsname in a different way than the upstream. This makes things really hard. Takashi > I think it comes down to figuring out the point between 'newer' and > 'older' kernels and modifying the configure test appropriately to use or > not use config.h as required. I'm not sure how that should be done. > > Ben. > > On Wed, 2008-06-04 at 14:15 +0200, Takashi Iwai wrote: > > At Fri, 30 May 2008 09:37:43 +1000, > > Ben Stanley wrote: > > > > > > [PING] > > > > > > I need some help on this one. I don't know how to determine whether or > > > not the config.h file should be included in the test program or not. > > > > config.h is needed for older kernels indeed. > > > > > > Takashi > > > > > > > > Ben Stanley. > > > > > > On Fri, 2008-05-23 at 22:20 +1000, Ben Stanley wrote: > > > > Further to my previous email, > > > > > > > > The config.log file shows the following output for the test for > > > > init_utsname in the case of the RHEL 5.1 kernel 2.6.18-53.1.19: > > > > > > > > configure:10792: checking for init_utsname > > > > configure:10822: gcc -c -Wall -fno-strict-aliasing -g -O2 > > > > -I/lib/modules/2.6.18-53.1.19.el5/build/include > > > > -I/lib/modules/2.6.18-53.1.19.el5/source/include -Iinclude -nostdinc > > > > -iwithprefix include -DKBUILD_STR(s)=#s > > > > -DKBUILD_BASENAME=KBUILD_STR(dummy) -DKBUILD_MODNAME=KBUILD_BASENAME > > > > -Werror conftest.c >&5 > > > > In file included from conftest.c:52: > > > > /lib/modules/2.6.18-53.1.19.el5/build/include/linux/config.h:6:2: error: > > > > #warning Including config.h is deprecated. > > > > configure:10828: $? = 1 > > > > configure: failed program was: > > > > > > > > I think it is failing because of the config.h warning. > > > > > > > > Ben Stanley. > > > > >