[patch] make-messages.py duplicates strings already translated in sub-directories

[Antti Kaihola]

Apps with app-specific translations must reside outside the project tree so make-messages.py won't traverse them when constructing the project gettext catalog.

The attached patch removes the need for this trick by not traversing ./apps by default. A "-p" option is included for the old behavior. The usage message is also improved.

[hugo]

Hmm. I don't think that is really needed. Apps that reside within a project should be translated with the project - and apps that should have their own translation should be stored outside the project. I think that's simple enough. Most definitely I won't like the default of make-messages.py to be to skip a folder just because it is named "apps" - that's some implicit rule that isn't obvious to users.

[Antti Kaihola]

The reason I wanted this functionality is that I really like to separate translations of different parts in my project into their own files. It also makes things much easier if I want to copy an app to another project or to a shared location.

I re-thought the implementation and my patch to make-messages.py now automatically excludes any directories which contain the locale/<langcode>/LC_MESSAGES/<domain>.po file. I think it makes sense to not duplicate strings which are already found.

I'm sorry if I'm insisting too much...

[Antti Kaihola]

re-thought implementation looks for .po files and excludes directories based on that

[Adrian Holovaty]

Hugo, what do you think of this?

[hugo]

I still think that applications within the project directory should be translated with the project, while separately translated applications should be outside - looking at project directory layout to decide what parts of the directory to traverse sounds a bit hackish to me.

[Adrian Holovaty]

I defer to Hugo's judgment on this one, so I'm closing as wontfix. See his comment above.