Bug 554261

Summary:	Publican not generating correct POTs from English XML files
Product:	[Community] Publican	Reporter:	Ankit Patel <ankit>
Component:	publican	Assignee:	Jeff Fearn 🐞 <jfearn>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Jeff Fearn 🐞 <jfearn>
Severity:	medium	Docs Contact:
Priority:	low
Version:	2.0	CC:	ccheng, mhideo, mmcallis, mospina, publican-list
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	1.4	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2010-03-01 00:19:28 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Ankit Patel 2010-01-11 06:56:11 UTC

Description of problem:

I have checked out Installation Guide for RHEL6 (from https://svn.devel.redhat.com/repos/ecs/Red_Hat_Enterprise_Linux/6.0/Installation_Guide) and tried to update the pot with "publican update_pot" command. It did create POT files, but noticed that there are many files with no messages for translation while there are messages in English XML files for it. 

e.g. compare below
* POT file - https://svn.devel.redhat.com/repos/ecs/Red_Hat_Enterprise_Linux/6.0/Installation_Guide/pot/X86_Uninstall-Windows-bootloader-para-1.pot
* English file - https://svn.devel.redhat.com/repos/ecs/Red_Hat_Enterprise_Linux/6.0/Installation_Guide/en-US/X86_Uninstall-Windows-bootloader-para-1.xml

* POT file - https://svn.devel.redhat.com/repos/ecs/Red_Hat_Enterprise_Linux/6.0/Installation_Guide/pot/X86_Uninstall-Windows-bootloader-para-2.pot
* English file - https://svn.devel.redhat.com/repos/ecs/Red_Hat_Enterprise_Linux/6.0/Installation_Guide/en-US/X86_Uninstall-Windows-bootloader-para-2.xml

* POT file - https://svn.devel.redhat.com/repos/ecs/Red_Hat_Enterprise_Linux/6.0/Installation_Guide/pot/X86_Uninstall-Windows-bootloader-para-3.pot
* English file - https://svn.devel.redhat.com/repos/ecs/Red_Hat_Enterprise_Linux/6.0/Installation_Guide/en-US/X86_Uninstall-Windows-bootloader-para-3.xml

There is a long list of such files.

Version-Release number of selected component (if applicable):
[ankit@ankit Installation_Guide]$ publican -v
version=1.3
[ankit@ankit Installation_Guide]$ rpm -q publican
publican-1.3-0.el5
[ankit@ankit Installation_Guide]$ cat /etc/redhat-release 
Red Hat Enterprise Linux Client release 5.4 (Tikanga)
[ankit@ankit Installation_Guide]$ 


How reproducible:
Everytime

Steps to Reproduce:
1. Check out RHEL6 Installation Guide
2. Run "publican update_pot"
3. And observe the POT files
  
Actual results:
You will notice many of the POT files have no message marked for translation

Expected results:
POT file should have messages for translation

Additional info:

Comment 1 Ruediger Landmann 2010-01-15 04:32:52 UTC

This only appears to happen when the root node of an XML file is <para> or <title>.

Comment 2 Jeff Fearn 🐞 2010-01-18 05:51:31 UTC

Changed logic in message parsing code to process root XML nodes.

workaround: stop breaking things down so tiny Mr Landmann ;)

Comment 3 Ankit Patel 2010-01-18 08:53:55 UTC

(In reply to comment #2)
> Changed logic in message parsing code to process root XML nodes.
> 
> workaround: stop breaking things down so tiny Mr Landmann ;)    

Thanks for solving the issue Jeff.

Could you please let's know when the new version (1.4) is out?

Thanks!
Ankit

Comment 4 Fedora Update System 2010-01-29 02:05:29 UTC

publican-1.4-1.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/publican-1.4-1.fc12

Comment 5 Fedora Update System 2010-01-29 02:06:46 UTC

publican-1.4-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/publican-1.4-1.fc11

Comment 6 Fedora Update System 2010-02-01 01:13:36 UTC

publican-1.4-1.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 7 Fedora Update System 2010-02-01 01:21:57 UTC

publican-1.4-1.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 8 Fedora Update System 2010-02-26 05:24:08 UTC

publican-1.5-0.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/publican-1.5-0.fc12

Comment 9 Fedora Update System 2010-02-26 05:27:45 UTC

publican-1.5-0.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/publican-1.5-0.fc13

Comment 10 Fedora Update System 2010-02-26 05:27:57 UTC

publican-1.5-0.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/publican-1.5-0.fc11

Comment 11 Fedora Update System 2010-02-26 05:28:04 UTC

publican-1.5-0.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/publican-1.5-0.fc12

Comment 12 Fedora Update System 2010-02-26 05:30:15 UTC

publican-1.5-0.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/publican-1.5-0.fc13

Comment 13 Fedora Update System 2010-02-26 05:34:20 UTC

publican-1.5-0.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/publican-1.5-0.fc11

Comment 14 Fedora Update System 2010-02-27 03:28:56 UTC

publican-1.5-0.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2010-02-27 03:44:45 UTC

publican-1.5-0.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.