]> granicus.if.org Git - python/commitdiff
use the same line terminator everywhere
authorTarek Ziade <tarek@ziade.org>
Sat, 21 May 2011 17:45:48 +0000 (19:45 +0200)
committerTarek Ziade <tarek@ziade.org>
Sat, 21 May 2011 17:45:48 +0000 (19:45 +0200)
Lib/packaging/command/install_distinfo.py
Lib/packaging/database.py
Lib/packaging/tests/test_database.py

index db49236b96a939af27037a83b315f871234f48f2..3390a1f0a7c6cabd9472ef04fd7911bf28eee443 100644 (file)
@@ -110,7 +110,7 @@ class install_distinfo(Command):
                     logger.info('creating %s', resources_path)
                     with open(resources_path, 'wb') as f:
                         writer = csv.writer(f, delimiter=',',
-                                            lineterminator=os.linesep,
+                                            lineterminator='\n',
                                             quotechar='"')
                         for tuple in install_data.get_resources_out():
                             writer.writerow(tuple)
@@ -122,7 +122,7 @@ class install_distinfo(Command):
                 logger.info('creating %s', record_path)
                 with open(record_path, 'w', encoding='utf-8') as f:
                     writer = csv.writer(f, delimiter=',',
-                                        lineterminator=os.linesep,
+                                        lineterminator='\n',
                                         quotechar='"')
 
                     install = self.get_finalized_command('install_dist')
index 087a6ecadde3b2fd25425a94472e52da963699af..b1071489f6e3264648c42f3d39ae0370225ab349 100644 (file)
@@ -159,7 +159,8 @@ class Distribution:
 
     def _get_records(self, local=False):
         with self.get_distinfo_file('RECORD') as record:
-            record_reader = csv.reader(record, delimiter=',')
+            record_reader = csv.reader(record, delimiter=',',
+                                       lineterminator='\n')
             # XXX needs an explaining comment
             for row in record_reader:
                 path, checksum, size = (row[:] +
@@ -171,7 +172,8 @@ class Distribution:
 
     def get_resource_path(self, relative_path):
         with self.get_distinfo_file('RESOURCES') as resources_file:
-            resources_reader = csv.reader(resources_file, delimiter=',')
+            resources_reader = csv.reader(resources_file, delimiter=',',
+                                           lineterminator='\n')
             for relative, destination in resources_reader:
                 if relative == relative_path:
                     return destination
index c8d63852356a4d6d582c458f849e147e7550ead0..ea63d3e4f15a59c194289340ed7b72e9a5927b7b 100644 (file)
@@ -111,7 +111,8 @@ class TestDistribution(CommonDistributionTests, unittest.TestCase):
             record_file = os.path.join(distinfo_dir, 'RECORD')
             with open(record_file, 'w') as file:
                 record_writer = csv.writer(
-                    file, delimiter=',', quoting=csv.QUOTE_NONE)
+                    file, delimiter=',', quoting=csv.QUOTE_NONE,
+                    lineterminator='\n')
 
                 dist_location = distinfo_dir.replace('.dist-info', '')
 
@@ -125,9 +126,11 @@ class TestDistribution(CommonDistributionTests, unittest.TestCase):
                 record_writer.writerow([relpath(record_file, sys.prefix)])
 
             with open(record_file) as file:
-                record_reader = csv.reader(file)
+                record_reader = csv.reader(file, lineterminator='\n')
                 record_data = {}
                 for row in record_reader:
+                    if row == []:
+                        continue
                     path, md5_, size = (row[:] +
                                         [None for i in range(len(row), 3)])
                     record_data[path] = md5_, size