All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: Mohamad Gebai <mohamad.gebai@polymtl.ca>
Cc: Stefan Hajnoczi <stefanha@gmail.com>,
	qemu-devel@nongnu.org, Mohamad Gebai <mohamad.gebai@gmail.com>
Subject: Re: [Qemu-devel] [PATCH v4 1/5] Fix configure script for LTTng 2.x
Date: Tue, 22 Oct 2013 10:03:51 +0100	[thread overview]
Message-ID: <p1hac9isnc.fsf@linaro.org> (raw)
In-Reply-To: <52658D7E.4010609@polymtl.ca>


mohamad.gebai@polymtl.ca writes:

> On 13-10-18 10:05 AM, Stefan Hajnoczi wrote:
<snip>
>> Please switch to pkg-config instead of hardcoding particular flags.  I
>> checked that Fedora and Debian ship the pkg-config file.
> Unfortunately, some missing files in the Ubuntu package make it 
> impossible to use pkg-config for versions of Ubuntu older than 13.10 
> (like Ubuntu 12.04 which is a LTS). Would you be okay with these flags?

Hmm this is tricky because pkg-config really should be the portable way
to get at these flags. Perhaps we could just fall back to sensible
defaults if pkg-config fails?

On my 12.04 system (with the lttng PPA) it works fine:

$>pkg-config --libs lttng-ust
-llttng-ust -ldl  
$>pkg-config --cflags lttng-ust
-I/usr/include/x86_64-linux-gnu  

Is there a bug raised with the Ubuntu upstream to fix their stable LTTNG package?

-- 
Alex Bennée

  reply	other threads:[~2013-10-22  9:04 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-18  6:39 [Qemu-devel] [PATCH v4 0/5] Fix UST backend for LTTng 2.x Mohamad Gebai
2013-10-18  6:39 ` [Qemu-devel] [PATCH v4 1/5] Fix configure script " Mohamad Gebai
2013-10-18 14:05   ` Stefan Hajnoczi
2013-10-21 20:24     ` Mohamad Gebai
2013-10-22  9:03       ` Alex Bennée [this message]
2013-10-18  6:39 ` [Qemu-devel] [PATCH v4 2/5] Modified the tracetool framework " Mohamad Gebai
2013-10-18  6:39 ` [Qemu-devel] [PATCH v4 3/5] Adapt Makefiles to the new LTTng ust interface Mohamad Gebai
2013-10-18 14:10   ` Stefan Hajnoczi
2013-10-18  6:39 ` [Qemu-devel] [PATCH v4 4/5] Update documentation for LTTng ust tracing Mohamad Gebai
2013-10-18 14:02   ` Stefan Hajnoczi
2013-10-18  6:39 ` [Qemu-devel] [PATCH v4 5/5] Add ust generated files to .gitignore Mohamad Gebai
2013-10-18  8:37 ` [Qemu-devel] [PATCH v4 0/5] Fix UST backend for LTTng 2.x Alex Bennée

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=p1hac9isnc.fsf@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=mohamad.gebai@gmail.com \
    --cc=mohamad.gebai@polymtl.ca \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.