From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jim Meyering Date: Thu, 25 Jun 2009 19:30:30 +0200 Subject: [Cluster-devel] [PATCH] avoid configure failure with F10's 2.6.27.21 kernel headers In-Reply-To: <1245501817.3665.338.camel@cerberus.int.fabbione.net> (Fabio M. Di Nitto's message of "Sat, 20 Jun 2009 14:43:37 +0200") References: <87hbycw048.fsf@meyering.net> <1245501817.3665.338.camel@cerberus.int.fabbione.net> Message-ID: <87y6rg9q6h.fsf@meyering.net> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Fabio M. Di Nitto wrote: > Hi Jim, > > On Fri, 2009-06-19 at 20:26 +0200, Jim Meyering wrote: >> Without the patch below, ./configure would fail like this on Fedora 10: >> >> checking linux/dlm_plock.h usability... no >> checking linux/dlm_plock.h presence... yes >> configure: WARNING: linux/dlm_plock.h: present but cannot be compiled >> configure: WARNING: linux/dlm_plock.h: check for missing prerequisite headers? >> configure: WARNING: linux/dlm_plock.h: see the Autoconf documentation >> configure: WARNING: linux/dlm_plock.h: section "Present But Cannot Be Compiled" >> configure: WARNING: linux/dlm_plock.h: proceeding with the compiler's result >> configure: WARNING: ## --------------------------------------- ## >> configure: WARNING: ## Report this to linux-cluster at redhat.com ## >> configure: WARNING: ## --------------------------------------- ## >> checking for linux/dlm_plock.h... no >> configure: error: Unable to find dlm kernel headers. >> Make sure to install kernel headers 2.6.26 (or higher) or use --with-kernel=path >> configure option to point to the kernel source. >> [Exit 1] >> >> and config.log showed the problem: >> >> In file included from conftest.c:79:0: >> /usr/include/linux/dlm_plock.h:25:2: error: expected specifier-qualifier-list before '__u32' >> >> Here's the patch: > > I haven't tested this patch directly but it looks good to go in. Pushed to dlm.git. > gfs2-utils.git might benefit for a similar change, let me know if you > have time to do go for it, or I'll do it at the next round of merges. Doing it there, too.