From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] divine: fix divine-config
Date: Fri, 4 Jan 2013 17:29:45 +0100 [thread overview]
Message-ID: <20130104172945.50d8efd8@skate> (raw)
In-Reply-To: <CAHXCMMKhDpanjesHYjMSh-dcAHKt-84B3gEgyzs6UgN_8VRZvQ@mail.gmail.com>
Dear Samuel Martin,
On Fri, 4 Jan 2013 15:17:42 +0100, Samuel Martin wrote:
> We quickly talked about this during the last Buildroot Dev. Days.
>
> IIRC, Thomas and Peter were not very fond of this kind of solution.
> (guys, feel free to correct me if I speak with my ass ;)).
I might remember wrong, but I think the solution I did not like was
blindly patching *ALL* the files matching *-config.
The solution proposed by Gustavo, where the foo-config files to be
patched are explicitly listed in a package variable sounds fine to me,
as long as the common thing is only to patch prefix and exec_prefix,
the other more specialized fixes (if any) have to be left to the
package specific .mk file.
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2013-01-04 16:29 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-03 22:09 [Buildroot] [PATCH] divine: fix divine-config Stefan Fröberg
2013-01-03 22:09 ` [Buildroot] [PATCH] giblib: fix giblib-config Stefan Fröberg
2013-01-03 22:09 ` [Buildroot] [PATCH] imlib2: fix imlib2-config Stefan Fröberg
2013-01-03 22:09 ` [Buildroot] [PATCH] libgcrypt: fix libgcrypt-config Stefan Fröberg
2013-01-03 22:09 ` [Buildroot] [PATCH] imagemagick: fix Magick[++|Core|Wand]-config Stefan Fröberg
2013-01-04 21:45 ` Peter Korsgaard
2013-01-03 22:09 ` [Buildroot] [PATCH] neon: fix neon-config Stefan Fröberg
2013-01-03 22:09 ` [Buildroot] [PATCH] libnspr: fix nspr-config Stefan Fröberg
2013-01-03 22:09 ` [Buildroot] [PATCH] xlib_libXft: fix xft-config Stefan Fröberg
2013-01-04 23:30 ` Peter Korsgaard
2013-01-05 13:33 ` Stefan Fröberg
2013-01-04 11:41 ` [Buildroot] [PATCH] divine: fix divine-config Gustavo Zacarias
2013-01-04 14:17 ` Samuel Martin
2013-01-04 14:29 ` Gustavo Zacarias
2013-01-04 14:56 ` Stefan Fröberg
2013-01-04 15:00 ` Gustavo Zacarias
2013-01-06 5:41 ` Baruch Siach
2013-01-06 8:27 ` Samuel Martin
2013-01-06 8:34 ` Baruch Siach
2013-01-06 12:39 ` Thomas Petazzoni
2013-01-06 14:15 ` Stefan Fröberg
2013-01-06 18:12 ` Stefan Fröberg
2013-01-07 22:17 ` Arnout Vandecappelle
2013-01-07 22:35 ` Stefan Fröberg
2013-01-08 7:48 ` Arnout Vandecappelle
2013-01-08 8:10 ` Thomas Petazzoni
2013-01-08 8:24 ` Arnout Vandecappelle
2013-01-04 16:29 ` Thomas Petazzoni [this message]
2013-01-04 16:31 ` Thomas Petazzoni
2013-01-04 17:10 ` Gustavo Zacarias
2013-01-04 17:13 ` Stefan Fröberg
2013-01-04 21:49 ` 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=20130104172945.50d8efd8@skate \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox