From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 02/36] package/sdl: add symlink to sdl-config
Date: Wed, 29 Aug 2012 01:09:18 +0200 [thread overview]
Message-ID: <503D4F9E.6070803@mind.be> (raw)
In-Reply-To: <20120813125710.13d0fc69@skate>
On 08/13/12 12:57, Thomas Petazzoni wrote:
> Le Mon, 13 Aug 2012 01:53:50 +0200,
> "Yann E. MORIN"<yann.morin.1998@free.fr> a ?crit :
>
>> > SDL installs a script-based sdl-config in the sysroot, but some
>> > packages (eg. QEMU) needs it to be either:
>> > - prefixed with the cross-prefix
>> > - in the PATH
>> >
>> > So, as sdl-config is a script, and although it is installed in
>> > the sysroot, it can be run on the host.
> Well, that's typically not how we handle this problem. In general, we
> pass the complete location of the<foo>-config script to
> the ./configure script of the package that needs it. I would prefer if
> we could use this solution everywhere instead of having a different
> solution in different places for the same problem.
Although I fully agree with using the same pattern everywhere, I think the
pattern is wrong here. Having a xxx-config script is supposed to make it
easy to discover a package; if you have to point to it explicitly in the
package.mk using it, that kind of defeats the purpose.
Of course, even so it is still shaky. If the xxx-config has to be looked up
from the PATH, we risk picking up /usr/bin/xxx-config... In an ideal world,
they'd all be using pkg-config... But in an ideal work we probably wouldn't
even need buildroot :-)
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286540
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
next prev parent reply other threads:[~2012-08-28 23:09 UTC|newest]
Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-12 23:53 [Buildroot] [PATCH 01/36] system/init: add option for no init system Yann E. MORIN
2012-08-12 23:53 ` [Buildroot] [PATCH 02/36] package/sdl: add symlink to sdl-config Yann E. MORIN
2012-08-13 10:57 ` Thomas Petazzoni
2012-08-13 11:32 ` Yann E. MORIN
2012-08-28 23:09 ` Arnout Vandecappelle [this message]
2012-08-12 23:53 ` [Buildroot] [PATCH 03/36] package/libfdt: new package Yann E. MORIN
2012-08-14 13:23 ` Thomas Petazzoni
2012-08-14 16:51 ` Yann E. MORIN
2012-08-12 23:53 ` [Buildroot] [PATCH 04/36] package/vde2: " Yann E. MORIN
2012-08-14 13:24 ` Thomas Petazzoni
2012-08-14 16:57 ` Yann E. MORIN
2012-08-12 23:53 ` [Buildroot] [PATCH 05/36] package/libiscsi: " Yann E. MORIN
2012-08-17 11:14 ` Thomas Petazzoni
2012-08-17 12:29 ` Samuel Martin
2012-08-17 12:42 ` Thomas Petazzoni
2012-08-12 23:53 ` [Buildroot] [PATCH 06/36] package/usbredir: " Yann E. MORIN
2012-08-17 11:15 ` Thomas Petazzoni
2012-08-17 13:37 ` Yann E. MORIN
2012-08-17 13:48 ` Thomas Petazzoni
2012-08-12 23:53 ` [Buildroot] [PATCH 07/36] package/celt051: " Yann E. MORIN
2012-08-17 11:18 ` Thomas Petazzoni
2012-08-17 13:27 ` Yann E. MORIN
2012-08-17 13:50 ` Thomas Petazzoni
2012-08-17 14:35 ` Yann E. MORIN
2012-08-12 23:53 ` [Buildroot] [PATCH 08/36] package/python-pyparsing: " Yann E. MORIN
2012-08-17 11:19 ` Thomas Petazzoni
2012-08-12 23:53 ` [Buildroot] [PATCH 09/36] package/spice-protocol: " Yann E. MORIN
2012-08-17 11:20 ` Thomas Petazzoni
2012-08-12 23:53 ` [Buildroot] [PATCH 10/36] package/spice-server: " Yann E. MORIN
2012-08-17 11:22 ` Thomas Petazzoni
2012-08-17 13:38 ` Yann E. MORIN
2012-08-12 23:53 ` [Buildroot] [PATCH 11/36] package/qemu: " Yann E. MORIN
2012-08-13 6:52 ` Yann E. MORIN
2012-08-13 7:22 ` Diego Iastrubni
2012-08-13 7:46 ` Yann E. MORIN
2012-08-14 13:34 ` Thomas Petazzoni
2012-08-14 17:03 ` Yann E. MORIN
2012-08-17 11:26 ` Thomas Petazzoni
2012-08-17 13:32 ` Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 12/36] package/qemu: add basic target selection Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 13/36] package/qemu: add fine-grained " Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 14/36] package/qemu: add SDL frontends Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 15/36] package/qemu: add option to enable/disable the VNC frontend Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 16/36] package/qemu: add VNC jpeg and png compression Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 17/36] package/qemu: add VNC TLS-encryption Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 18/36] package/qemu: add option to not install blobs Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 19/36] package/qemu: add option to remove unwanted keymaps Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 20/36] package/qemu: add uuid support Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 21/36] package/qemu: add support for capabilities Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 22/36] package/qemu: add attr/xattr option Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 23/36] package/qemu: add support for virtfs Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 24/36] package/qemu: add support for cURL Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 25/36] package/qemu: enable use of the curses frontend Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 26/36] package/qemu: add BlueZ connectivity Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 27/36] package/qemu: add AIO support Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 28/36] package/qemu: add support for FDT Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 29/36] package/qemu: add support for VDE switches Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 30/36] package/qemu: add iSCSI support Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 31/36] package/qemu: add support for USB redirection Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 32/36] package/qemu: add support for Spice Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 33/36] package/qemu: option to build the docs Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 34/36] package/qemu: enable guest-base Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 35/36] package/qemu: move sub-options into a sub-menu Yann E. MORIN
2012-08-12 23:54 ` [Buildroot] [PATCH 36/36] package/qemu: enable a static build Yann E. MORIN
2012-08-13 0:00 ` [Buildroot] [PATCH 01/36] system/init: add option for no init system Yann E. MORIN
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=503D4F9E.6070803@mind.be \
--to=arnout@mind.be \
--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.