public inbox for util-linux@vger.kernel.org
 help / color / mirror / Atom feed
From: Benno Schulenberg <bensberg@justemail.net>
To: "Util-Linux" <util-linux@vger.kernel.org>
Subject: [attempt at patch] adding --no-wrap to msgmerge
Date: Tue, 17 Dec 2013 11:02:14 +0100	[thread overview]
Message-ID: <1387274534.32475.60577705.14EC62A9@webmail.messagingengine.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 782 bytes --]


Hi,

Attached is a patch that should add the option --no-wrap to the runs
of msgmerge (after a renewal of the POT file) to reduce the number
of spurious differences.

However, I can't get the thing to work here.  When i run autogen.sh
it says "Copying file po/Makefile.in.in", and when I then look in that
file it mentions "# Origin: gettext-0.14".  But the only Makefile.in.in
I have on this testing system is /usr/share/gettext/po/Makefile.in.in
and in that file it says "# Origin: gettext-0.18".  (That version is needed,
otherwise it doesn't know the MSGMERGE_OPTIONS variable.)

So... where does autogen.sh/autopoint take its Makefile.in.in from?

Regards,

Benno

-- 
http://www.fastmail.fm - Or how I learned to stop worrying and
                          love email again


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-po-add-no-wrap-to-msgmerge-to-not-wrap-long-message-.patch --]
[-- Type: text/x-patch; name="0001-po-add-no-wrap-to-msgmerge-to-not-wrap-long-message-.patch", Size: 1057 bytes --]

From 4d9a629b2d3a9e1768eb268acaf9e73fb169978f Mon Sep 17 00:00:00 2001
From: Benno Schulenberg <bensberg@justemail.net>
Date: Mon, 16 Dec 2013 22:41:56 +0100
Subject: [PATCH] po: add '--no-wrap' to msgmerge, to not wrap long message lines

This avoids unneeded churn during POT-file renewal and PO-file updates,
as the PO files at the TP are made with '--no-wrap'.

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>
---
 po/Makevars |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/po/Makevars b/po/Makevars
index 9938e44..4d5629c 100644
--- a/po/Makevars
+++ b/po/Makevars
@@ -10,6 +10,9 @@ top_builddir = ..
 # These options get passed to xgettext.
 XGETTEXT_OPTIONS = --keyword=_ --keyword=N_ --keyword=P_:1,2
 
+# This option gets passed to msgmerge.
+MSGMERGE_OPTIONS = --no-wrap
+
 # This is the copyright holder that gets inserted into the header of the
 # $(DOMAIN).pot file.  Set this to the copyright holder of the surrounding
 # package.  (Note that the msgstr strings, extracted from the package's
-- 
1.7.0.4


             reply	other threads:[~2013-12-17 10:02 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-17 10:02 Benno Schulenberg [this message]
2013-12-17 17:16 ` [attempt at patch] adding --no-wrap to msgmerge Sami Kerola
2013-12-18 22:47   ` [patch] " Benno Schulenberg
2013-12-19 10:30     ` Sami Kerola
2014-01-06  7:49     ` Karel Zak

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=1387274534.32475.60577705.14EC62A9@webmail.messagingengine.com \
    --to=bensberg@justemail.net \
    --cc=util-linux@vger.kernel.org \
    /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