6
0
Fork 0

Use tde_l10n_prepare_xml macro instead of extractrc script #10

Zusammengeführt
SlavekB hat 1 Commits von feat/replace-extractrc-by-macro nach master vor 5 Jahren zusammengeführt
SlavekB hat vor 5 Jahren kommentiert
Besitzer

Because the new macro tde_l10n_prepare_xml provides the same extraction of strings, only in a way other than the extractrc script, there is the option to replace the extractrc script with the macro tde_l10n_prepare_xml.

The advantage is that xgettext will then generate location information corresponding directly to rows in the original XML file.

For interest: For multiple line strings, extractrc generates a location information, that points to the last line of text, while with the tde_10n_prepare_xml macro is generated location information, that points to the first line of text.

Because the new macro tde_l10n_prepare_xml provides the same extraction of strings, only in a way other than the extractrc script, there is the option to replace the extractrc script with the macro tde_l10n_prepare_xml. The advantage is that xgettext will then generate location information corresponding directly to rows in the original XML file. For interest: For multiple line strings, extractrc generates a location information, that points to the last line of text, while with the tde_10n_prepare_xml macro is generated location information, that points to the first line of text.
MicheleC hat die Änderungen vor 5 Jahren genehmigt
MicheleC hat einen Kommentar hinterlassen
Besitzer

looks good. I like the fact that the reference points to the first line instead of the last line 👍

looks good. I like the fact that the reference points to the first line instead of the last line :+1:
SlavekB hat vor 5 Jahren kommentiert
Ersteller
Besitzer

Note0: For this PR, it is required that #9 be already merged.

Note1: If you want to test #8, #9 and #10 together, there are two small conflicts between #8 and #10, because #10 removes some of the parts that were changed in #8.

Note0: For this PR, it is required that #9 be already merged. Note1: If you want to test #8, #9 and #10 together, there are two small conflicts between #8 and #10, because #10 removes some of the parts that were changed in #8.
SlavekB hat diesen Pull-Request vor 5 Jahren geschlossen
SlavekB löschte die Branch feat/replace-extractrc-by-macro vor 5 Jahren
SlavekB hat diesen Issue vor 5 Jahren zum R14.0.6 release Meilenstein hinzugefügt
Der Pull Request wurde als 3c40942680 gemergt.
Anmelden, um an der Diskussion teilzunehmen.
Keine Reviewer
Kein Meilenstein
Niemand zuständig
2 Beteiligte
Nachrichten
Fällig am

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: TDE/tde-cmake#10
Laden…
Hier gibt es bis jetzt noch keinen Inhalt.