hi,

> On Mon, Apr 22, 2013 at 10:20:00PM +0900, YAMAMOTO Takashi wrote:
>> execinfo for NetBSD and ubacktrace for uClibc.
>> i don't know if the latter is relevant to Open vSwitch, though.
>> 
>> Signed-off-by: YAMAMOTO Takashi <y...@mwd.biglobe.ne.jp>
> 
> I'm pretty sure this change is going to break (or at least reduce
> functionality of) lib/timeval.c on GNU/Linux because it checks for
> HAVE_EXECINFO_H and I think that this change removes that check.

thanks for reviewing.

it seems HAVE_EXECINFO_H is provided by another execinfo.h check in
configure.ac.  (why having the check in two places?)

my change seems to break the "Defines HAVE_BACKTRACE" part, though.

is there any reason why you check HAVE_EXECINFO_H in lib/timeval.c
and HAVE_BACKTRACE in lib/backtrace.c?

YAMAMOTO Takashi
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to