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

Re: [RFC] Define "precious" attribute and support it in `git clean`

From
Junio C Hamano <gitster@pobox.com>
Date
Oct 12, 2023, 16:58 UTC
Message-ID
<xmqqttqvg4lw.fsf@gitster.g>
In-Reply-To
<0E44CB2C-57F2-4075-95BE-60FBFDD3CEE2@icloud.com>
Sebastian Thiel <sebastian.thiel@icloud.com> writes:
> ### What about a `$` syntax in `.gitignore` files?
>
> I looked into adding a new prefix, `$` to indicate the following path is
> precious or… valuable. It can be escaped with `\$` just like `\!`. 

I have been regretting that I did not make the quoting syntax not obviously extensible in f87f9497 (git-ls-files: --exclude mechanism updates., 2005-07-24), which technically was a breaking change (as a relative pathname that began with '!' were not special, but after the change, it became necessary to '\'-quote it). A relative pathname that begins with '$' would be now broken the same way, but hopefully the fallout would be minor. I presume you picked '$' exactly because of this reason?

I do not think it will be the end of the world if we don't do so, but it would be really really nice if we at least explored a way (or two) to make a big enough hole in the syntax to not just add "precious", but leave room to later add other traits, without having to worry about breaking the backward compatibility again. A simplest and suboptimal way may be to declare that a path that begins with '$' now needs '\'-quoting (just like your proposal), reserve '$$' as the precious prefix, and '$' followed by any other byte reserved for future use, but there may be better ideas.

> *Unfortunately*, users can't just add a local `.git/info/exclude` file with
> `$.config` in it and expect `.config` to be considered precious as the pattern
> search order will search this last as it's part of the exclude-globals.

That it nothing new and is the same for ignored files. The lower precedence files do not override higher precedence files.

> Thus, to make this work, projects that ship the `.gitignore` files would *have
> to add patterns* that make certain files precious.

Not really. They do not have to do anything if they are content with the current Git ecosystem. And users who have precious stuff can mark them in the.git/info/excludes no? The only case that is problematic is when the project says 'foo' is ignored and expendable but the user thinks otherwise. So to make this work, projects that ship the ".gitignore" files have to avoid adding patterns to ignore things that it may reasonably be expected for its users to mark precious.

> Such opted-in projects would produce `.gitignore` files like these:
>
>     .*
>     $.config
I would understand if you ignored "*~" or "*.o", but why ignore ".*"?
THanks.
Previous: Sebastian ThielNext: Sebastian Thiel
Message 13 of 28 in “[RFC] Define "precious" attribute and support it in `git clean`”
  1. Sebastian ThielOct 10, 2023
  2. Kristoffer HaugsbakkOct 10, 2023
  3. Josh TriplettOct 10, 2023
  4. Junio C HamanoOct 10, 2023
  5. Josh TriplettOct 12, 2023
  6. Kristoffer HaugsbakkOct 10, 2023
  7. Josh TriplettOct 12, 2023
  8. Junio C HamanoOct 10, 2023
  9. Richard KerryOct 11, 2023
  10. Jeff KingOct 11, 2023
  11. Junio C HamanoOct 11, 2023
  12. Sebastian ThielOct 12, 2023
  13. Junio C HamanoOct 12, 2023
  14. Sebastian ThielOct 13, 2023
  15. Junio C HamanoOct 13, 2023
  16. Sebastian ThielOct 14, 2023
  17. Phillip WoodOct 13, 2023
  18. Junio C HamanoOct 14, 2023
  19. Oswald BuddenhagenOct 13, 2023
  20. Josh TriplettOct 14, 2023
  21. Junio C HamanoOct 14, 2023
  22. Elijah NewrenOct 15, 2023
  23. Sebastian ThielOct 15, 2023
  24. Junio C HamanoOct 15, 2023
  25. Sebastian ThielOct 16, 2023
  26. Sebastian ThielOct 23, 2023
  27. Elijah NewrenOct 29, 2023
  28. Kristoffer HaugsbakkOct 11, 2023

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.