All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gustavo Zacarias <gustavo@zacarias.com.ar>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 2/3] infra: add gettextize run in AUTORECONF
Date: Tue, 15 Jul 2014 14:25:00 -0300	[thread overview]
Message-ID: <53C563EC.6090904@zacarias.com.ar> (raw)
In-Reply-To: <53C562F6.8080705@zacarias.com.ar>

On 07/15/2014 02:20 PM, Gustavo Zacarias wrote:

> On 07/15/2014 02:15 PM, Thomas Petazzoni wrote:
> 
>> Except that trying to detect if it's required will still make the
>> dependency on host-gettext mandatory.
>>
>> What would be good to have is <pkg>_GETTEXTIZE + a check that verifies
>> whether the gettextize is still needed or not. This way, on the
>> packages on which we'll be adding <pkg>_GETTEXTIZE = YES, we will
>> notice when the <pkg>_GETTEXTIZE = YES can be remove after an upstream
>> bump.
> 
> Detecting if it's required via gettext.m4 says if it's using the infra,
> but you must do a grep for the serial usually in $(@D)/m4/gettext.m4 to
> know if it's a different version.
> If we do a silent warning it will probably pass unnoticed, making it
> fatal would be best probably - though we never did that kind of
> behaviour before it would only affect bumps.
> Regards.

And i forgot to cc@ the list again, broke it when adding Yann before :)
Looking at the source upstream had the great idea of NOT updating the
serial so that won't work, sigh...
Regards.

  parent reply	other threads:[~2014-07-15 17:25 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-12 17:27 [Buildroot] [PATCH 1/3] gettext: add gettextize definition Gustavo Zacarias
2014-07-12 17:27 ` [Buildroot] [PATCH 2/3] infra: add gettextize run in AUTORECONF Gustavo Zacarias
2014-07-14 11:01   ` Thomas Petazzoni
2014-07-14 11:04     ` Gustavo Zacarias
2014-07-14 11:07     ` Gustavo Zacarias
2014-07-14 12:30     ` Yann E. MORIN
2014-07-14 12:40       ` Gustavo Zacarias
2014-07-14 12:47         ` Yann E. MORIN
2014-07-14 12:51           ` Gustavo Zacarias
2014-07-14 16:30     ` Arnout Vandecappelle
2014-07-14 19:23       ` Thomas Petazzoni
2014-07-15 13:44         ` Gustavo Zacarias
     [not found]         ` <53C52F0D.70004@zacarias.com.ar>
     [not found]           ` <20140715191515.160512e4@free-electrons.com>
     [not found]             ` <53C562F6.8080705@zacarias.com.ar>
2014-07-15 17:25               ` Gustavo Zacarias [this message]
2014-07-12 17:27 ` [Buildroot] [PATCH 3/3] wget: remove redundant gettextize run Gustavo Zacarias
2014-07-13  3:26   ` Baruch Siach

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=53C563EC.6090904@zacarias.com.ar \
    --to=gustavo@zacarias.com.ar \
    --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.