Use a smarter way to detect git repositories about setuptools_scm HOT 2 CLOSED

Replacing

#!python

if not os.path.exists(os.path.join(root, '.git')):
        return

with

#!python


    from subprocess import call, STDOUT
    import os
    if call(["git", "status"], stderr=STDOUT, stdout=open(os.devnull, 'w')) != 0:
        return

should be an appropriate fix.

Original comment by: Amit Ambasta

from setuptools_scm.

the intent is to quickly skip if one is not in a git related directory at all

if there is a .git folder but it invalid it will bail out as the revparse as intended

so unless there is something huge i have overseen the check works as intended right now

feel free to reopen with an direct example detailing why the check should be done as you lined out (its perfectly valid, but invokes a fork+exec instead of a stat which is what i wanted to avoid)

Original comment by: Ronny Pfannschmidt

from setuptools_scm.