Apr 14, 2016 · Apr 13, 2016 · Apr 13, 2016
diff --git a/git/index/base.py b/git/index/base.py

            # resolve globs if possible
            if '?' in path or '*' in path or '[' in path:
                for f in self._iter_expand_paths(glob.glob(abs_path)):
                    yield f.replace(rs, '')
                continue
                resolved_paths = glob.glob(abs_path)
                # not abs_path in resolved_paths:
                #   a glob() resolving to the same path we are feeding it with
                #   is a glob() that failed to resolve. If we continued calling
                #   ourselves we'd endlessly recurse. If the condition below
                #   evaluates to true then we are likely dealing with a file
                #   whose name contains wildcard characters.
                if abs_path not in resolved_paths:
                    for f in self._iter_expand_paths(glob.glob(abs_path)):
                        yield f.replace(rs, '')
                    continue
            # END glob handling
            try:
                for root, dirs, files in os.walk(abs_path, onerror=raise_exc):
diff --git a/git/test/test_index.py b/git/test/test_index.py
        r = Repo.init(rw_dir)
        r.index.add([fp])
        r.index.commit('Added orig and prestable')

    @with_rw_directory
    def test_add_a_file_with_wildcard_chars(self, rw_dir):
        # see issue #407
        fp = os.path.join(rw_dir, '[.exe')
        with open(fp, "wb") as f:
            f.write(b'something')

        r = Repo.init(rw_dir)
        r.index.add([fp])
        r.index.commit('Added [.exe')
Original file line number	Diff line number	Diff line change
Expand Up		@@ -380,9 +380,17 @@ def raise_exc(e):

		# resolve globs if possible
		if '?' in path or '*' in path or '[' in path:
		for f in self._iter_expand_paths(glob.glob(abs_path)):
		yield f.replace(rs, '')
		continue
		resolved_paths = glob.glob(abs_path)
		# not abs_path in resolved_paths:
		# a glob() resolving to the same path we are feeding it with
		# is a glob() that failed to resolve. If we continued calling
		# ourselves we'd endlessly recurse. If the condition below
		# evaluates to true then we are likely dealing with a file
		# whose name contains wildcard characters.
		if abs_path not in resolved_paths:
		for f in self._iter_expand_paths(glob.glob(abs_path)):
		yield f.replace(rs, '')
		continue
		# END glob handling
		try:
		for root, dirs, files in os.walk(abs_path, onerror=raise_exc):
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -796,3 +796,14 @@ def test_add_utf8P_path(self, rw_dir):
		r = Repo.init(rw_dir)
		r.index.add([fp])
		r.index.commit('Added orig and prestable')

		@with_rw_directory
		def test_add_a_file_with_wildcard_chars(self, rw_dir):
		# see issue #407
		fp = os.path.join(rw_dir, '[.exe')
		with open(fp, "wb") as f:
		f.write(b'something')

		r = Repo.init(rw_dir)
		r.index.add([fp])
		r.index.commit('Added [.exe')