skilleter-thingy 0.0.50__py3-none-any.whl → 0.0.52__py3-none-any.whl

skilleter_thingy/git_mr.py

@@ -49,12 +49,13 @@ def main():
     else:
         parents, _ = git.parents()
 
+        if not parents:
+            colour.error('Unable to determine parent branch. Use the [BLUE]--parent[NORMAL] option to specify the appropriate one.')
+
         if len(parents) > 1:
             parent_list = ', '.join(parents)
             colour.error(
                 f'Branch has multiple potential parents: [BLUE]{parent_list}[NORMAL]. Use the [BLUE]--parent[NORMAL] option to specify the appropriate one.')
-        elif not parents:
-            colour.error('Unable to determine parent branch. Use the [BLUE]--parent[NORMAL] option to specify the appropriate one.')
 
     options = ['merge_request.create', f'merge_request.target={parents[0]}']
 

skilleter_thingy/thingy/files.py

@@ -73,6 +73,8 @@ def format_size(size, always_suffix=False):
         # to 1 (set to 0 when we don't have a divisor).
 
         div *= 1024
+    else:
+        units = ' PiB'
 
     # Calculate the size in 10ths so the we get the first digit after
     # the decimal point, doing all the work in the integer domain to
@@ -89,9 +91,13 @@ def format_size(size, always_suffix=False):
 
 ################################################################################
 
-def backup(filename, extension='bak', copyfile=True):
+def backup(filename, extension='bak', copyfile=True, timestamps=True):
     """ Create a backup of a file by copying or renaming it into a file with extension
-        .bak, deleting any existing file with that name """
+        .bak, deleting any existing file with that name.
+
+        Copies the file by default, unless copyfile is False
+        Sets the timetamps of the backup file to be the same as the original, unless
+        timestamps is False."""
 
     # Do nothing if the file does not exist
 
@@ -117,13 +123,20 @@ def backup(filename, extension='bak', copyfile=True):
     if os.path.isfile(backupname):
         os.unlink(backupname)
 
-    # Create the backup by copying or renaming the file
+    # Create the backup by copying or renaming the file, optionally preserving the
+    # timestamp of the original file
+
+    if timestamps:
+        file_info = os.stat(filename)
 
     if copyfile:
         shutil.copyfile(filename, backupname)
     else:
         os.rename(filename, backupname)
 
+    if timestamps:
+        os.utime(backupname, ns=(file_info.st_atime_ns, file_info.st_mtime_ns))
+
 ################################################################################
 # Test code
 

skilleter_thingy/thingy/git.py

@@ -515,6 +515,13 @@ def difftool(commit_1=None, commit_2=None, files=None, tool=None):
 
 ################################################################################
 
+# Match 'git diff --numstat' output - first re splits into lines added, removed
+# and name. Second one is used if a file has been renamed, to get the old and
+# new name components.
+
+_DIFF_OUTPUT_RE = re.compile(r'(-|\d+)\s+(-|\d+)\s+(.*)')
+_DIFF_OUTPUT_RENAME_RE = re.compile(r'(.*)\{(.*) => (.*)\}(.*)')
+
 def commit_info(commit_1=None, commit_2=None, paths=None, diff_stats=False):
     """ Return details of changes either in single commit (defaulting to the most
         recent one) or between two commits, optionally restricted a path or paths
@@ -523,22 +530,35 @@ def commit_info(commit_1=None, commit_2=None, paths=None, diff_stats=False):
     """
 
     def parse_diff_output(result):
-        """Extract current filename and lines added/deleted from output from git diff --numstat"""
+        """Extract previous and current filename (which may be the same) and lines added/deleted
+           from output from git diff --numstat"""
+
+        p = _DIFF_OUTPUT_RE.fullmatch(result)
+
+        # This shouldn't happen...
 
-        p = re.fullmatch(r'(-|\d+)\s+(-|\d+)\s+(.*)', result)
+        if not p:
+            return 'ERROR', 'ERROR', 0, 0
+
+        # Extract number of lines added/removed
 
         lines_ins = 0 if p.group(1) == '-' else int(p.group(1))
         lines_del = 0 if p.group(2) == '-' else int(p.group(2))
 
+        # Check for rename and get both old and new names
+
         if ' => ' in p.group(3):
-            filepath = p.group(3).split('{')[0]
-            filename = p.group(3).split(' => ')[1]
-            filename = filename.replace('}', '')
-            filename = os.path.join(filepath, filename)
+            q = _DIFF_OUTPUT_RENAME_RE.fullmatch(p.group(3))
+
+            if q:
+                old_filename = q.group(1) + q.group(2) + q.group(4)
+                new_filename = q.group(1) + q.group(3) + q.group(4)
+            else:
+                old_filename, new_filename = p.group(3).split(' => ')
         else:
-            filename = p.group(3)
+            old_filename = new_filename = p.group(3)
 
-        return filename, lines_ins, lines_del
+        return old_filename, new_filename, lines_ins, lines_del
 
     # Either get changes between the two commits, or changes in the specified commit
 
@@ -588,22 +608,23 @@ def commit_info(commit_1=None, commit_2=None, paths=None, diff_stats=False):
         results = git(['diff', '--numstat'] + params)
 
         for result in results:
-            filename, lines_ins, lines_del = parse_diff_output(result)
+            old_filename, new_filename, lines_ins, lines_del = parse_diff_output(result)
 
-            info[filename]['deleted'] = lines_del
-            info[filename]['added'] = lines_ins
+            info[new_filename]['deleted'] = lines_del
+            info[new_filename]['added'] = lines_ins
 
         # Run git diff to get stats ignoring whitespace changes and add them
 
         results = git(['diff', '--numstat', '--ignore-all-space', '--ignore-blank-lines'] + params)
 
         for result in results:
-            filename, lines_ins, lines_del = parse_diff_output(result)
+            old_filename, new_filename, lines_ins, lines_del = parse_diff_output(result)
 
-            info[filename]['non-ws deleted'] = lines_del
-            info[filename]['non-ws added'] = lines_ins
+            info[new_filename]['non-ws deleted'] = lines_del
+            info[new_filename]['non-ws added'] = lines_ins
 
         # Fill in the blanks - files
+
         for filename in info:
             if 'deleted' not in info[filename]:
                 info[filename]['deleted'] = info[filename]['added'] = 0
@@ -1326,7 +1347,7 @@ def run_tests():
         if value is None:
             print('  Successfully failed to read the newly-deleted config data')
         else:
-            raise GitError
+            raise GitError('Unexpected lack of error reading configuration data', 1)
 
         config_set('user', 'email', 'user@localhost')
         config_set('user', 'name', 'User Name')

skilleter_thingy/thingy/git2.py

@@ -1259,7 +1259,7 @@ if __name__ == '__main__':
         if value is None:
             print('  Successfully failed to read the newly-deleted config data')
         else:
-            raise GitError
+            raise GitError('Unexpectd lack of error reading configuration data', 1)
 
         config_set('user', 'email', 'user@localhost')
         config_set('user', 'name', 'User Name')

skilleter_thingy/thingy/run.py

@@ -60,7 +60,7 @@ def capture_output(cmd, input_stream, output_streams, ansi_clean):
                         if stream in (sys.stdout, sys.stderr):
                             stream.write(tidy.convert_ansi(output))
                         elif ansi_clean:
-                            stream.write(colour.clean(output))
+                            stream.write(tidy.remove_ansi(output))
                         else:
                             stream.write(output)
 

skilleter_thingy/thingy/tfm_pane.py

@@ -63,7 +63,7 @@ class Pane():
 
         self.index = index
 
-        self.current_dir = None
+        self.current_dir = ''
 
         self.ino = inotify.adapters.Inotify() if sys.platform == 'linux' else None
 
@@ -290,13 +290,11 @@ class Pane():
                 else:
                     entry = name + ' ' * (self.width - len(name) - len(data)) + data
 
+                file_colour = self.dircolours.get_colour_pair(current_file['name'], current_file['mode'])
             else:
                 filename = entry = None
                 current = False
-
-            # Render the current line
-
-            file_colour = self.dircolours.get_colour_pair(current_file['name'], current_file['mode']) if filename else normal_colour
+                file_colour = normal_colour
 
             # Reverse the colours if this the cursor line
 

{skilleter_thingy-0.0.50.dist-info → skilleter_thingy-0.0.52.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: skilleter_thingy
-Version: 0.0.50
+Version: 0.0.52
 Summary: A collection of useful utilities, mainly aimed at making Git more friendly
 Author-email: John Skilleter <john@skilleter.org.uk>
 Project-URL: Home, https://skilleter.org.uk

{skilleter_thingy-0.0.50.dist-info → skilleter_thingy-0.0.52.dist-info}/RECORD

@@ -14,7 +14,7 @@ skilleter_thingy/git_cleanup.py,sha256=TnUPYAUnByVyY_dwANzDg2BGGNh3jskNF1DgU3pc8
 skilleter_thingy/git_co.py,sha256=gBES0ltlomy7vXX8IHh0-Vt1af54t2Ie5UnRYYfP4tM,8212
 skilleter_thingy/git_common.py,sha256=ZjNkvIBRDGNLFYwOu9FjeqdDKJdm0sndX5QATtmq290,1879
 skilleter_thingy/git_hold.py,sha256=Zk6YUhr08znUOdpVJkJAt0rOtrCXnKE0NHoZzoaEEGo,4616
-skilleter_thingy/git_mr.py,sha256=BCE68hhz2bV0zwHkEv5tyfRY0aXoUdb_aJ-yWYDu_Wo,3087
+skilleter_thingy/git_mr.py,sha256=2vLzpd5waddNBRlOvQUxwqHVQaK9Rg6FYHb4VMEpS8c,3086
 skilleter_thingy/git_parent.py,sha256=D47D01dPs6kActusA0ircJYTv8tBjDoe8MzMpJVWJpA,2683
 skilleter_thingy/git_review.py,sha256=WCBw49OP3WHiaxku8jUUk6I_NqtjScLGaQHvPxX6LnU,52053
 skilleter_thingy/git_update.py,sha256=uYd4ubnDbHzkwwrtzVatNIg-L15ap1X8UMO-2PafseY,13971
@@ -49,21 +49,21 @@ skilleter_thingy/thingy/dc_defaults.py,sha256=ahcteQvoWZrO5iTU68zkIY1Zex6iX5uR5u
 skilleter_thingy/thingy/dc_util.py,sha256=Df73imXhHx3HzcPHiRcHAoea0e3HURdLcrolUsMhOFs,1783
 skilleter_thingy/thingy/dircolors.py,sha256=5NbXMsGWdABLvvZfB70VPmN6N5HyyihfpgoQq1NRJbg,12264
 skilleter_thingy/thingy/docker.py,sha256=9EFatudoVPfB1UbDEtzdJDB3o6ToHiNHv8-oLsUeqiQ,2449
-skilleter_thingy/thingy/files.py,sha256=8wXUHEGALhHvYtJewjSNwHFw-hGHFYbWH9_40lJj5ZY,4257
-skilleter_thingy/thingy/git.py,sha256=9y58KhFx6t9nNyqMbixM5cl8LBcD2sdCh9UgyVPv4bo,38043
-skilleter_thingy/thingy/git2.py,sha256=HdLar1rN5-Uop6GvoWM-NERDBOI7s93dC6Ry7iuEumA,35473
+skilleter_thingy/thingy/files.py,sha256=oW6E6WWwVFSUPdrZnKMx7P_w_hh3etjoN7RrqvYHCHc,4705
+skilleter_thingy/thingy/git.py,sha256=qXWIduF4jbP5pKFYt_hW9Ex5iL9mSBBrcNKBkULhRTg,38834
+skilleter_thingy/thingy/git2.py,sha256=UgdAC1rZWf6Tdl9xB6PAIT6K_3OjD2juCOEQSHEEb90,35530
 skilleter_thingy/thingy/gitlab.py,sha256=uXAF918xnPk6qQyiwPQDbMZfqtJzhiRqDS7yEtJEIAg,6079
 skilleter_thingy/thingy/logger.py,sha256=xKgPAq8KGXmtaXIFjFs1AmZJXtYrXJn2sqL3oxHZjfQ,3107
 skilleter_thingy/thingy/path.py,sha256=me__Ukw-7NiD70Yd9tOWyj7QX79-deFvsQaQ9AGzWzU,4732
 skilleter_thingy/thingy/popup.py,sha256=jW-nbpdeswqEMTli7OmBv1J8XQsvFoMI0J33O6dOeu8,2529
 skilleter_thingy/thingy/process.py,sha256=88pKHQZXBP1m3Ja7t3DtKJ4Njn7HS2OtcI0Z0i1KwUs,3560
-skilleter_thingy/thingy/run.py,sha256=051lGahG4liYLckQFpmSaGuE9Chd-lFdmJO85LdmeXE,12607
-skilleter_thingy/thingy/tfm_pane.py,sha256=40DmQeLMEUPiKKIJkgN1MEpIen00V70I1HB7Q6git44,19814
+skilleter_thingy/thingy/run.py,sha256=6SNKWF01fSxzB10GMU9ajraXYZqAL1w0PXkqjJdr1Uo,12611
+skilleter_thingy/thingy/tfm_pane.py,sha256=oqy5zBzKwfbjbGqetbbhpKi4x5He7sl4qkmhUeqtdZc,19789
 skilleter_thingy/thingy/tidy.py,sha256=0EVUP2XyUTtoIpqaDwvrHL58YxM4Elvu5MO1nzhwLG4,5854
 skilleter_thingy/thingy/venv_template.py,sha256=SsVNvSwojd8NnFeQaZPCRQYTNdwJRplpZpygbUEXRnY,1015
-skilleter_thingy-0.0.50.dist-info/LICENSE,sha256=ljOS4DjXvqEo5VzGfdaRwgRZPbNScGBmfwyC8PChvmQ,32422
-skilleter_thingy-0.0.50.dist-info/METADATA,sha256=itzZGWAuQ6uaklD1TJH1F75YFn0AQCmpv7clMQ6XHD8,5313
-skilleter_thingy-0.0.50.dist-info/WHEEL,sha256=R0nc6qTxuoLk7ShA2_Y-UWkN8ZdfDBG2B6Eqpz2WXbs,91
-skilleter_thingy-0.0.50.dist-info/entry_points.txt,sha256=IT6cZSbGHrd2UzIQbiMRotKiTJnYJBkESC4fAe8gjsE,2026
-skilleter_thingy-0.0.50.dist-info/top_level.txt,sha256=8-JhgToBBiWURunmvfpSxEvNkDHQQ7r25-aBXtZv61g,17
-skilleter_thingy-0.0.50.dist-info/RECORD,,
+skilleter_thingy-0.0.52.dist-info/LICENSE,sha256=ljOS4DjXvqEo5VzGfdaRwgRZPbNScGBmfwyC8PChvmQ,32422
+skilleter_thingy-0.0.52.dist-info/METADATA,sha256=buYbiQqFhFBdTi6MM02Jy8ZH8wuXmLxI2StMjulMVoo,5313
+skilleter_thingy-0.0.52.dist-info/WHEEL,sha256=cVxcB9AmuTcXqmwrtPhNK88dr7IR_b6qagTj0UvIEbY,91
+skilleter_thingy-0.0.52.dist-info/entry_points.txt,sha256=IT6cZSbGHrd2UzIQbiMRotKiTJnYJBkESC4fAe8gjsE,2026
+skilleter_thingy-0.0.52.dist-info/top_level.txt,sha256=8-JhgToBBiWURunmvfpSxEvNkDHQQ7r25-aBXtZv61g,17
+skilleter_thingy-0.0.52.dist-info/RECORD,,

{skilleter_thingy-0.0.50.dist-info → skilleter_thingy-0.0.52.dist-info}/WHEEL

@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: setuptools (72.1.0)
+Generator: setuptools (74.1.2)
 Root-Is-Purelib: true
 Tag: py3-none-any