Add an --index option to "stg refresh"

[stgit] / stgit / commands / refresh.py

diff --git a/stgit/commands/refresh.py b/stgit/commands/refresh.py
index f032375fce6381a891f9695a743bb029fea54eea..4695c6277ec5ea2cda7269b519e4cc5cbfe8b196 100644 (file)
--- a/stgit/commands/refresh.py
+++ b/stgit/commands/refresh.py
@@ -27,17 +27,15 @@ from stgit.config import config
 
 
 help = 'generate a new commit for the current patch'
 
 
 help = 'generate a new commit for the current patch'

-usage = """%prog [options] [<files...>]
+usage = """%prog [options] [<files or dirs>]

 
 Include the latest tree changes in the current patch. This command
 generates a new GIT commit object with the patch details, the previous
 
 Include the latest tree changes in the current patch. This command
 generates a new GIT commit object with the patch details, the previous

-one no longer being visible. The patch attributes like author,
-committer and description can be changed with the command line
-options. The '--force' option is useful when a commit object was
-created with a different tool but the changes need to be included in
-the current patch."""
+one no longer being visible. The '--force' option is useful
+when a commit object was created with a different tool
+but the changes need to be included in the current patch."""

 
 

-directory = DirectoryGotoToplevel()
+directory = DirectoryHasRepository()

 options = [make_option('-f', '--force',
                        help = 'force the refresh even if HEAD and '\
                        'top differ',
 options = [make_option('-f', '--force',
                        help = 'force the refresh even if HEAD and '\
                        'top differ',


@@ -45,6 +43,9 @@ options = [make_option('-f', '--force',
            make_option('--update',
                        help = 'only update the current patch files',
                        action = 'store_true'),
            make_option('--update',
                        help = 'only update the current patch files',
                        action = 'store_true'),

+           make_option('--index',
+                       help = 'use the current contents of the index instead of looking at the working directory',
+                       action = 'store_true'),

            make_option('--undo',
                        help = 'revert the commit generated by the last refresh',
                        action = 'store_true'),
            make_option('--undo',
                        help = 'revert the commit generated by the last refresh',
                        action = 'store_true'),


@@ -55,8 +56,12 @@ options = [make_option('-f', '--force',
            ]
 
 def func(parser, options, args):
            ]
 
 def func(parser, options, args):

-    autoresolved = config.get('stgit.autoresolved')
+    """Generate a new commit for the current or given patch.
+    """
+    args = git.ls_files(args)
+    directory.cd_to_topdir()

 
 

+    autoresolved = config.get('stgit.autoresolved')

     if autoresolved != 'yes':
         check_conflicts()
 
     if autoresolved != 'yes':
         check_conflicts()
 


@@ -72,8 +77,16 @@ def func(parser, options, args):
         if not patch:
             raise CmdException, 'No patches applied'
 
         if not patch:
             raise CmdException, 'No patches applied'
 

+    if options.index:
+        if args or options.update:
+            raise CmdException, \
+                  'Only full refresh is available with the --index option'
+        if options.patch:
+            raise CmdException, \
+                  '--patch is not compatible with the --index option'
+

     if not options.force:
     if not options.force:

-        check_head_top_equal()
+        check_head_top_equal(crt_series)

 
     if options.undo:
         out.start('Undoing the refresh of "%s"' % patch)
 
     if options.undo:
         out.start('Undoing the refresh of "%s"' % patch)


@@ -81,18 +94,17 @@ def func(parser, options, args):
         out.done()
         return
 
         out.done()
         return
 

-    files = [path for (stat,path) in git.tree_status(verbose = True)]
-    if args:
-        files = [f for f in files if f in args]
+    if not options.index:
+        files = [path for (stat, path) in git.tree_status(files = args, verbose = True)]

 
 

-    if files or not crt_series.head_top_equal():
+    if options.index or files or not crt_series.head_top_equal():

         if options.patch:
             applied = crt_series.get_applied()
             between = applied[:applied.index(patch):-1]
         if options.patch:
             applied = crt_series.get_applied()
             between = applied[:applied.index(patch):-1]

-            pop_patches(between, keep = True)
+            pop_patches(crt_series, between, keep = True)

         elif options.update:
         elif options.update:

-            rev1 = git_id('//bottom')
-            rev2 = git_id('//top')
+            rev1 = git_id(crt_series, '//bottom')
+            rev2 = git_id(crt_series, '//top')

             patch_files = git.barefiles(rev1, rev2).split('\n')
             files = [f for f in files if f in patch_files]
             if not files:
             patch_files = git.barefiles(rev1, rev2).split('\n')
             files = [f for f in files if f in patch_files]
             if not files:


@@ -103,8 +115,13 @@ def func(parser, options, args):
 
         if autoresolved == 'yes':
             resolved_all()
 
         if autoresolved == 'yes':
             resolved_all()

-        crt_series.refresh_patch(files = files,
-                                 backup = True, notes = options.annotate)
+
+        if options.index:
+            crt_series.refresh_patch(cache_update = False,
+                                     backup = True, notes = options.annotate)
+        else:
+            crt_series.refresh_patch(files = files,
+                                     backup = True, notes = options.annotate)

 
         if crt_series.empty_patch(patch):
             out.done('empty patch')
 
         if crt_series.empty_patch(patch):
             out.done('empty patch')


@@ -113,7 +130,7 @@ def func(parser, options, args):
 
         if options.patch:
             between.reverse()
 
         if options.patch:
             between.reverse()

-            push_patches(between)
+            push_patches(crt_series, between)

     elif options.annotate:
         # only annotate the top log entry as there is no need to
         # refresh the patch and generate a full commit
     elif options.annotate:
         # only annotate the top log entry as there is no need to
         # refresh the patch and generate a full commit