All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Stefan Fröberg" <stefan.froberg@petroprogram.com>
To: buildroot@busybox.net
Subject: [Buildroot] new <pkg>_CONFIG_FIXUP variable
Date: Tue, 08 Jan 2013 00:27:18 +0200	[thread overview]
Message-ID: <50EB4BC6.6020807@petroprogram.com> (raw)
In-Reply-To: <201301072310.10765.yann.morin.1998@free.fr>

Hi Yann

8.1.2013 0:10, Yann E. MORIN kirjoitti:
> Stefan, All,
>
> On Monday 07 January 2013 Stefan Fr?berg wrote:
>> This patch provides new variable called <pkg>_CONFIG_FIXUP.
>> It's purpose is to fix wrong prefixes of $(STAGING_DIR)/usr/bin/*-config file(s)
>> that many library packages install after staging installation phase.
>>
>> Some packages provide multible *-config files.
>>
>> So as suggested by Thomas Petazzoni and Gustavo Zacarias this 
>> variable can be feeded multible files.
>>
>> All filenames are relative to $(STAGING_DIR)/usr/bin
>>
>> Example:
>>
>> DIVINE_CONFIG_FIXUP = divine-config
>>
>> or:
>>
>> IMAGEMAGICK_CONFIG_FIXUP = Magick-config Wand-config
> Please, put this explanation in the commit message.
>
> Also, add this variable to the documentation:
>     docs/manual/adding-packages-generic.txt
>
> Next time, also add a snall changelog to your message, so we know there
> was prior discussion and to help Peter to apply the correct patch series.
> I do something like:
>   Changes v1 -> v2:
>     - accept multiple files (Thomas, Gustavo)
>     - use paths relative to $(STAGING)/usr/bin
>
> And keep this changelog in each iteration. Add the version of the iteration
> in your mail subject.
>
> Thank you! :-)
>
> Regards,
> Yann E. MORIN.
>
God...
It seems that doing documentation for all these things is going to take
more work than the actual patch itself
;-)

Thanks!

Stefan

  reply	other threads:[~2013-01-07 22:27 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-07 22:01 [Buildroot] new <pkg>_CONFIG_FIXUP variable Stefan Fröberg
2013-01-07 22:01 ` [Buildroot] [PATCH] new variable <pkg>_CONFIG_FIXUP Stefan Fröberg
2013-01-07 22:10   ` Thomas Petazzoni
2013-01-07 22:23     ` Stefan Fröberg
2013-01-07 22:42       ` Thomas Petazzoni
2013-01-08 20:53         ` Arnout Vandecappelle
2013-01-08 21:10           ` Thomas Petazzoni
2013-01-08 21:10   ` Arnout Vandecappelle
2013-01-07 22:10 ` [Buildroot] new <pkg>_CONFIG_FIXUP variable Yann E. MORIN
2013-01-07 22:27   ` Stefan Fröberg [this message]
2013-01-08 23:21     ` Yann E. MORIN
2013-01-09  7:54       ` Thomas Petazzoni
2013-01-09 15:21         ` Stefan Fröberg
2013-01-09 20:31           ` Arnout Vandecappelle
2013-01-10  0:16             ` Stefan Fröberg

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=50EB4BC6.6020807@petroprogram.com \
    --to=stefan.froberg@petroprogram.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.