Merge lp://staging/~mblayman/entertainer/sh-to-py into lp://staging/entertainer
Proposed by
Matt Layman
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Merged at revision: | not available | ||||||||
Proposed branch: | lp://staging/~mblayman/entertainer/sh-to-py | ||||||||
Merge into: | lp://staging/entertainer | ||||||||
To merge this branch: | bzr merge lp://staging/~mblayman/entertainer/sh-to-py | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paul Hummer | Approve | ||
Review via email: mp+2443@code.staging.launchpad.net |
To post a comment you must log in.
Hi guys,
I've rewritten the shell scripts in python and put it in a module to boot. I did this because I'm trying to fix a bug that wants us to remove mo files (and we should) so that we have a clean source tarball. Since the mo file creation is now in python, it was very easy to add the mo creation to the setup.py file (without using subprocess.call to call some shell scripts or trying to invoke make). It is still possible to call `make mo` which will make all the mo files for all the languages we support. I also removed the locale directory because we were just going to get a bunch of lintian errors about empty directories as soon as we took the old mo files out.
If Paul hears from Barry Warsaw about pythongettext status, it should be pretty easy to add that module to replace some of the call commands I have in this code. But since we don't know yet, I figured that the call commands were appropriate.
I looked into using disutils-extra's build_i18n command, but it didn't handle glade files so I had to rule out that option.
This module is pretty straight forward. Hopefully it makes sense to all of you. I'm sure I'll hear about it in any code reviews.
Oh, as a bonus, this also eliminates the stupid pytools directory that I had to create a while ago (a problem of my own doing). Now the entire tools directory is python so I was able to simplify `make lint` a little.