On 04/15/2016 05:45 PM, Scott Kostyshak wrote:
On Tue, Mar 29, 2016 at 06:09:56PM -0400, Scott Kostyshak wrote:
On Tue, Mar 29, 2016 at 10:49:55AM +0200, Jean-Marc Lasgouttes wrote:
Le 28/03/2016 00:04, Scott Kostyshak a écrit :
It seems we currently use "git show ${hash}" in our script. Instead of
doing two commands like I naively suggested above, what about the
following?
What about "git show --stat"?
Yes that is better than what I proposed. We should do
git show --patch-with-stat
so that both the patch and the stat are shown.
Richard are you able to change this? I think it would mean changing the
following in git_show:
p = subprocess.Popen(['git', 'show', hash], stdout=subprocess.PIPE)
to
p = subprocess.Popen(['git', 'show', '--patch-with-stat', hash],
stdout=subprocess.PIPE)
git show is also used in get_commit_info():, but if I understand
correctly we don't need to change it there (I'm not confident about this
though).
I will be able to do so, but I'll need to read this thread through and
understand exactly what we are trying to do.
Richard