changeset:   95785:d1b706e57fbe
parent:      95783:f60f65507d8e
parent:      95784:0f9c43fb189d
user:        Christian Heimes <christian@python.org>
date:        Thu Apr 23 11:25:41 2015 +0200
files:       Misc/NEWS
description:
Issue #24031: make patchcheck now supports git checkouts, too.


diff -r f60f65507d8e -r d1b706e57fbe Misc/NEWS
--- a/Misc/NEWS	Thu Apr 23 11:36:40 2015 +0300
+++ b/Misc/NEWS	Thu Apr 23 11:25:41 2015 +0200
@@ -41,6 +41,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.5.0 alpha 4?
 ===================================
diff -r f60f65507d8e -r d1b706e57fbe Tools/scripts/patchcheck.py
--- a/Tools/scripts/patchcheck.py	Thu Apr 23 11:36:40 2015 +0300
+++ b/Tools/scripts/patchcheck.py	Thu Apr 23 11:25:41 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):