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

[PATCH v5 1/8] clarify %f documentation

From
Joey Hess <joeyh@joeyh.name>
Date
Jul 11, 2016, 22:45 UTC
Message-ID
<1468277112-9909-2-git-send-email-joeyh@joeyh.name>
In-Reply-To
<1468277112-9909-1-git-send-email-joeyh@joeyh.name>

It's natural to expect %f to be an actual file on disk; help avoid that mistake.

Signed-off-by: Joey Hess <joeyh@joeyh.name>
---
 Documentation/gitattributes.txt | 5 +++++
 1 file changed, 5 insertions(+)
diff --git a/Documentation/gitattributes.txt b/Documentation/gitattributes.txt
index f2afdb6..197ece8 100644
--- a/Documentation/gitattributes.txt
+++ b/Documentation/gitattributes.txt
@@ -379,6 +379,11 @@ substitution.  For example:
 	smudge = git-p4-filter --smudge %f
 ------------------------
 
+Note that "%f" is the name of the path that is being worked on. Depending
+on the version that is being filtered, the corresponding file on disk may
+not exist, or may have different contents. So, smudge and clean commands
+should not try to access the file on disk, but only act as filters on the
+content provided to them on standard input.
 
 Interaction between checkin/checkout attributes
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-- 
2.8.1
Previous: Joey HessNext: Joey Hess
Message 2 of 11 in “extend smudge/clean filters with direct file access (for pu)”
  1. 0/8 extend smudge/clean filters with direct file access (for pu)Joey Hess, Jul 11, 2016
  2. 1/8 clarify %f documentationJoey Hess, Jul 11, 2016
  3. 3/8 use cleanFromFile in git addJoey Hess, Jul 11, 2016
  4. 2/8 add smudgeToFile and cleanFromFile filter configsJoey Hess, Jul 11, 2016
  5. Lars SchneiderJul 13, 2016
  6. 5/8 warn on unusable smudgeToFile/cleanFromFile configJoey Hess, Jul 11, 2016
  7. 4/8 use smudgeToFile in git checkout etcJoey Hess, Jul 11, 2016
  8. 8/8 use smudgeToFile filter in recursive mergeJoey Hess, Jul 11, 2016
  9. 7/8 use smudgeToFile filter in git amJoey Hess, Jul 11, 2016
  10. 6/8 better recovery from failure of smudgeToFile filterJoey Hess, Jul 11, 2016
  11. Junio C HamanoJul 12, 2016

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.