mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Alexander Aring <alex.aring@gmail.com>
To: ptxdist@pengutronix.de
Cc: Alexander Aring <alex.aring@gmail.com>
Subject: [ptxdist] [PATCH] powertop: disable native language support
Date: Mon,  7 Apr 2014 16:08:29 +0200	[thread overview]
Message-ID: <1396879709-14187-1-git-send-email-alex.aring@gmail.com> (raw)

This patch should fix a gettext issue and disable native language
support.

Original notice from powertop-2.5 "ABOUT-NLS" file:

"By default, this package will be installed to allow translation of
messages.  It will automatically detect whether the system already
provides the GNU `gettext' functions.  If not, the included GNU
`gettext' library will be used.  This library is wholly contained
within this package, usually in the `intl/' subdirectory, so prior
installation of the GNU `gettext' package is _not_ required.
Installers may use special options at configuration time for changing
the default behaviour.  The commands:

     ./configure --with-included-gettext
     ./configure --disable-nls

will, respectively, bypass any pre-existing `gettext' to use the
internationalizing routines provided within this package, or else,
_totally_ disable translation of messages."

This patch disable the native language support. We don't need such
support for translate messsages in another languages. The
documententation is somewhat outdated because there is no
"--with-included-gettext" option there. I hope --disable-nls should do
that also, then we don't need gettext anymore.

Also added arguments for "--without-libintl-prefix" and
"--without-libiconv-prefix". Making no sense with --disable-nls, but I saw
that in other rules as well.

Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Reported-by: Jürgen Beisert <jbe@pengutronix.de>
---
 rules/powertop.make | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/rules/powertop.make b/rules/powertop.make
index 36bc52a..6c376e0 100644
--- a/rules/powertop.make
+++ b/rules/powertop.make
@@ -36,7 +36,10 @@ POWERTOP_LICENSE	:= GPLv2
 POWERTOP_CONF_TOOL	:= autoconf
 POWERTOP_CONF_OPT	:= \
 	$(CROSS_AUTOCONF_USR) \
-	--$(call ptx/endis, PTXCONF_POWERTOP_PCI_SUPPORT)-pci
+	--$(call ptx/endis, PTXCONF_POWERTOP_PCI_SUPPORT)-pci \
+	--disable-nls \
+	--without-libiconv-prefix \
+	--without-libintl-prefix
 
 # ----------------------------------------------------------------------------
 # Target-Install
-- 
1.9.1


-- 
ptxdist mailing list
ptxdist@pengutronix.de

             reply	other threads:[~2014-04-07 14:08 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-07 14:08 Alexander Aring [this message]
2014-04-10  7:45 ` Michael Olbrich

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=1396879709-14187-1-git-send-email-alex.aring@gmail.com \
    --to=alex.aring@gmail.com \
    --cc=ptxdist@pengutronix.de \
    /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