Bug 554261

Summary: Publican not generating correct POTs from English XML files
Product: [Community] Publican Reporter: Ankit Patel <ankit>
Component: publicanAssignee: Jeff Fearn 🐞 <jfearn>
Status: CLOSED CURRENTRELEASE QA Contact: Jeff Fearn 🐞 <jfearn>
Severity: medium Docs Contact:
Priority: low    
Version: 2.0CC: 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.