All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] package/gettext: Turn into virtual package
Date: Fri, 4 Jan 2019 09:39:38 +0100	[thread overview]
Message-ID: <20190104093938.00fb138b@windsurf> (raw)
In-Reply-To: <CAMw6YJLvioTwKbgAdYvtZqbVg1W-RAADT_umZkGMaw9MnVpHMw@mail.gmail.com>

Hello,

Please keep the mailing list in Cc, i.e use "Reply All" when replying
to mailing list e-mails. Thanks!

I'm also adding Andr? and Yann in Cc, since they are involved in the
host-gettext dependency in wine.

On Fri, 4 Jan 2019 03:49:49 +0200, Vadim Kochan wrote:

> So the wine builds OK w/o host-gettext (tested with dockerfile), but
> this libgettextpo is required by (optionally)
> 'tools/wrc' which allows to convert window's resources into *.po files
> (https://linux.die.net/man/1/wrc). And
> looks like it might be needed on the host only if NLS=y.

The dependency on host-gettext was added in commit
164ac197fef763dfa61154ad41588b6701baca86, which says:

""
    wine: Add gettext dependency for host-wine
    
    Mostly based on a comment by Yann E. MORIN
""

I am not sure what "comment" Yann made about this, and especially if
the dependency really needs to be mandatory.

Andr?, Yann, do you remember ? Could you give more details about the
host-gettext dependency in wine ?

Do we need it unconditionally, or only when BR2_SYSTEM_ENABLE_NLS=y ?

Best regards,

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

  parent reply	other threads:[~2019-01-04  8:39 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-23 15:04 [Buildroot] [PATCH 0/2] Add gettext-tiny package Vadim Kochan
2018-12-23 15:04 ` [Buildroot] [PATCH 1/2] package/gettext: Turn into virtual package Vadim Kochan
2018-12-23 15:36   ` Yann E. MORIN
2018-12-30 17:10   ` Thomas Petazzoni
2018-12-31  4:03     ` Vadim Kochan
2018-12-31 15:06       ` Thomas Petazzoni
2019-01-03 14:28         ` Vadim Kochan
2019-01-03 14:28           ` Thomas Petazzoni
     [not found]             ` <CAMw6YJ+xU9GiZ809dCQTmPEgtpu3iCOErT5w7fqoee09R-hpaQ@mail.gmail.com>
     [not found]               ` <CAMw6YJLvioTwKbgAdYvtZqbVg1W-RAADT_umZkGMaw9MnVpHMw@mail.gmail.com>
2019-01-04  8:39                 ` Thomas Petazzoni [this message]
2018-12-23 15:04 ` [Buildroot] [PATCH 2/2] package/gettext-tiny: Add new package Vadim Kochan
2018-12-23 21:21   ` Yann E. MORIN
2018-12-31  4:07     ` Vadim Kochan

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=20190104093938.00fb138b@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.