Re: [LTP] [PATCH] cgroups: conditionally enable building cgroup tests

[Francesco RUNDO <francesco.rundo@st.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1504 bytes --]



Mike Frysinger wrote:

>On Monday 06 July 2009 13:28:00 Garrett Cooper wrote:
>  
>
>>On Mon, Jul 6, 2009 at 4:38 AM, Francesco RUNDO wrote:
>>    
>>
>>>I've already fixed the same issue you reported. I've tested it
>>>successfully.
>>>
>>>Subrata has already merged my patch (please, find it in attachment).
>>>
>>>Please, check it before to submit another patch.
>>>      
>>>
>>At the end of the day, the real problem is that we're not using the
>>--with- functionality in autoconf, e.g. --with-cgroups, and instead
>>we're hacking a lot of noise into Makefile's.
>>    
>>
>
>selectively compiling groups of code needs to be thought out before we start 
>throwing --with-foo options at the problem.  otherwise we'll still end up with 
>crap, just in a different form.
>
>  
>
>>I can tell at first glance that that Makefile is a mess anyhow,
>>because it's referencing _hardcoded_ /proc references and as such will
>>fail to cross-compile properly if the target or the host are setup
>>differently from one another, in the following two scenarios:
>>    
>>
>
>yes, the merged patch and that makefile suck.  there should never be any 
>filesystem check in a Makefile anymore.  while peeking in /proc is somewhat 
>forgivable, looking for headers never is ok.  we have autoconf tests now and 
>Gowri's approach looks like the correct one.
>-mike
>
I agree with your explanation. My patch tried to address an issue on the 
testcase when  LTP is cross-compiled.

Thanks so much.
Best Regards
--
FR



[-- Attachment #1.2: Type: text/html, Size: 2099 bytes --]

[-- Attachment #2: Type: text/plain, Size: 390 bytes --]

------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have 
the opportunity to enter the BlackBerry Developer Challenge. See full prize 
details at: http://p.sf.net/sfu/blackberry

[-- Attachment #3: Type: text/plain, Size: 155 bytes --]

_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list