From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mathieu Desnoyers Subject: Re: [PATCH 3/5] Read CTF by the ctf target Date: Tue, 7 May 2013 09:07:41 -0400 Message-ID: <20130507130741.GE4186@Krystal> References: <1361931459-3953-1-git-send-email-yao@codesourcery.com> <1361931459-3953-4-git-send-email-yao@codesourcery.com> <512F9182.3080104@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Content-Disposition: inline In-Reply-To: To: Hui Zhu Cc: Yao Qi , gdb-patches@sourceware.org, Pedro Alves , lttng-dev List-Id: lttng-dev@lists.lttng.org * Hui Zhu (teawater@gmail.com) wrote: > On Fri, Mar 1, 2013 at 1:18 AM, Pedro Alves wrote: > > (not a complete review) > > > > On 02/27/2013 02:17 AM, Yao Qi wrote: > > > >> +if test "x$with_babeltrace" != "xno"; then > >> + saved_CFLAGS="$CFLAGS" > >> + CFLAGS="$CFLAGS $btinc" > >> + AC_MSG_CHECKING([for babeltrace]) > >> + AC_TRY_COMPILE([ > >> +#include > >> +#include > >> +#include > > > > We know from previous patches that the latest released > > babeltrace breaks gdb, due to the lookup_enum function. Does > > this catch that? I see namespacing fixes going into > > babeltrace as recently as last 15th. Do we know if > > babeltrace's API is more or less done and close to a release? > > Hi Mathieu, > > Could you help us with this question? We did the 1.1.0 release specifically for namespace fixes on March 23rd. commit c99b1910bea848e8f0ae5641bb63b8f4f84f3ec0 Author: Mathieu Desnoyers Date: Sat Mar 23 13:34:22 2013 -0400 Version 1.1.0 Signed-off-by: Mathieu Desnoyers Thanks, Mathieu > > Thanks, > Hui > > > > >> + ],[], > >> + [AC_MSG_RESULT([yes]); AC_DEFINE(HAVE_LIBBABELTRACE, 1, [Define if libbabeltrace is available])], > >> + [AC_MSG_RESULT([no]); btlibs= ; btinc= ]) > >> + CFLAGS="$saved_CFLAGS" > >> +fi > >> + > >> +# Flags needed for UST > > > > babeltrace > > > >> +AC_SUBST(btlibs) > >> +AC_SUBST(btinc) > >> + > > > > -- > > Pedro Alves > > -- Mathieu Desnoyers EfficiOS Inc. http://www.efficios.com