All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v3 0/2] Add gettext-tiny package
Date: Mon, 11 Mar 2019 09:02:35 +0100	[thread overview]
Message-ID: <20190311090235.4e64625b@windsurf> (raw)
In-Reply-To: <20190310230532.4dhczaa4sygw75ie@vkochan-ThinkPad-T470p>

Hello,

On Mon, 11 Mar 2019 01:05:32 +0200
Vadim Kochan <vadim4j@gmail.com> wrote:

> But, it looks like that there should be no problem if NLS is disabled
> (host-gettext is provided by gettext-tiny) and gettext (target)
> (provided by gettext-gnu) are enabled in the same time (of course 1
> small fix is required in latest gettext-tiny series v3):
> 
> in file package/gettext/Config.in
> 
> > config BR2_PACKAGE_GETTEXT
> >          bool "gettext"
> >          select BR2_PACKAGE_HAS_GETTEXT  
> *remove -> depends on BR2_SYSTEM_ENABLE_NLS
> 		    
> remove dependens on NLS to select gettext-gnu for the target.
> 
> So, both host-gettext-tiny and gettext-gnu should coexist as they
> provides different instances, right ?
> 
> If this is ok, then of course ecryptfs-utils needs support the gettext-less
> version, but it might be done later out of this scope ?

I don't really like to do this. It's a lot better if we can avoid
gettext-gnu entirely if NLS is disabled. So I'd prefer if we could
stick to one of the two options I proposed in my reply to your v3:

Option 1: we fix ecryptfs-utils to not unconditionally use gettext.

Option 2: we improve gettext-tiny so that it provides a fake gettext
implementation, and we allow gettext-tiny as a gettext implementation
for the target.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2019-03-11  8:02 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-12 21:55 [Buildroot] [PATCH v3 0/2] Add gettext-tiny package Vadim Kochan
2019-02-12 21:55 ` [Buildroot] [PATCH v3 1/2] package/gettext: Turn into virtual package Vadim Kochan
2019-02-12 21:55 ` [Buildroot] [PATCH v3 2/2] package/gettext-tiny: Add new package Vadim Kochan
2019-03-09 22:37 ` [Buildroot] [PATCH v3 0/2] Add gettext-tiny package Thomas Petazzoni
2019-03-10  0:20   ` Vadim Kochan
2019-03-10  9:11     ` Thomas Petazzoni
2019-03-10 10:18       ` Yann E. MORIN
2019-03-10 23:05         ` Vadim Kochan
2019-03-11  8:02           ` Thomas Petazzoni [this message]
2019-03-11 10:46             ` Vadym Kochan
2019-03-11  8:01         ` Thomas Petazzoni

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=20190311090235.4e64625b@windsurf \
    --to=thomas.petazzoni@bootlin.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.