Add 'Language' header

Bug #558682 reported by AJenbo
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

Poedit needs an additional header to enable spellchecking.

Heres a sample:
"X-Poedit-Language: Danish"

Revision history for this message
Václav Slavík (vslavik) wrote :

Poedit developer here.

Poedit uses X-Poedit-Language (*and* X-Poedit-Country) to determine the language, true. But it doesn't *need* it if it can determine the language by other means, i.e. by looking at the file name. Poedit 1.4 understands only names that are lang codes ("cs.po", "en_GB.po" etc.), Poedit 1.5 understands anything.LANG.po style too.

Future versions (1.5 or 1.6) will understand the "Language" header (e.g. "Language: en_GB") that will be part of gettext 0.18 and will deprecate these X-Poedit-* headers.

Revision history for this message
Adi Roiban (adiroiban) wrote :

I don't think that we should add support for X-Poedit-Language , since this is a application specific header. I have nothing agains Poedit and it is my favorite editor :)
If we are going on this way we will have to also add X-Virtall or X-Kbabel specific headers.

I guess we can add the "Language" header if it is a standard header in gettext and since it will deprecate the X-Poedit headers.

Since Poedit is very smart and there are many other important things to solve first in Launchpad Translations, this will be a low priority task.

Revision history for this message
Václav Slavík (vslavik) wrote :

FWIW, I completely agree with you, adding support for this Poedit-specific hack would be waste of time. OTOH, the sooner every producer of PO files supports `Language`, the better.

For reference, here's where `Language` was added to gettext:
http://git.savannah.gnu.org/cgit/gettext.git/commit/?id=2ae1594f7651733bedc7020bce9870a5b5baef48

Adi Roiban (adiroiban)
summary: - add X-Poedit-Language for automatic spellchecking in Poedit
+ Add 'Language' header
Revision history for this message
Данило Шеган (danilo) wrote :

Agreed, we should add it. It should be pretty simple too.

Changed in rosetta:
status: New → Triaged
importance: Undecided → Low
tags: added: feature
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.