Package: debian-installer Severity: normal Tags: patch On 6/28/05, zinosat@tiscali.it <zinosat@tiscali.it> wrote: > Hi Eddy, > I'd like to use your script for extracting strings from the po files; there's > still > the problem with the path in the "*** path" string and I'm experimenting > a > bit to find out a good solution: you might have to make some changes. I have clearly expressed two possible solutions to this, and adding a hard coded path to the file is ugly. The proposed solutions are: - set an environment variable with the path to be added to the input file before running the script; read that ENV and - give the path prefix as a parameter Running a cwd to get the output and concatenating it with the given input file is a plain hack 'cause one must eliminate a part of it (was /home/zinosat or something like that) Well, all are kind of wrong; if one uses concatenation, 'cause the input file's path is relative and one might give it as "../../some.po" and concatenating that with the detected path is plain wrong and ugly. > > I noticed that the script fails to work with some particular strings (found > in level2 > po files): > > for example base-config.po contains this (I pasted the "de" because atm I > have this file already open :)) > > . Several notes for translators: > #. * The singular form is only there to make use of plural forms; > #. * If package count is 0, a separate error message should be displayed. > #. * This sentence is part of a longer paragraph, full stop is omitted here. > #. * "${PACKAGE_COUNT}" must not be translated. > #: ../apt-setup:308 > #, sh-format > msgid "The Debian package management tool, apt, is now configured, and can > install ${PACKAGE_COUNT} package" > msgid_plural "The Debian package management tool, apt, is now configured, > and can install ${PACKAGE_COUNT} packages" > msgstr[0] "Der Debian Paketmanager Apt wurde konfiguriert, es können allerdings > keine Pakete installiert werden" > msgstr[1] "Der Debian Paketmanager Apt wurde konfiguriert und kann ${PACKAGE_COUNT} > Pakete installieren" > > could you please see if you can make your script catch this strings? > thanx in advance > The attached patch should be applied to the script scripts/l10n/l10nspellcheck/extract_msg.pl in order to catch the plural forms (catches also msgid and msgid_plural). Please apply and commit the patch (commit comment: "add suport for plural forms") -- Regards, EddyP ============================================= "Imagination is more important than knowledge" A.Einstein
Attachment:
plural_forms.patch
Description: Binary data