The error message does not correspond to your code: the code in question would produce SyntaxError, not TypeError.

You don't need shell=True. To get git's output, you could use subprocess.check_output() function:

from subprocess import check_output

date_string = check_output('git log -1 --pretty=format:"%ci"'.split()).decode()

It appears I hadn't 'tokenized' my cmd array appropriately.
Before:

grep_option = '--grep=\'' + grepStr + '\''
format_option = '--format="' + '%H' + '"'
cmd = ['git', 'log', '--all', '-i', grep_option, format_option]

After:

format_option = '--format="' + '%H' + '"'
cmd = ['git', 'log', '--all', '-i', '--grep', grepStr, format_option]

Can it be the two invocations have different environments? Like, different current dirs?

Try this without | wc -l and/or with diagnostic flags to have a better understanding of what exactly gets done.

When you want to parse git output, you should use the form of its commands intended for scripting. In this case it means the --pretty=format:1 option. The %D specifier provides the ref names pointing to the commit, and of course you need to explicitly specify all the fields you want.

You might also want to use the lower level git rev-list command instead of log; it has similar options, but is promised to have stable interface for script use.

Looks like you are manipulating a git log's output, making it a JSON file, then you'll transfer it to some other JSON parser, and found an error there?

Yes, your output is not a valid JSON: As an "array", a bracket wrapping the beginning and end are expected.

See https://stackoverflow.com/a/4600561/9035237 → https://gist.github.com/textarcana/1306223 for a post-processing example. All the code in your mentioned link said this too.

If you are using Python, you may:

output = "[" + output + "]"
output = output.replace("},]", "}]")

However, there are still problems in your format: JSON doesn't accept a line separator inside string, and a " in any field will break the format forever, but these will probably happen in a commit message. So your format should change.

As per https://gist.github.com/varemenos/e95c2e098e657c7688fd?permalink_comment_id=3260906#gistcomment-3260906 says, you can do a hack: use some string that will probably not occur in any field, for example ^^^^, as a temporary quote placeholder, then do any character escaping, for example \n → \\n and \" → \\", and ^^^^ → \" at last. Don't do JSON prettify at this step, hand it up to a JSON formatter.

I think you are passing arguments to Popen the wrong way. If you pass a list then each argument should be a separate list element. That way you don't even have to bother with quotes. So git commit -m 'no message spacing works' should be passed as ['git', 'commit', '-m', 'no message spacing works']. See also the examples in the reference documentation of Popen.

You can get all the repository commits using:

import git
repo = git.Repo("/home/user/.emacs.d")
commits = list(repo.iter_commits("master", max_count=5))

Then you can identify yourself what kind of data gitpython offers:

dir(commits[0])

Some of them are:

• author
• committed_datetime
• hexsha
• message
• stats

Take an example:

>>> commits[0].author
<git.Actor "azzamsa <[email protected]>">

>>> commits[0].hexsha
'fe4326e94eca2e651bf0081bee02172fedaf0b90'

>>> commits[0].message
'Add ocaml mode\n'

>>> commits[0].committed_datetime
datetime.datetime(1970, 1, 1, 0, 0, 0, tzinfo=<git.objects.util.tzoffset object at 0x7fb4fcd01790>)

(committed_datetime outputs datetime object with locale object)

If you want to check if a commit contains a file (which is usable if you want to grab all commit from that file). You can use:

def is_exists(filename, sha):
    """Check if a file in current commit exist."""
    files = repo.git.show("--pretty=", "--name-only", sha)
    if filename in files:
        return True

Then to get all commit from a file:

def get_file_commits(filename):
    file_commits = []
    for commit in commits:
        if is_exists(filename, commit.hexsha):
            file_commits.append(commit)

    return file_commits

e.g I want to take all commits from 'init.el' file:

initel_file_commits = get_file_commits('init.el')

>>> initel_file_commits
[<git.Commit "fe4326e94eca2e651bf0081bee02172fedaf0b90">, <git.Commit
"e4f39891fb484a95ea76e8e07244b908e732e7b3">]

See that the function working correctly:

>>> initel_file_commits[0].stats.files
{'init.el': {'insertions': 1, 'deletions': 0, 'lines': 1}, 'modules/aza-ocaml.el': {'insertions': 28, 'deletions': 0, 'lines': 28}}

>>> initel_file_commits[1].stats.files
{'init.el': {'insertions': 1, 'deletions': 0, 'lines': 1}, 'modules/aza-calfw.el': {'insertions': 65, 'deletions': 0, 'lines': 65}, 'modules/aza-home.el': {'insertions': 0, 'deletions': 57, 'lines': 57}}

Hope it helps.

Seems like the new line is not right, try:

sb.Popen(['git' , 'diff' ,'--name-status' ,test.strip(), 'HEAD'])

Have you considered using GitPython? It's designed to handle all this nonsense for you.

import git  # pip install gitpython


g = git.cmd.Git(git_dir)
g.pull()

To install the module use pip install GitPython

Project can be found here, GitPython

You need to specify the working directory.

Functions Popen, call, check_call, and check_output have a cwd keyword argument to do so, e.g.:

subprocess.call([gitPath] + dirList + ['add','.'], cwd='/home/me/workdir')

See also Specify working directory for popen

Execute the command giving the arguments in an array or in a single string. Like this:

subprocess.check_output(["git", "log", "-1", "--pretty-format:%H"])

Please see https://docs.python.org/2/library/subprocess.html as reference.

UPDATE Edited command according to phd's suggestion, in the comments.