Refactoring: move shared options to their own module

[stgit] / stgit / commands / files.py

diff --git a/stgit/commands/files.py b/stgit/commands/files.py
index 59893d842414ee097ada715e84aa3687aae8fc69..cdb0b526f9bdafb739b3894930a257f05e975de0 100644 (file)
--- a/stgit/commands/files.py
+++ b/stgit/commands/files.py
@@ -21,11 +21,11 @@ from optparse import OptionParser, make_option
 
 from stgit.commands.common import *
 from stgit.utils import *
 
 from stgit.commands.common import *
 from stgit.utils import *

-from stgit import stack, git
-
+from stgit.out import *
+from stgit import argparse, stack, git

 
 help = 'show the files modified by a patch (or the current patch)'
 
 help = 'show the files modified by a patch (or the current patch)'

-usage = """%prog [options] [<patch>]
+usage = """%prog [options] [[<branch>:]<patch>]

 
 List the files modified by the given patch (defaulting to the current
 one). Passing the '--stat' option shows the diff statistics for the
 
 List the files modified by the given patch (defaulting to the current
 one). Passing the '--stat' option shows the diff statistics for the


@@ -33,32 +33,32 @@ given patch. Note that this command doesn't show the files modified in
 the working tree and not yet included in the patch by a 'refresh'
 command. Use the 'diff' or 'status' commands for these files."""
 
 the working tree and not yet included in the patch by a 'refresh'
 command. Use the 'diff' or 'status' commands for these files."""
 

+directory = DirectoryHasRepository()

 options = [make_option('-s', '--stat',
                        help = 'show the diff stat',
                        action = 'store_true'),
 options = [make_option('-s', '--stat',
                        help = 'show the diff stat',
                        action = 'store_true'),

-           make_option('-b', '--branch',
-                       help = 'use BRANCH instead of the default one'),

            make_option('--bare',
                        help = 'bare file names (useful for scripting)',
            make_option('--bare',
                        help = 'bare file names (useful for scripting)',

-                       action = 'store_true')]
-
+                       action = 'store_true')
+           ] + argparse.diff_opts_option()

 
 def func(parser, options, args):
     """Show the files modified by a patch (or the current patch)
     """
     if len(args) == 0:
 
 def func(parser, options, args):
     """Show the files modified by a patch (or the current patch)
     """
     if len(args) == 0:

-        patch = ''
+        patch = 'HEAD'

     elif len(args) == 1:
         patch = args[0]
     else:
         parser.error('incorrect number of arguments')
 
     elif len(args) == 1:
         patch = args[0]
     else:
         parser.error('incorrect number of arguments')
 

-    rev1 = git_id('%s//bottom' % patch)
-    rev2 = git_id('%s//top' % patch)
+    rev1 = git_id(crt_series, '%s^' % patch)
+    rev2 = git_id(crt_series, '%s' % patch)

 
     if options.stat:
 
     if options.stat:

-        out.stdout_raw(git.diffstat(rev1 = rev1, rev2 = rev2) + '\n')
+        out.stdout_raw(git.diffstat(git.diff(rev1 = rev1, rev2 = rev2)) + '\n')

     elif options.bare:
         out.stdout_raw(git.barefiles(rev1, rev2) + '\n')
     else:
     elif options.bare:
         out.stdout_raw(git.barefiles(rev1, rev2) + '\n')
     else:

-        out.stdout_raw(git.files(rev1, rev2) + '\n')
+        out.stdout_raw(git.files(rev1, rev2, diff_flags = options.diff_flags)
+                       + '\n')