Request: Please avoid to use underscores in message contexts

Bug #505157 reported by Masato HASHIMOTO
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
jazzynico

Bug Description

Please avoid to use underscores in message contexts because `msgfmt -c -v` reports warning as asynced keyboard accelerators.

Attached is a patch to replace existing underscores with space.

Regards

Tags: translation

Related branches

Revision history for this message
Masato HASHIMOTO (hashimo) wrote :
jazzynico (jazzynico)
Changed in inkscape:
assignee: nobody → JazzyNico (jazzynico)
importance: Undecided → Low
milestone: none → 0.48
status: New → Triaged
Revision history for this message
jazzynico (jazzynico) wrote :

Hi,

I'm a bit reluctant to apply this patch because on my computer, msgfmt -c -v doesn't return warnings, and I see no restriction about underscores (in the context string) in the gettext documentation.
I'm using gettext 0.17-8ubuntu2.
Could you please give your gettext version and attach a sample of your warnings?
Thanks!

Regards.

Revision history for this message
Masato HASHIMOTO (hashimo) wrote :

Oops, sorry, it's my fault.
Exact options are "msgfmt -c -v **--check-accelerators=_**".

The result should be as follows (in inkscape0.47):

$ LANG=C msgfmt ja.po -c -v --check-accelerators=_ -o /dev/null
ja.po:9402: msgstr lacks the keyboard accelerator mark '_'
ja.po:18559: msgstr lacks the keyboard accelerator mark '_'
ja.po:18563: msgstr lacks the keyboard accelerator mark '_'
ja.po:18573: msgstr lacks the keyboard accelerator mark '_'
ja.po:18577: msgstr lacks the keyboard accelerator mark '_'
ja.po:18587: msgstr lacks the keyboard accelerator mark '_'
ja.po:18591: msgstr lacks the keyboard accelerator mark '_'
ja.po:18609: msgstr lacks the keyboard accelerator mark '_'
ja.po:18613: msgstr lacks the keyboard accelerator mark '_'
4960 translated messages, 18 untranslated messages.

Revision history for this message
Masato HASHIMOTO (hashimo) wrote :

I forgot one more info.
The patch is for bzr/trunk.

Regards

Revision history for this message
jazzynico (jazzynico) wrote :

Fix committed in revision 9154.
Thanks for the patch!

Changed in inkscape:
status: Triaged → Fix Committed
jazzynico (jazzynico)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.