skilleter-thingy 0.1.7__py3-none-any.whl → 0.1.9__py3-none-any.whl

skilleter_thingy/git_review.py

@@ -102,7 +102,7 @@ class GitReviewError(BaseException):
 def error(msg, status=1):
     """ Report an error """
 
-    sys.stderr.write('%s\n' % msg)
+    sys.stderr.write(f'{msg}\n')
     sys.exit(status)
 
 ################################################################################
@@ -254,7 +254,7 @@ class GitReview():
         self.__update_changed_files()
 
         if not self.changed_files:
-            msg = ['There are no changes between %s and ' % args.commits[0]]
+            msg = [f'There are no changes between {args.commits[0]} and ']
 
             if args.commits[1]:
                 msg.append(args.commits[1])
@@ -262,7 +262,7 @@ class GitReview():
                 msg.append('local files')
 
             if args.paths:
-                msg.append(' in the %s directory' % args.paths[0])
+                msg.append(f' in the {args.paths[0]} directory')
 
             raise GitReviewError(''.join(msg))
 
@@ -1389,7 +1389,7 @@ def parse_command_line():
             else:
                 colour.error(f'[RED:ERROR]: Multiple commits match {entry}')
         else:
-            colour.error(f'[RED:ERROR] {entry} is not a valid commit ID' % entry)
+            colour.error(f'[RED:ERROR] {entry} is not a valid commit ID')
 
     # Things work easier if we always have two commits to compare
 

skilleter_thingy/multigit.py

@@ -141,6 +141,14 @@ def error(msg, status=1):
 
 ################################################################################
 
+def verbose(args, msg):
+    """Output a message to stderr if running verbosely"""
+
+    if args.verbose:
+        colour.write(f'>>>{msg}', stream=sys.stderr)
+
+################################################################################
+
 def absolute_repo_path(args, relative_path=''):
     """Given a path relative to the multigit configuration file, return
        the absolute path thereto"""
@@ -149,6 +157,14 @@ def absolute_repo_path(args, relative_path=''):
 
 ################################################################################
 
+def relative_repo_path(args, relative_path=''):
+    """Given a path relative to the multigit configuration file, return
+       the relative path from the current directory"""
+
+    return os.path.relpath(absolute_repo_path(args, relative_path))
+
+################################################################################
+
 def find_configuration(default_config_file):
     """If the configuration file name has path elements, try and read it, otherwise
        search up the directory tree looking for the configuration file.
@@ -432,9 +448,13 @@ def mg_dir(args, config, console):
     _ = console
     _ = config
 
+    verbose(args, f'dir: Parameters: {", ".join(args.parameters)}')
+
     if len(args.parameters) > 1:
         error('The +dir command takes no more than one parameter - the name of the working tree to search for')
 
+    # TODO: mg_dir _should_ use these options
+
     if args.modified or args.branched or args.tag or args.subdirectories:
         error('The "--tag", "--modified" "--sub", and "--branched" options cannot be used with the "dir" subcommand')
 
@@ -443,29 +463,25 @@ def mg_dir(args, config, console):
 
     if args.parameters:
         location = []
-        wild_prefix_location = []
         wild_location = []
 
         search_name = args.parameters[0]
 
-        # Search for exact matches, matches if prefixed by '*' or prefixed and suffixed with '*'
-        # unless it already contains '*'
+        # Search for exact matches, or matches that contain the search term if it
+        # doesn't already contain a wildcard
 
         for repo in select_git_repos(args, config):
             if fnmatch.fnmatch(repo['repo name'], search_name):
                 location.append(repo.name)
 
             elif '*' not in search_name:
-                if fnmatch.fnmatch(repo['repo name'], f'*{search_name}'):
-                    wild_prefix_location.append(repo.name)
-
-                elif fnmatch.fnmatch(repo['repo name'], f'*{search_name}*'):
+                if fnmatch.fnmatch(repo['repo name'], f'*{search_name}*'):
                     wild_location.append(repo.name)
 
         # Look for a single exact match, a prefix with '*' match or prefix+suffix
 
         destination = None
-        for destinations in (location, wild_prefix_location, wild_location):
+        for destinations in (location, wild_location):
             if len(destinations) == 1:
                 destination = destinations
                 break
@@ -476,11 +492,7 @@ def mg_dir(args, config, console):
         if not destination:
             error(f'No matches with [BLUE:{search_name}]')
 
-        if len(destination) > 1:
-            dest_list = "\n\t".join([os.path.relpath(d) for d in destination])
-            error(f'Multiple matches with [BLUE:{search_name}]: \n\t{dest_list}')
-
-        colour.write(absolute_repo_path(args, destination[0]))
+        colour.write("\n".join([relative_repo_path(args, d) for d in destination]))
 
     else:
         colour.write(os.path.dirname(args.configuration_file))
@@ -578,82 +590,83 @@ def parse_command_line():
 
     args = Arguments()
 
-    # Expand arguments so that, for instance '-dv' is parsed as '-d -v'
-
-    argv = []
+    # Parse the command line, setting options in the args dataclass appropriately
 
-    for arg in sys.argv:
-        if arg[0] != '-' or arg.startswith('--'):
-            argv.append(arg)
-        else:
-            for c in arg[1:]:
-                argv.append('-' + c)
+    arg_list = sys.argv[1:]
 
-    # Parse the command line, setting options in the args dataclass appropriately
+    while arg_list and arg_list[0].startswith('-'):
+        # Split a parameter into a list, so --x becomes [x] but -xyz becomes [x, y, z]
 
-    i = 1
-    while i < len(argv):
-        param = argv[i]
+        arg_entry = arg_list.pop(0)
+        if arg_entry.startswith('--'):
+            params = [arg_entry[2:]]
+        else:
+            params = list(arg_entry[1:])
 
-        if param in ('--verbose', '-v'):
-            args.verbose = True
+        while params:
+            param = params.pop(0)
 
-        elif param in ('--quiet', '-q'):
-            args.quiet = True
+            if param in ('verbose', 'v'):
+                args.verbose = True
 
-        elif param in ('--config', '-c'):
-            try:
-                i += 1
-                args.default_configuration_file = argv[i]
-            except IndexError:
-                error('--config - missing configuration file parameter')
+            elif param in ('quiet', 'q'):
+                args.quiet = True
 
-        elif param in ('--repos', '-r'):
-            try:
-                i += 1
-                args.repos.append(argv[i])
-            except IndexError:
-                error('--repos - missing repo parameter')
+            elif param in ('config', 'c'):
+                if params:
+                    error('--config - missing configuration file parameter')
+                else:
+                    try:
+                        args.default_configuration_file = arg_list.pop(0)
+                    except IndexError:
+                        error('--config - missing configuration file parameter')
 
-        elif param in ('--tag', '-t'):
-            try:
-                i += 1
-                args.tag.append(argv[i])
-            except IndexError:
-                error('--tag - missing tag parameter')
+            elif param in ('repos', 'r'):
+                if params:
+                    error('--repos - missing repo parameter')
+                try:
+                    args.repos.append(arg_list.pop(0))
+                except IndexError:
+                    error('--repos - missing repo parameter')
 
-        elif param in ('--modified', '-m'):
-            args.modified = True
+            elif param in ('tag', 't'):
+                if params:
+                    error('--tag - missing tag parameter')
+                try:
+                    args.tag.append(arg_list.pop(0))
+                except IndexError:
+                    error('--tag - missing tag parameter')
 
-        elif param in ('--branched', '-b'):
-            args.branched = True
+            elif param in ('modified', 'm'):
+                args.modified = True
 
-        elif param in ('--sub', '-s'):
-            args.subdirectories = True
+            elif param in ('branched', 'b'):
+                args.branched = True
 
-        elif param in ('--continue', '-C'):
-            args.error_continue = True
+            elif param in ('sub', 's'):
+                args.subdirectories = True
 
-        elif param in ('--help', '-h'):
-            colour.write(HELP_INFO)
-            sys.exit(0)
+            elif param in ('continue', 'C'):
+                args.error_continue = True
 
-        elif param[0] == '-':
-            error(f'Invalid option: "{param}"')
-        else:
-            break
+            elif param in ('help', 'h'):
+                colour.write(HELP_INFO)
+                sys.exit(0)
 
-        i += 1
+            else:
+                error(f'Invalid option: "{param}"')
 
     # After the options, we either have a multigit command (prefixed with '+') or a git command
     # followed by parameter
 
     try:
-        if argv[i][0] == '+':
-            args.command = argv[i][1:]
+        command = arg_list.pop(0)
+
+        if command[0] == '+':
+            args.command = command[1:]
             args.internal_command = True
         else:
-            args.command = argv[i]
+            args.command = command
             args.internal_command = False
 
     except IndexError:
@@ -661,7 +674,7 @@ def parse_command_line():
 
     # Save the command parameters
 
-    args.parameters = argv[i+1:]
+    args.parameters = arg_list
 
     # Locate the configuration file
 

{skilleter_thingy-0.1.7.dist-info → skilleter_thingy-0.1.9.dist-info}/METADATA

@@ -1,6 +1,6 @@
-Metadata-Version: 2.2
+Metadata-Version: 2.4
 Name: skilleter_thingy
-Version: 0.1.7
+Version: 0.1.9
 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
@@ -19,6 +19,7 @@ Requires-Dist: pygit2
 Requires-Dist: python-dateutil
 Requires-Dist: requests
 Requires-Dist: filetype
+Dynamic: license-file
 
 # Thingy
 
@@ -103,7 +104,7 @@ Multigit supports a small list of subcommands, each of which are prefixed with a
 
 `+init` - Create or update the configuration file
 
-`+dir` - Given the name of a working tree, output the location within the multigit tree of that working tree if the name matches uniquely, or the name of the directory where the multigit configuration file resides if no parameter is specified.
+`+dir NAME` - Given the name of a working tree, output relative path(s) to one or more matching working trees. If `NAME` does NOT contains wildcard characters (`*` or `?`) then matching takes place as if `*` were prefixed and appended to it, otherwise, the wildcards are used as specified. If `NAME` is not specified then the location of the directory where the multigit configuration file resides is output.
 
 `+config` - Print the name and location of the multigit configuration file.
 

{skilleter_thingy-0.1.7.dist-info → skilleter_thingy-0.1.9.dist-info}/RECORD

@@ -15,7 +15,7 @@ skilleter_thingy/git_common.py,sha256=ZjNkvIBRDGNLFYwOu9FjeqdDKJdm0sndX5QATtmq29
 skilleter_thingy/git_hold.py,sha256=Zk6YUhr08znUOdpVJkJAt0rOtrCXnKE0NHoZzoaEEGo,4616
 skilleter_thingy/git_mr.py,sha256=g33FaRtJTbIQI0tfXv_a042YpGwtbg5fKw072aqjAdw,3086
 skilleter_thingy/git_parent.py,sha256=DLy38mo06R0YDyJWQjrP8pQS-qc7tvLYcn8unKqzEiQ,2774
-skilleter_thingy/git_review.py,sha256=e0yvDD9fIVhscnEP7hY26Od8VUbPEb7iMVMbY63Xjbc,52462
+skilleter_thingy/git_review.py,sha256=Vmepd1qHPM9eHYE19KNO0UWJIYhGhlwPlVBjbgFZ2PM,52448
 skilleter_thingy/git_update.py,sha256=nw-bnDFrMi7yNf7P6gGF8gzmJUjXU5PhWhOYPgApcX4,14350
 skilleter_thingy/git_wt.py,sha256=Xm4VBBXMI4GxNtH3QvF-QL9rtvo4RrlWSbNQDFpGZu8,2954
 skilleter_thingy/gitcmp_helper.py,sha256=wutDHpBPOX8ZsXbsC5ApHxMY5tOKgD3P0eyFVs_mSAI,11214
@@ -24,7 +24,7 @@ skilleter_thingy/gl.py,sha256=9zbGpKxw6lX9RghLkdy-Q5sZlqtbB3uGFO04qTu1dH8,5954
 skilleter_thingy/gphotosync.py,sha256=M0yzt5IxCHezkCOQwz3Chn8oudY_M7PRffboIUFLedk,22477
 skilleter_thingy/linecount.py,sha256=ehTN6VD76i4U5k6dXuYoiqSRHI67_BP-bziklNAJSKY,4309
 skilleter_thingy/moviemover.py,sha256=QzUAWQzQ1AWWREIhl-VMaLo2h8MMhOekBnao5jGWV1s,4470
-skilleter_thingy/multigit.py,sha256=o_gW9vlPJ_uUNRxRYm92kBUl1pQKpt76xoVIBg6XaLo,27814
+skilleter_thingy/multigit.py,sha256=zrn8kIa9gP5WJztWJURz7HNu90XAsecdAWTc2upU-jc,28519
 skilleter_thingy/photodupe.py,sha256=l0hbzSLb2Vk2ceteg-x9fHXCEE1uUuFo84hz5rsZUPA,4184
 skilleter_thingy/phototidier.py,sha256=BOu-cKHMivDlBqlRqv7sL3J6Ix1K2dxWWNcderldyZo,7818
 skilleter_thingy/py_audit.py,sha256=xJm5k5qyeA6ii8mODa4dOkmP8L1drv94UHuxR54RsIM,4384
@@ -60,9 +60,9 @@ skilleter_thingy/thingy/run.py,sha256=6SNKWF01fSxzB10GMU9ajraXYZqAL1w0PXkqjJdr1U
 skilleter_thingy/thingy/tfm_pane.py,sha256=oqy5zBzKwfbjbGqetbbhpKi4x5He7sl4qkmhUeqtdZc,19789
 skilleter_thingy/thingy/tidy.py,sha256=UWpBWuIMCE1UonLJErb41yW3RtpXrK_bt4Z4cZR-eDU,5910
 skilleter_thingy/thingy/venv_template.py,sha256=SsVNvSwojd8NnFeQaZPCRQYTNdwJRplpZpygbUEXRnY,1015
-skilleter_thingy-0.1.7.dist-info/LICENSE,sha256=ljOS4DjXvqEo5VzGfdaRwgRZPbNScGBmfwyC8PChvmQ,32422
-skilleter_thingy-0.1.7.dist-info/METADATA,sha256=Rq3d_ivZOANFFtlWwG-FQo72gVMunqY4QAhKCpZpnKQ,29686
-skilleter_thingy-0.1.7.dist-info/WHEEL,sha256=52BFRY2Up02UkjOa29eZOS2VxUrpPORXg1pkohGGUS8,91
-skilleter_thingy-0.1.7.dist-info/entry_points.txt,sha256=u5ymS-KPljIGTnprV5yJsAjz7qgeT2BZ-Qo_Con_PFM,2145
-skilleter_thingy-0.1.7.dist-info/top_level.txt,sha256=8-JhgToBBiWURunmvfpSxEvNkDHQQ7r25-aBXtZv61g,17
-skilleter_thingy-0.1.7.dist-info/RECORD,,
+skilleter_thingy-0.1.9.dist-info/licenses/LICENSE,sha256=ljOS4DjXvqEo5VzGfdaRwgRZPbNScGBmfwyC8PChvmQ,32422
+skilleter_thingy-0.1.9.dist-info/METADATA,sha256=XJJ2S92lPWLgy0mgs3jGptwlpJaQXE-zPUHD9AwoNU4,29873
+skilleter_thingy-0.1.9.dist-info/WHEEL,sha256=CmyFI0kx5cdEMTLiONQRbGQwjIoR1aIYB7eCAQ4KPJ0,91
+skilleter_thingy-0.1.9.dist-info/entry_points.txt,sha256=u5ymS-KPljIGTnprV5yJsAjz7qgeT2BZ-Qo_Con_PFM,2145
+skilleter_thingy-0.1.9.dist-info/top_level.txt,sha256=8-JhgToBBiWURunmvfpSxEvNkDHQQ7r25-aBXtZv61g,17
+skilleter_thingy-0.1.9.dist-info/RECORD,,

{skilleter_thingy-0.1.7.dist-info → skilleter_thingy-0.1.9.dist-info}/WHEEL

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