* Re: [PATCH 3/5] Read CTF by the ctf target [not found] ` <512F9182.3080104@redhat.com> @ 2013-03-01 2:37 ` Hui Zhu 2013-05-07 13:07 ` Mathieu Desnoyers 0 siblings, 1 reply; 4+ messages in thread From: Hui Zhu @ 2013-03-01 2:37 UTC (permalink / raw) To: Mathieu Desnoyers; +Cc: Yao Qi, gdb-patches, Pedro Alves, lttng-dev On Fri, Mar 1, 2013 at 1:18 AM, Pedro Alves <palves@redhat.com> 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 <babeltrace/babeltrace.h> >> +#include <babeltrace/ctf/events.h> >> +#include <babeltrace/ctf/iterator.h> > > 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? 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 > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 3/5] Read CTF by the ctf target 2013-03-01 2:37 ` [PATCH 3/5] Read CTF by the ctf target Hui Zhu @ 2013-05-07 13:07 ` Mathieu Desnoyers 2013-05-07 13:24 ` Yao Qi 0 siblings, 1 reply; 4+ messages in thread From: Mathieu Desnoyers @ 2013-05-07 13:07 UTC (permalink / raw) To: Hui Zhu; +Cc: Yao Qi, gdb-patches, Pedro Alves, lttng-dev * Hui Zhu (teawater@gmail.com) wrote: > On Fri, Mar 1, 2013 at 1:18 AM, Pedro Alves <palves@redhat.com> 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 <babeltrace/babeltrace.h> > >> +#include <babeltrace/ctf/events.h> > >> +#include <babeltrace/ctf/iterator.h> > > > > 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 <mathieu.desnoyers@efficios.com> Date: Sat Mar 23 13:34:22 2013 -0400 Version 1.1.0 Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com> 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 3/5] Read CTF by the ctf target 2013-05-07 13:07 ` Mathieu Desnoyers @ 2013-05-07 13:24 ` Yao Qi 2013-05-07 13:28 ` Mathieu Desnoyers 0 siblings, 1 reply; 4+ messages in thread From: Yao Qi @ 2013-05-07 13:24 UTC (permalink / raw) To: Mathieu Desnoyers; +Cc: Hui Zhu, gdb-patches, Pedro Alves, lttng-dev On 05/07/2013 09:07 PM, Mathieu Desnoyers wrote: > We did the 1.1.0 release specifically for namespace fixes on March 23rd. > > commit c99b1910bea848e8f0ae5641bb63b8f4f84f3ec0 > Author: Mathieu Desnoyers<mathieu.desnoyers@efficios.com> > Date: Sat Mar 23 13:34:22 2013 -0400 > > Version 1.1.0 > > Signed-off-by: Mathieu Desnoyers<mathieu.desnoyers@efficios.com> Mathieu, We noticed this timely release and start to use babeltrace 1.1.0 in GDB. GDB is able read ctf via babeltrace now, just fyi... and thanks for your help on some ctf explanations. -- Yao (齐尧) ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 3/5] Read CTF by the ctf target 2013-05-07 13:24 ` Yao Qi @ 2013-05-07 13:28 ` Mathieu Desnoyers 0 siblings, 0 replies; 4+ messages in thread From: Mathieu Desnoyers @ 2013-05-07 13:28 UTC (permalink / raw) To: Yao Qi; +Cc: Hui Zhu, gdb-patches, Pedro Alves, lttng-dev * Yao Qi (yao@codesourcery.com) wrote: > On 05/07/2013 09:07 PM, Mathieu Desnoyers wrote: >> We did the 1.1.0 release specifically for namespace fixes on March 23rd. >> >> commit c99b1910bea848e8f0ae5641bb63b8f4f84f3ec0 >> Author: Mathieu Desnoyers<mathieu.desnoyers@efficios.com> >> Date: Sat Mar 23 13:34:22 2013 -0400 >> >> Version 1.1.0 >> >> Signed-off-by: Mathieu Desnoyers<mathieu.desnoyers@efficios.com> > > Mathieu, > We noticed this timely release and start to use babeltrace 1.1.0 in GDB. > GDB is able read ctf via babeltrace now, just fyi... and thanks for > your help on some ctf explanations. That's great ! Thank you, Mathieu -- Mathieu Desnoyers EfficiOS Inc. http://www.efficios.com ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-05-07 13:28 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1361931459-3953-1-git-send-email-yao@codesourcery.com>
[not found] ` <1361931459-3953-4-git-send-email-yao@codesourcery.com>
[not found] ` <512F9182.3080104@redhat.com>
2013-03-01 2:37 ` [PATCH 3/5] Read CTF by the ctf target Hui Zhu
2013-05-07 13:07 ` Mathieu Desnoyers
2013-05-07 13:24 ` Yao Qi
2013-05-07 13:28 ` Mathieu Desnoyers
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.