Commit 1e8a0cb7 by Ned Batchelder

Tighten up the extraction I/O

We were writing the main file twice.  No need.
parent 2abd6473
...@@ -41,9 +41,6 @@ def main(): ...@@ -41,9 +41,6 @@ def main():
create_dir_if_necessary(LOCALE_DIR) create_dir_if_necessary(LOCALE_DIR)
source_msgs_dir = CONFIGURATION.source_messages_dir source_msgs_dir = CONFIGURATION.source_messages_dir
generated_files = ['django-partial.po', 'djangojs.po', 'mako.po']
# Prepare makemessages command.
makemessages = "django-admin.py makemessages -l en" makemessages = "django-admin.py makemessages -l en"
ignores = " ".join('--ignore="{}/*"'.format(d) for d in CONFIGURATION.ignore_dirs) ignores = " ".join('--ignore="{}/*"'.format(d) for d in CONFIGURATION.ignore_dirs)
if ignores: if ignores:
...@@ -70,10 +67,9 @@ def main(): ...@@ -70,10 +67,9 @@ def main():
# Segment the generated files. # Segment the generated files.
segmented_files = segment_pofiles("en") segmented_files = segment_pofiles("en")
generated_files.extend(segmented_files)
# Finish each file. # Finish each file.
for filename in generated_files: for filename in segmented_files:
LOG.info('Cleaning %s' % filename) LOG.info('Cleaning %s' % filename)
po = pofile(source_msgs_dir.joinpath(filename)) po = pofile(source_msgs_dir.joinpath(filename))
# replace default headers with edX headers # replace default headers with edX headers
......
...@@ -93,13 +93,13 @@ def segment_pofile(filename, segments): ...@@ -93,13 +93,13 @@ def segment_pofile(filename, segments):
else: else:
msg_segments.add(filename) msg_segments.add(filename)
assert msg_segments
if len(msg_segments) == 1: if len(msg_segments) == 1:
# This message belongs in this segment. # This message belongs in this segment.
segment_file = msg_segments.pop() segment_file = msg_segments.pop()
segment_po_files[segment_file].append(msg) segment_po_files[segment_file].append(msg)
else: else:
# Either it's in more than one segment, or none, so put it back in # It's in more than one segment, so put it back in the main file.
# the main file.
remaining_po.append(msg) remaining_po.append(msg)
# Write out the results. # Write out the results.
...@@ -113,9 +113,6 @@ def segment_pofile(filename, segments): ...@@ -113,9 +113,6 @@ def segment_pofile(filename, segments):
pofile.save(out_file) pofile.save(out_file)
files_written.add(out_file) files_written.add(out_file)
LOG.info(writing_msg.format(file=filename, num=len(remaining_po)))
remaining_po.save(filename)
return files_written return files_written
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment