All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [git commit master 1/1] pkg-config: add --with-sysroot option for compiled in default sysroot setting
Date: Sun, 30 Jan 2011 15:23:58 +0100	[thread overview]
Message-ID: <20110130152358.436fc691@surf> (raw)
In-Reply-To: <20110129234038.9CB708E076@busybox.osuosl.org>

Hello,

On Sun, 30 Jan 2011 00:40:13 +0100
Peter Korsgaard <jacmet@sunsite.dk> wrote:

> Similar to the --with-pc-path option. It works just like the existing
> PKG_CONFIG_SYSROOT_DIR environment variable, but compiled in.
> The environment variable overrides this default setting if set.
> 
> This way we don't need to pass PKG_CONFIG_SYSROOT_DIR in the environment
> when building for the target, and it is easier to reuse pkg-config outside
> BR (E.G. for the SDK) without having to setup special environment
> variables.

I am not a big fan of this: we compile a single pkg-config to handle
both target and host packages. So I don't see why this pkg-config
should be specifically configured to handle packages and not for host
packages. I know it was already configured specifically for target
packages with --with-pc-path, but I wasn't a fan of this.

I think I'd prefer if we didn't pass any --with-sysroot nor
--with-pc-path, and instead use the proper environment variables all
the time. Or either build two pkg-config, one for host packages, one
for target packages.

Regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

  reply	other threads:[~2011-01-30 14:23 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-29 23:40 [Buildroot] [git commit master 1/1] pkg-config: add --with-sysroot option for compiled in default sysroot setting Peter Korsgaard
2011-01-30 14:23 ` Thomas Petazzoni [this message]
2011-01-30 19:02   ` Peter Korsgaard

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=20110130152358.436fc691@surf \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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.