git/list[1] front-page[2] threads[3] people[4] search[5] about
 

git silently ignores include directive with single quotes

From
SBStas Bekman <stas@stason.org>
Date
Sep 8, 2018, 18:58 UTC
Message-ID
<ca2b192e-1722-092e-2c54-d79d21a66ba2@stason.org>
Hi,

One of the windows users discovered this bug, and I was able to reproduce it on linux.

We are using a custom content filter configuration REPO/.gitconfig which needs to be enabled inside REPO/.git/config:

This works:
[include]
        path = ../.gitconfig
This doesn’t:
[include]
        path = '../.gitconfig'

Notice the single quotes around the filename. When this is the case git silently (!) ignores the custom configuration, which is clearly a bug.

I found the easiest to debug this is by using:
git config --list --show-origin
In the former case it shows the custom config, in the latter it does not.

Yet, git gives no indication of any errors, not even with GIT_TRACE and other debug vars.

The original problem cropped up due to using:
 git config --local include.path '../.gitconfig'

which on linux stripped the single quotes, but on some windows git bash emulation it kept them.

What am I suggesting is that git:

(1) should complain if it encounters an invalid configuration and not silently ignore it. It took quite some effort and time to figure the culprit.

(2) probably allow the quoted location of the file, but it's much less important, as it's easy to rectify once git gives user #1

I don't have the details about the windows user setup, but I was able to reproduce this bug with git version 2.17.1 on linux.

Thank you.
-- 
________________________________________________
Stas Bekman       <'))))><       <'))))><
https://stasosphere.com  https://chestofbooks.com
https://experientialsexlab.com https://stason.org
https://stasosphere.com/experience-life/my-books
Next: Martin Ågren
Message 1 of 37 in “git silently ignores include directive with single quotes”
  1. Stas BekmanSep 8, 2018
  2. Martin ÅgrenSep 8, 2018
  3. Stas BekmanSep 8, 2018
  4. Stas BekmanSep 8, 2018
  5. Ævar Arnfjörð BjarmasonSep 8, 2018
  6. Stas BekmanSep 8, 2018
  7. Ævar Arnfjörð BjarmasonSep 8, 2018
  8. Stas BekmanSep 8, 2018
  9. Paul SmithSep 9, 2018
  10. Stas BekmanSep 9, 2018
  11. Ævar Arnfjörð BjarmasonSep 8, 2018
  12. Stas BekmanSep 8, 2018
  13. Ævar Arnfjörð BjarmasonSep 8, 2018
  14. Jeff KingSep 8, 2018
  15. Ramsay JonesSep 8, 2018
  16. Jeff KingSep 9, 2018
  17. Junio C HamanoSep 11, 2018
  18. Jeff KingSep 11, 2018
  19. Stas BekmanSep 23, 2018
  20. Ævar Arnfjörð BjarmasonSep 24, 2018
  21. Stas BekmanSep 24, 2018
  22. 0/1 Re: git silently ignores include directive with single quotesPhilip Oakley, Sep 24, 2018
  23. Stas BekmanSep 24, 2018
  24. 1/1 config doc: highlight the name=value syntaxPhilip Oakley, Sep 24, 2018
  25. Junio C HamanoSep 25, 2018
  26. Ævar Arnfjörð BjarmasonSep 8, 2018
  27. Jeff KingSep 9, 2018
  28. Jeff KingSep 8, 2018
  29. Stas BekmanSep 8, 2018
  30. Jeff KingSep 9, 2018
  31. Junio C HamanoSep 10, 2018
  32. Jonathan NiederSep 10, 2018
  33. Junio C HamanoSep 10, 2018
  34. Jonathan NiederSep 10, 2018
  35. Junio C HamanoSep 10, 2018
  36. Stas BekmanSep 10, 2018
  37. Junio C HamanoSep 10, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.