changeset:   95784:0f9c43fb189d
branch:      3.4
parent:      95782:0eb5968c15ad
user:        Christian Heimes <christian@python.org>
date:        Thu Apr 23 11:24:14 2015 +0200
files:       Misc/NEWS Tools/scripts/patchcheck.py
description:
Issue #24031: make patchcheck now supports git checkouts, too.


diff -r 0eb5968c15ad -r 0f9c43fb189d Misc/NEWS
--- a/Misc/NEWS	Thu Apr 23 11:35:59 2015 +0300
+++ b/Misc/NEWS	Thu Apr 23 11:24:14 2015 +0200
@@ -225,6 +225,10 @@
 
 - Issue #24029: Document the name binding behavior for submodule imports.
 
+Tools/Demos
+-----------
+
+- Issue #24031: make patchcheck now supports git checkouts, too.
 
 What's New in Python 3.4.3?
 ===========================
diff -r 0eb5968c15ad -r 0f9c43fb189d Tools/scripts/patchcheck.py
--- a/Tools/scripts/patchcheck.py	Thu Apr 23 11:35:59 2015 +0300
+++ b/Tools/scripts/patchcheck.py	Thu Apr 23 11:24:14 2015 +0200
@@ -49,15 +49,31 @@
 @status("Getting the list of files that have been added/changed",
         info=lambda x: n_files_str(len(x)))
 def changed_files():
-    """Get the list of changed or added files from Mercurial."""
-    if not os.path.isdir(os.path.join(SRCDIR, '.hg')):
-        sys.exit('need a checkout to get modified files')
-
-    cmd = 'hg status --added --modified --no-status'
-    if mq_patches_applied():
-        cmd += ' --rev qparent'
-    with subprocess.Popen(cmd.split(), stdout=subprocess.PIPE) as st:
-        return [x.decode().rstrip() for x in st.stdout]
+    """Get the list of changed or added files from Mercurial or git."""
+    if os.path.isdir(os.path.join(SRCDIR, '.hg')):
+        cmd = 'hg status --added --modified --no-status'
+        if mq_patches_applied():
+            cmd += ' --rev qparent'
+        with subprocess.Popen(cmd.split(), stdout=subprocess.PIPE) as st:
+            return [x.decode().rstrip() for x in st.stdout]
+    elif os.path.isdir(os.path.join(SRCDIR, '.git')):
+        cmd = 'git status --porcelain'
+        filenames = []
+        with subprocess.Popen(cmd.split(), stdout=subprocess.PIPE) as st:
+            for line in st.stdout:
+                line = line.decode().rstrip()
+                status = set(line[:2])
+                # modified, added or unmerged files
+                if not status.intersection('MAU'):
+                    continue
+                filename = line[3:]
+                if ' -> ' in filename:
+                    # file is renamed
+                    filename = filename.split(' -> ', 2)[1].strip()
+                filenames.append(filename)
+        return filenames
+    else:
+        sys.exit('need a Mercurial or git checkout to get modified files')
 
 
 def report_modified_files(file_paths):