handle missing files/directories in extract

openedx · Apr 4, 2017 · 6d2d7e5 · 6d2d7e5
1 parent ee649fe
commit 6d2d7e5
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 4 deletions.
diff --git a/README.rst b/README.rst
@@ -86,6 +86,11 @@ Details of the config.yaml file are in `edx-platform/conf/locale/config.yaml
 Changes
 =======
 
+v0.3.8
+------
+
+* Fixes for extract to handle when expected directories/files are not present.
+
 v0.3.7
 ------
 

diff --git a/i18n/__init__.py b/i18n/__init__.py
@@ -6,7 +6,7 @@
 
 from . import config
 
-__version__ = '0.3.7'
+__version__ = '0.3.8'
 
 
 class Runner:

diff --git a/i18n/extract.py b/i18n/extract.py
@@ -56,15 +56,22 @@ def rename_source_file(self, src, dst):
         """
         Rename a file in the source directory.
         """
-        os.rename(self.source_msgs_dir.joinpath(src), self.source_msgs_dir.joinpath(dst))
+        src_path = self.source_msgs_dir.joinpath(src)
+        dst_path = self.source_msgs_dir.joinpath(dst)
 
-    def run(self, args):
+        # Only rename files that exist.
+        if os.path.exists(src_path):
+            os.rename(src_path, dst_path)
+
+    def run(self, args):  # pylint: disable=too-many-statements
         """
         Main entry point of script
         """
         logging.basicConfig(stream=sys.stdout, level=logging.INFO)
         configuration = self.configuration
         configuration.locale_dir.parent.makedirs_p()
+        configuration.source_messages_dir.makedirs_p()
+
         # pylint: disable=attribute-defined-outside-init
         self.source_msgs_dir = configuration.source_messages_dir
 

diff --git a/setup.py b/setup.py
@@ -4,7 +4,7 @@
 
 setup(
     name='edx-i18n-tools',
-    version='0.3.7',
+    version='0.3.8',
     description='edX Internationalization Tools',
     author='edX',
     author_email='oscm@edx.org',