Re: lttng-modules 2.6.1 compilation error on Ubuntu 14.04

[Engel Sanchez <engel@basho.com>]

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

Thanks Mathieu! In the meantime I'm using 2.6.0 for the moment and will try
your fix when it's ready.

On Thu, May 14, 2015 at 11:28 AM, Mathieu Desnoyers <
mathieu.desnoyers@efficios.com> wrote:

> ----- Original Message -----
> > ----- Original Message -----
> >
> > > Hello there. I was wondering if anybody knew what is causing this
> > > compilation
> > > error I'm getting on a new Ubuntu 14.04 VM with a 3.13.0 kernel. It
> does
> > > not
> > > find the DEBIAN_API_VERSION symbol. The use of that was introduced in
> 2.6.1
> > > apparently:
> >
> > >
> http://fossies.org/diffs/lttng-modules/2.6.0_vs_2.6.1/lttng-kernel-version.h-diff.html
> >
> > > I'm following the instructions in the Download page to install from
> the PPA
> > > :
> > > http://lttng.org/download/#ubuntu
> >
> > > This the error I'm getting. Thanks in advance for any help!
> >
> > > In file included from
> > >
> /var/lib/dkms/lttng-modules/2.6.x+stable+bzr727+pack22+201504291801~ubuntu14.04.1/build/wrapper/page_alloc.h:28:0,
> > >     from
> > >
> /var/lib/dkms/lttng-modules/2.6.x+stable+bzr727+pack22+201504291801~ubuntu14.04.1/build/lttng-events.c:27:
> > >
>  /var/lib/dkms/lttng-modules/2.6.x+stable+bzr727+pack22+201504291801~ubuntu14.04.1/build/wrapper/../lttng-kernel-version.h:60:57:
> > > error: operator '+' has no right operand    ((LINUX_VERSION_CODE *
> > > 1000000ULL)
> > > + DEBIAN_API_VERSION)
> >
> > I pushed the following fix into master and stable-2.6 branches
> > of lttng-modules. Can you give it a try ?
> >
> > commit a748963b76f353d459a3faf85eecc96f52811eee
> > Author: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> > Date:   Thu May 14 09:13:49 2015 -0400
> >
> >     Fix: build against Ubuntu kernels
> >
> >     Recent Ubuntu kernels appear to define DEBIAN_API_VERSION, but they
> are
> >     not really a Debian kernel per-se: the abi-debian-version.sh cannot
> find
> >     the expected Debian kernel version string layout.
> >
> >     Therefore, change the conditional check to use directly the
> >     LTTNG_DEBIAN_VERSION_CODE define, which is defined only when the
> >     abi-debian-version.sh script figures out it builds against a Debian
> >     kernel.
> >
> >     Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
>
> I reverted this commit. DEBIAN_API_VERSION is actually
> the define I use to communicate between the makefile
> and the C code. It should work even if not defined
> (see the check in LTTNG_DEBIAN_KERNEL_RANGE()).
>
> Hrm. But it appears that the operators generated by
> those macros end up being invalid if DEBIAN_API_VERSION
> is undefined. Will fix and push upstream.
>
> Thanks,
>
> Mathieu
>
> >
> > Thanks,
> >
> > Mathieu
> >
> > > _______________________________________________
> > > lttng-dev mailing list
> > > lttng-dev@lists.lttng.org
> > > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
> >
> > --
> >
> > --
> > Mathieu Desnoyers
> > EfficiOS Inc.
> > http://www.efficios.com
> >
> > _______________________________________________
> > lttng-dev mailing list
> > lttng-dev@lists.lttng.org
> > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
> >
>
> --
> Mathieu Desnoyers
> EfficiOS Inc.
> http://www.efficios.com
>

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

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

_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev