{"thread":{"id":"54960","subject":"[PATCH] doc: remove \"directory cache\" from man pages","startedAt":"2021-01-08T16:55:55Z","lastAt":"2021-01-10T06:59:23Z","messageCount":5,"participants":["Utku Gultopu via GitGitGadget","Taylor Blau","Utku","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"413831","messageId":"pull.943.git.git.1610124896385.gitgitgadget@gmail.com","threadId":"54960","inReplyTo":null,"subject":"[PATCH] doc: remove \"directory cache\" from man pages","fromName":"Utku Gultopu via GitGitGadget","fromEmail":"gitgitgadget@gmail.com","sentAt":"2021-01-08T16:54:56Z","receivedAt":"2021-01-08T16:55:55Z","isPatch":true,"sender":{"key":"name:Utku Gultopu","avatar":null},"body":"From: Utku Gultopu \n\n\"directory cache\" (or \"directory cache index\", \"cache\") are obsolete\nterms which have been superseded by \"index\". Keeping them in the\ndocumentation may be a source of confusion. This commit replaces\nthem with the current term, \"index\", on man pages.\n\nSigned-off-by: Utku Gultopu \n---\n doc: remove \"directory cache\" from man pages\n \n \"directory cache\" (or \"directory cache index\", \"cache\") are obsolete\n terms which have been superseded by \"index\". Keeping them in the\n documentation may be a source of confusion. This commit replaces them\n with the current term, \"index\", on man pages.\n\nPublished-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-943%2Fugultopu%2Fremove-directory-cache-from-man-pages-v1\nFetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-943/ugultopu/remove-directory-cache-from-man-pages-v1\nPull-Request: https://github.com/git/git/pull/943\n\n Documentation/git-ls-files.txt | 5 ++---\n Documentation/git-update-index.txt | 5 ++---\n 2 files changed, 4 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/git-ls-files.txt b/Documentation/git-ls-files.txt\nindex cbcf5263dd0..0a3b5265b34 100644\n--- a/Documentation/git-ls-files.txt\n+++ b/Documentation/git-ls-files.txt\n@@ -23,9 +23,8 @@ SYNOPSIS\n \n DESCRIPTION\n -----------\n-This merges the file listing in the directory cache index with the\n-actual working directory list, and shows different combinations of the\n-two.\n+This merges the file listing in the index with the actual working\n+directory list, and shows different combinations of the two.\n \n One or more of the options below may be used to determine the files\n shown:\ndiff --git a/Documentation/git-update-index.txt b/Documentation/git-update-index.txt\nindex 1489cb09a09..2853f168d97 100644\n--- a/Documentation/git-update-index.txt\n+++ b/Documentation/git-update-index.txt\n@@ -30,9 +30,8 @@ SYNOPSIS\n \n DESCRIPTION\n -----------\n-Modifies the index or directory cache. Each file mentioned is updated\n-into the index and any 'unmerged' or 'needs updating' state is\n-cleared.\n+Modifies the index. Each file mentioned is updated into the index and\n+any 'unmerged' or 'needs updating' state is cleared.\n \n See also linkgit:git-add[1] for a more user-friendly way to do some of\n the most common operations on the index.\n\nbase-commit: 72c4083ddf91b489b7b7b812df67ee8842177d98\n-- \ngitgitgadget\n"},{"id":"413862","messageId":"X/ilPZ71woYI5vT+@nand.local","threadId":"54960","inReplyTo":"pull.943.git.git.1610124896385.gitgitgadget@gmail.com","subject":"Re: [PATCH] doc: remove \"directory cache\" from man pages","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2021-01-08T18:32:29Z","receivedAt":"2021-01-08T18:33:30Z","isPatch":true,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"Hi Utku,\n\nOn Fri, Jan 08, 2021 at 04:54:56PM +0000, Utku Gultopu via GitGitGadget wrote:\n> From: Utku Gultopu \n>\n> \"directory cache\" (or \"directory cache index\", \"cache\") are obsolete\n> terms which have been superseded by \"index\". Keeping them in the\n> documentation may be a source of confusion. This commit replaces\n> them with the current term, \"index\", on man pages.\n\nThis patch looks good to me. There is another mention of the old-style\n\"directory cache\" in the user manual (Documentation/user-manual.txt)\nwhich could probably be removed:\n\n Note that in older documentation you may see the index called the\n \"current directory cache\" or just the \"cache\". It has three important\n properties:\n\nThanks,\nTaylor\n"},{"id":"413869","messageId":"EB7BB287-A18B-45FE-B147-F3CF92FE3114@gmail.com","threadId":"54960","inReplyTo":"X/ilPZ71woYI5vT+@nand.local","subject":"Re: [PATCH] doc: remove \"directory cache\" from man pages","fromName":"Utku","fromEmail":"ugultopu@gmail.com","sentAt":"2021-01-08T19:51:05Z","receivedAt":"2021-01-08T19:51:51Z","isPatch":true,"sender":{"key":"ugultopu@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9937444?v=4"},"body":"Hi Taylor,\n\nI left that one on purpose. I initially thought that\n\"Documentation/user-manual.txt\" was just an outdated, \"dead\" file that\nwasn't being updated anymore and Pro Git 2 was the de-facto user manual\nnow. Hence, I left it there so that it would provide historical context.\n\nHowever, after having a look at the history of it, I realize that it is\nnot a dead file at all. Also a web search for \"git user manual\" returns\nit as the first result, so I understand that it is still very much\nrelevant.\n\nOn the other hand, the document clearly states that the terms \"directory\ncache\" and \"cache\" have been obsoleted by \"index\", so there is no source\nof confusion there. On the man pages, there was, which was the reason I\nfelt the need to make this change. I thought that keeping at least one\ndefinition of \"directory cache\" (where this definition clearly states\nthat it is an obsolete term for \"index\") would be helpful.\n\nSo what are your thoughts about it? If you think it is better to remove\nit, please let me know and I will remove it.\n\nBest\n\n> On Jan 8, 2021, at 1:32 PM, Taylor Blau wrote:\n> \n> Hi Utku,\n> \n> On Fri, Jan 08, 2021 at 04:54:56PM +0000, Utku Gultopu via GitGitGadget wrote:\n>> From: Utku Gultopu \n>> \n>> \"directory cache\" (or \"directory cache index\", \"cache\") are obsolete\n>> terms which have been superseded by \"index\". Keeping them in the\n>> documentation may be a source of confusion. This commit replaces\n>> them with the current term, \"index\", on man pages.\n> \n> This patch looks good to me. There is another mention of the old-style\n> \"directory cache\" in the user manual (Documentation/user-manual.txt)\n> which could probably be removed:\n> \n> Note that in older documentation you may see the index called the\n> \"current directory cache\" or just the \"cache\". It has three important\n> properties:\n> \n> Thanks,\n> Taylor\n\n"},{"id":"413870","messageId":"X/i4EMmpCzFVtDiT@nand.local","threadId":"54960","inReplyTo":"EB7BB287-A18B-45FE-B147-F3CF92FE3114@gmail.com","subject":"Re: [PATCH] doc: remove \"directory cache\" from man pages","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2021-01-08T19:52:48Z","receivedAt":"2021-01-08T19:53:33Z","isPatch":true,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Fri, Jan 08, 2021 at 02:51:05PM -0500, Utku wrote:\n> On the other hand, the document clearly states that the terms \"directory\n> cache\" and \"cache\" have been obsoleted by \"index\", so there is no source\n> of confusion there. On the man pages, there was, which was the reason I\n> felt the need to make this change. I thought that keeping at least one\n> definition of \"directory cache\" (where this definition clearly states\n> that it is an obsolete term for \"index\") would be helpful.\n>\n> So what are your thoughts about it? If you think it is better to remove\n> it, please let me know and I will remove it.\n\nI think that is a good reason to leave it as-is. Since you hadn't\nmentioned it in the patch text, I was wondering whether it was an\nomission, or you had intended to leave it that way.\n\nThis patch looks good to me as-is.\n\nThanks,\nTaylor\n"},{"id":"413954","messageId":"xmqq5z45cm7o.fsf@gitster.c.googlers.com","threadId":"54960","inReplyTo":"X/i4EMmpCzFVtDiT@nand.local","subject":"Re: [PATCH] doc: remove \"directory cache\" from man pages","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-01-10T06:58:19Z","receivedAt":"2021-01-10T06:59:23Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Taylor Blau writes:\n\n> On Fri, Jan 08, 2021 at 02:51:05PM -0500, Utku wrote:\n>> On the other hand, the document clearly states that the terms \"directory\n>> cache\" and \"cache\" have been obsoleted by \"index\", so there is no source\n>> of confusion there. On the man pages, there was, which was the reason I\n>> felt the need to make this change. I thought that keeping at least one\n>> definition of \"directory cache\" (where this definition clearly states\n>> that it is an obsolete term for \"index\") would be helpful.\n>>\n>> So what are your thoughts about it? If you think it is better to remove\n>> it, please let me know and I will remove it.\n>\n> I think that is a good reason to leave it as-is. Since you hadn't\n> mentioned it in the patch text, I was wondering whether it was an\n> omission, or you had intended to leave it that way.\n>\n> This patch looks good to me as-is.\n\nYup, I agree with the reasoning to keep the \"older documentation may\nsay dircache\". Will queue. Thanks.\n"}]}