django-admin.py makemessages with commas separeted multiple locales doesn't work

[Romain Beylerian]

Hello everyone,

The Django 1.6 ​documentation says that we can use the command django-admin.py makemessages with commas separated locales to specify multiple locales.

With the command django-admin.py makemessages -l en,fr, I expected Django to create two folders (en and fr) in my locale directory, howerver only one folder is created with the name en,fr.

If I repeat several times the -l option, it works. Here is an example:

$ django-admin.py makemessages -l en,fr
processing locale en,fr

$ django-admin.py makemessages -l en -l fr
processing locale en
processing locale fr

[Claude Paroz]

See commit [6158c79dbef832bc85301]. Tests in that commit are using call_command, which is not always the best way to mimic command line behaviour.

[Romain Beylerian]

The compilemessages command seems to have the same problem.

[Simon Charette]

Changing locale='pt,de,ch' to locale=['pt,de', 'ch'] at this ​line should trigger the regression for makemessages.

compilemessages test should be added to ​this case.

[Ramiro Morales]

See ​https://github.com/django/django/pull/1968/files and ​https://github.com/django/django/pull/1969/files

We need to choose between them.

[Claude Paroz]

Considering we already support that syntax for -e, I'd vote for keeping it for -l (not a strong opinion, though).

[Ramiro Morales <cramm0@…>]

In 8750296918072c97a51f6809d19828ce3f1b8d40:

[1.6.x] Fixed #21488 -- Multiple locales treatment in i18n commands.

Removed multiple locales separated by commas variation (that wasn't
working as documented) in favor of simply allowing use of the
--locale/-l options more than once for makemessages and
compilemessages.

Thanks Romain Beylerian for the report and Claude, Simon for their help.

[Ramiro Morales <cramm0@…>]

In 62b393c5ae774c9d5d85bf6eda620354d21e61b1:

Fixed #21488 -- Multiple locales treatment in i18n commands.

Removed multiple locales separated by commas variation (that wasn't
working as documented) in favor of simply allowing use of the
--locale/-l options more than once for makemessages and
compilemessages.

Thanks Romain Beylerian for the report and Claude, Simon for their help.

8750296918072c97a51f6809d19828ce3f1b8d40 from stable/1.6.x.