Bug 568201

Summary: publican doesn't replace some of the translated entries.
Product: [Community] Publican Reporter: Manuel Ospina <mospina>
Component: publicanAssignee: Jeff Fearn 🐞 <jfearn>
Status: CLOSED ERRATA QA Contact: Joshua Wulf <jwulf>
Severity: medium Docs Contact:
Priority: medium    
Version: 2.0CC: jfearn, lcarlon, mmcallis, publican-list, yshao
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 1.6.1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-03-24 00:53:22 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:

Description Manuel Ospina 2010-02-25 00:09:12 UTC
Description of problem:

When building HTML/HTML-Single or PDF, publican doesn't replace some of the entries that has been translated in the PO file. 

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


How reproducible: Always


Steps to Reproduce:
1. publican build --formats html-single --langs it-IT
2. check with firefox
  
Actual results:
Several entries are still in English in the HTML version. 

An example from the Installation Guide:

--------------------------
A .parm file is still required for the real kernel parameters, such as root=/dev/ram0 ro ip=off ramdisk_size=40000, and single parameters which are not assigned to variables, such as vnc. Two parameters which are used in z/VM installs to point the installation program at the new CMS configuration file need to be added to the .parm file: 
--------------------------

The entry in the PO file (it-IT/S390_Steps_Vm.po) is correct and properly translated:

---------------------------
#. Tag: para
#, no-c-format
msgid ""
"A <filename>.parm</filename> file is still required for the <emphasis>real</"
"emphasis> kernel parameters, such as <parameter>root=/dev/ram0 ro ip=off "
"ramdisk_size=40000</parameter>, and single parameters which are not assigned "
"to variables, such as <parameter>vnc</parameter>. Two parameters which are "
"used in z/VM installs to point the installation program at the new CMS "
"configuration file need to be added to the <filename>.parm</filename> file:"
msgstr ""
"Il file <filename>.parm</filename> è ancora necessario per i parametri del "
"kernel <emphasis>real</emphasis>, come ad esempio <parameter>root=/dev/ram0 "
"ro ip=off ramdisk_size=40000</parameter>, ed i parametri singoli non ancora "
"assegnati alle variabili, come <parameter>vnc</parameter>. È altresì "
"necessario aggiungere al file <filename>.parm</filename>, due parametri "
"utilizzati nelle installazioni z/VM per indicare il programma "
"d'installazione al nuovo file di configurazione CMS ." 
------------------------------

There are several entries like this one but I couldn't find a pattern between them. 

Expected results:
The HTML should display the entry in the correct language (Italian in this case).


Additional info:

Comment 1 Jeff Fearn 🐞 2010-02-25 02:27:19 UTC
Attribute matching regex was too loose, so it was mmatching nested tags of similar name. In this case para amd parameter.

Tightened up regex to avoid matching sub-strings.

Comment 2 Fedora Update System 2010-02-26 05:25:14 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 3 Fedora Update System 2010-02-26 05:29:04 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 4 Fedora Update System 2010-02-26 05:29:17 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 5 Fedora Update System 2010-02-26 05:29:21 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 6 Fedora Update System 2010-02-26 05:31:38 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 7 Fedora Update System 2010-02-26 05:35:29 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 8 Fedora Update System 2010-02-27 03:29:50 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 9 Fedora Update System 2010-02-27 03:45:35 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.

Comment 10 Manuel Ospina 2010-03-05 04:06:46 UTC
There are still some entries that are not replaced:

#. Tag: para
#, no-c-format
msgid ""
"Valgrind has been updated to version 3.5.0, providing enhanced support for a "
"wide range of system architectures. This update introduces many improvements "
"to performance, scalability and usability of the Valgrind. Notably, the "
"usability and scalability of the Helgrind tool — which is used to detect "
"race contiditions — has been improved. The leak checking capabilities of the "
"<filename>Memcheck</filename> tool are also improved. Additionally, support "
"for DWARF debugging information has been enhanced."
msgstr "Valgrind ha sido actualizada a la versión 3.5.0, proporcionando soporte mejorado para un amplio rango de arquitecturas de sistema.   Esta actualización introduce muchas mejoras en rendimiento, escalabilidad y utilidad de Valgrind. En particular, la utilidad y escalabilidad de la herramienta Helgrind — la cual sirve para detectar condiciones de carrera — ha sido mejorada. También han sido mejorados la filtración de verificación de funcionalidades de la herramienta <filename>Memcheck</filename> y el soporte para información de depuración DWARF."


XML:
<para>
        Valgrind has been updated to version 3.5.0, providing enhanced support for a wide range of system architectures. This update introduces many improvements to performance, scalability and usability of the Valgrind. Notably, the usability and scalability of the Helgrind tool — which is used to detect race contiditions — has been improved. The leak checking capabilities of the <filename>Memcheck</filename> tool are also improved. Additionally, support for DWARF debugging information has been enhanced.
    </para>

from: Release Notes rhel5.5

Comment 11 Jeff Fearn 🐞 2010-03-05 05:14:36 UTC
It's the fat dash, —, that is doing it, not sure why.

The work around is to use a normal dash or &mdash;

Comment 12 Jeff Fearn 🐞 2010-03-22 05:54:38 UTC
Forced UTF8 on all the time.

Comment 13 Ruediger Landmann 2010-03-22 06:43:07 UTC
Verified that UTF8 em dashes and en dashes get matched and replaced in 1.6.1

Comment 14 Fedora Update System 2010-03-24 00:12:53 UTC
publican-1.6.1-0.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/publican-1.6.1-0.fc12

Comment 15 Fedora Update System 2010-03-24 00:13:29 UTC
publican-1.6.1-0.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/publican-1.6.1-0.fc11

Comment 16 Fedora Update System 2010-03-24 00:13:59 UTC
publican-1.6.1-0.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/publican-1.6.1-0.fc13

Comment 17 Fedora Update System 2010-03-24 00:46:13 UTC
publican-1.6.1-0.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 18 Fedora Update System 2010-03-24 23:35:47 UTC
publican-1.6.1-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 19 Fedora Update System 2010-03-24 23:37:29 UTC
publican-1.6.1-0.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.