On Friday 28 August 2009 11:24:15 Cyril Hrubis wrote:
> Hi!
>
> > this patch adds autoconf checks for some build failures reported by Cyril
> > Hrubis. Please note I have not tested this on an old kernel.
> >
> > Also, it corrects the names of the preprocessor macros defined by
> > autoconf
> >
> > The remaining issue (the cgroupstats.h file) should be solved using the
> > autoconf test that was committed recently.
>
> The autoconf part is okay, but getdelays.c needs some more #ifdef
> HAVE_LINUX_CGROUPSTAT_H to build correctly on all I have here.
>
> Patch attached, however it may need minor cleanups.

preprocessor macros should never be intended

can the test case pass at all if the linux headers are missing ?  if they 
cant, there's no point in sprinkling #ifdef's throughout the code.  stub the 
whole thing out.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to