cf grangier#13 - Fixes multiplatform paths

Author: grangier

goose/__init__.py

@@ -74,8 +74,7 @@ def initialize(self):
 
         # test to write a dummy file to the directory
         # to check is directory is writtable
-		# FIXEME : use os.path.join
-        path = '%s/test.txt' % self.config.local_storage_path
+        path = os.path.join(self.config.local_storage_path, 'test.txt')
         try:
             f = open(path, 'w')
             f.close()

goose/crawler.py

@@ -126,8 +126,7 @@ def get_extractor(self):
         return StandardContentExtractor(self.config)
 
     def relase_resources(self, article):
-		# FIXEME : use os.path.join
-        path = '%s/%s_*' % (self.config.local_storage_path, article.link_hash)
+        path = os.path.join(self.config.local_storage_path, '%s_*' % article.link_hash)
         for fname in glob.glob(path):
             try:
                 os.remove(fname)

goose/images/utils.py

@@ -103,8 +103,7 @@ def write_localfile(self, entity, link_hash, src, config):
     @classmethod
     def get_localfile_name(self, link_hash, src, config):
         image_hash = hashlib.md5(smart_str(src)).hexdigest()
-		# FIXEME : use os.path.join
-        return config.local_storage_path + "/" + link_hash + "_py_" + image_hash
+        return os.path.join(config.local_storage_path, '%s_%s' % (link_hash, image_hash))
 
     @classmethod
     def clean_src_string(self, src):