Bug 661569

Summary:	[l10n] [RFE] Replace msgmerge
Product:	[Community] Publican	Reporter:	Jeff Fearn 🐞 <jfearn>
Component:	publican	Assignee:	Jeff Fearn 🐞 <jfearn>
Status:	CLOSED CURRENTRELEASE	QA Contact:	tools-bugs <tools-bugs>
Severity:	low	Docs Contact:
Priority:	low
Version:	2.5	CC:	mhideo, mmcallis, publican-list, rlandman
Target Milestone:	3.0
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	3.0.0	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-10-31 03:11:20 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	820023

Description Jeff Fearn 🐞 2010-12-09 01:09:12 UTC

Description of problem:
To reduce external deps look in to replacing msgmerge with a perl module.

Doing this should also make it easier to support non PO translations, xliff, etc.

Version-Release number of selected component (if applicable):
2.5

Additional info:

CPAN Modules that may assist with this

Text::Levenshtein
Text::LevenshteinXS
Text::JaroWinkler
Text::WagnerFischer
Text::Brew
String::Similarity

Comment 1 Jeff Fearn 🐞 2011-02-17 06:00:26 UTC

I have checked in a patch to trunk that enables using String::Similarity when merging POT changes. --TEST_PO_MERGE has been added to allow testing the new code path:

$ publican update_po --langs de-DE --TEST_PO_MERGE

Needs some serious testing.

Comment 2 Jeff Fearn 🐞 2011-04-18 05:44:33 UTC

This behaviour has been changed.

By default the new code path is used, and the deps on msgmerge have been removed. To use msgmerge it must be manually installed, if missing, and --msgmerg must be used to use that instead of the new perl code.

Comment 3 Jared MORGAN 2012-05-08 23:18:47 UTC

Documentation required for this new command-line flag. Any questions about the new flag, ask in this ticket. Publican 3.0 umbrella ticket added as a dependency of this issue.

Comment 4 Michael Hideo 2012-06-08 02:07:45 UTC

create vm, ensure gettext tools are not present,  install publican 3, build translated book, verify that the book builds w/o gettext.

Comment 5 Ruediger Landmann 2012-06-16 02:16:00 UTC

Verified on publican-3.0-0.fc17.t180.noarch

Ran rpm -e --nodeps gettext

could still build a translated book; PO entries appeared to merge correctly.