plural-forms parsing rejects commas though msgfmt supports them

[Colin Watson]

Launchpad mailed me an "Import problem" message as follows:

  On 2009-03-04 11:52+0000 (39 minutes ago), you uploaded a file with
  Tagalog (tl) translations for debconf in Ubuntu Jaunty package "debconf"
  to Launchpad.

  We were unable to import the file because of errors in its format:

  Invalid nplurals declaration in header: '2, plural=n>1' (should be a
  number).

  If you use gettext, you can check your file for correct formatting with
  the 'msgfmt -c' command. Please fix any errors raised by msgfmt and
  upload the file again. If you check the file and you don't find any
  error in it, please look for an answer or file a question at
  https://answers.launchpad.net/rosetta/

  For your convenience, you can get the file you uploaded at:
  http://launchpadlibrarian.net/23417401/tl.po

The offending line is:

  "Plural-Forms: nplurals=2, plural=n>1;\n"

While the gettext documentation does say that this should be "Plural-Forms: nplurals=2; plural=n>1;\n" instead, I don't think gettext itself actually cares, and certainly 'msgfmt -c' as recommended by Launchpad's mail doesn't report any errors. Perhaps you could make this a little less strict, and permit comma-separation too?

[Данило Шеган (danilo)]

That's right, gettext doesn't seem to care, so we should allow it as well.

[Данило Шеган (danilo)]

And just to clarify, I've checked that comma actually works.