threads / patch / 9703

patch, 2 partsCleanup tree_status and use -z

Subject: [StGit PATCH 2/2] Cleanup tree_status and use -z

## tl;dr

7 messages between Aug 30, 2007 and Aug 30, 2007. Diffs are folded; open one to read it.

replies: 6people: 2as markdown or json

David Kågedal· Aug 30, 2007, 11:11 UTC · lore

[StGit PATCH 1/1] Add basic test for stg status

Signed-off-by: David KÃ¥gedal <davidk@lysator.liu.se>
---

This is very basic, but there was no test at all that ran stg status, so nothing excercised the code changed in the following patch.

 t/t0002-status.sh |   15 +++++++++++++++
 1 files changed, 15 insertions(+), 0 deletions(-)
 create mode 100755 t/t0002-status.sh
Show changes to t/t0002-status.sh +15 −0
diff --git a/t/t0002-status.sh b/t/t0002-status.sh
new file mode 100755
index 0000000..c088d56
--- /dev/null
+++ b/t/t0002-status.sh
@@ -0,0 +1,15 @@
+#!/bin/sh
+#
+# Copyright (c) 2007 David Kågedal
+#
+
+test_description='Basic stg status
+
+Test that "stg status" works.'
+
+. ./test-lib.sh
+
+test_expect_success 'Run status on empty' \
+  'stg status'
+
+test_done
-- 
1.5.3.rc6.31.g3c3b
David Kågedal· Aug 30, 2007, 11:11 UTC · re: David Kågedal · lore
From: David KÃ¥gedal <david@virtutech.com>

Improved the python code, eliminating temporary variables and using destructuring binds. And use NUL-separation instead of newlines.

Signed-off-by: David KÃ¥gedal <davidk@lysator.liu.se>
---
There are still no good test cases for this code, though.
 stgit/commands/patches.py |    2 +-
 stgit/commands/refresh.py |    2 +-
 stgit/git.py              |   22 +++++++++++-----------
 3 files changed, 13 insertions(+), 13 deletions(-)
Show changes to 3 files +13 −13

stgit/commands/patches.py, stgit/commands/refresh.py, stgit/git.py

diff --git a/stgit/commands/patches.py b/stgit/commands/patches.py
index b3defb6..fb65b62 100644
--- a/stgit/commands/patches.py
+++ b/stgit/commands/patches.py
@@ -51,7 +51,7 @@ def func(parser, options, args):
     """Show the patches modifying a file
     """
     if not args:
-        files = [stat[1] for stat in git.tree_status(verbose = True)]
+        files = [path for (stat,path) in git.tree_status(verbose = True)]
     else:
         files = args
 
diff --git a/stgit/commands/refresh.py b/stgit/commands/refresh.py
index 218075b..f44c58c 100644
--- a/stgit/commands/refresh.py
+++ b/stgit/commands/refresh.py
@@ -121,7 +121,7 @@ def func(parser, options, args):
     else:
         sign_str = None
 
-    files = [x[1] for x in git.tree_status(verbose = True)]
+    files = [path for (stat,path) in git.tree_status(verbose = True)]
     if args:
         files = [f for f in files if f in args]
 
diff --git a/stgit/git.py b/stgit/git.py
index 7962cdb..f315b05 100644
--- a/stgit/git.py
+++ b/stgit/git.py
@@ -168,7 +168,7 @@ def exclude_files():
 
 def tree_status(files = None, tree_id = 'HEAD', unknown = False,
                   noexclude = True, verbose = False, diff_flags = []):
-    """Returns a list of pairs - [status, filename]
+    """Returns a list of pairs - (status, filename)
     """
     if verbose:
         out.start('Checking for changes in the working directory')
@@ -181,16 +181,16 @@ def tree_status(files = None, tree_id = 'HEAD', unknown = False,
 
     # unknown files
     if unknown:
-        if noexclude:
-            exclude = []
-        else:
-            exclude = (['--exclude=%s' % s for s in
-                        ['*.[ao]', '*.pyc', '.*', '*~', '#*', 'TAGS', 'tags']]
-                       + ['--exclude-per-directory=.gitignore']
-                       + ['--exclude-from=%s' % fn for fn in exclude_files()
-                          if os.path.exists(fn)])
-        lines = GRun('git-ls-files', '--others', '--directory', *exclude
-                     ).output_lines()
+        cmd = ['git-ls-files', '-z', '--others', '--directory']
+        if not noexclude:
+            cmd += ['--exclude=%s' % s for s in
+                    ['*.[ao]', '*.pyc', '.*', '*~', '#*', 'TAGS', 'tags']]
+            cmd += ['--exclude-per-directory=.gitignore']
+            cmd += ['--exclude-from=%s' % fn
+                    for fn in exclude_files()
+                    if os.path.exists(fn)]
+
+        lines = GRun(*cmd).raw_output().split('\0')
         cache_files += [('?', line) for line in lines]
 
     # conflicted files
-- 
1.5.3.rc6.31.g3c3b
Karl Hasselström· Aug 30, 2007, 12:11 UTC · re: David Kågedal · lore

Re: [StGit PATCH 2/2] Cleanup tree_status and use -z

On 2007-08-30 13:11:10 +0200, David Kågedal wrote:
> Improved the python code, eliminating temporary variables and using
> destructuring binds. And use NUL-separation instead of newlines.

Good changes, both of them. It all looks OK to me (but I haven't tried to run it yet).

-- 
Karl Hasselström, kha@treskal.com
      www.treskal.com/kalle
David Kågedal· Aug 30, 2007, 11:14 UTC · re: David Kågedal · lore

Re: [StGit PATCH 1/1] Add basic test for stg status

David Kågedal <davidk@lysator.liu.se> writes:
> Signed-off-by: David KÃ¥gedal <davidk@lysator.liu.se>

This was probably confused by the fact that the patch itself contained my name in latin1 encoding.

-- 
David Kågedal
David Kågedal· Aug 30, 2007, 11:34 UTC · re: David Kågedal · lore

Re: [StGit PATCH 1/1] Add basic test for stg status

David Kågedal <davidk@lysator.liu.se> writes:
Show 6 quoted lines
> David Kågedal <davidk@lysator.liu.se> writes:
>
>> Signed-off-by: David KÃ¥gedal <davidk@lysator.liu.se>
>
> This was probably confused by the fact that the patch itself contained
> my name in latin1 encoding.

Or not. It's the same problem as before. I'll stop using git-send-email until I or someone else fixes it.

-- 
David Kågedal
Karl Hasselström· Aug 30, 2007, 12:07 UTC · re: David Kågedal · lore

Re: [StGit PATCH 1/1] Add basic test for stg status

On 2007-08-30 13:34:19 +0200, David Kågedal wrote:
Show 11 quoted lines
> David Kågedal <davidk@lysator.liu.se> writes:
>
> > David Kågedal <davidk@lysator.liu.se> writes:
> >
> > > Signed-off-by: David KÃ¥gedal <davidk@lysator.liu.se>
> >
> > This was probably confused by the fact that the patch itself
> > contained my name in latin1 encoding.
>
> Or not. It's the same problem as before. I'll stop using
> git-send-email until I or someone else fixes it.

Try "stg mail", since you are already using StGIT. It used to have this kind of problem, but it doesn't anymore (at least for me).

-- 
Karl Hasselström, kha@treskal.com
      www.treskal.com/kalle
David Kågedal· Aug 30, 2007, 12:45 UTC · re: Karl Hasselström · lore

Re: [StGit PATCH 1/1] Add basic test for stg status

Karl Hasselström <kha@treskal.com> writes:
Show 16 quoted lines
> On 2007-08-30 13:34:19 +0200, David Kågedal wrote:
>
>> David Kågedal <davidk@lysator.liu.se> writes:
>>
>> > David Kågedal <davidk@lysator.liu.se> writes:
>> >
>> > > Signed-off-by: David KÃ¥gedal <davidk@lysator.liu.se>
>> >
>> > This was probably confused by the fact that the patch itself
>> > contained my name in latin1 encoding.
>>
>> Or not. It's the same problem as before. I'll stop using
>> git-send-email until I or someone else fixes it.
>
> Try "stg mail", since you are already using StGIT. It used to have
> this kind of problem, but it doesn't anymore (at least for me).
Yes, much better. Thank you.
-- 
David Kågedal

← back to recent threads