{"thread":{"id":"39889","subject":"[PATCH 2/6] worktree: remove references to \"git checkout --to\" from the manpage","startedAt":"2015-07-19T02:10:52Z","lastAt":"2015-07-19T03:21:39Z","messageCount":14,"participants":["Michael Haggerty","Eric Sunshine"],"isPatch":true,"patchVersion":1,"patchTotal":6},"messages":[{"id":"266355","messageId":"cover.1437271363.git.mhagger@alum.mit.edu","threadId":"39889","inReplyTo":null,"subject":"[PATCH 0/6] Various tweaks to 'worktree'-related manpages","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2015-07-19T02:10:52Z","receivedAt":"2015-07-19T02:10:52Z","isPatch":true,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"I haven't been following the discussion of this feature on the mailing\nlist, so I apologize if these changes overlap with any in-flight\npatches, and these changes should definitely be checked over by\nsomebody more familiar with the feature. These patches mostly fix what\nseem like inconsistencies in the documentation. The last patch\ncontains some wordsmithing. Feel free to squash them together if you\nprefer.\n\nThis patch series is also available from my GitHub account [1] as\nbranch \"worktree-docs\".\n\n[1] https://github.com/mhagger/git\n\nMichael Haggerty (6):\n worktree: consistently use term \"linked working tree\" in manpages\n worktree: remove references to \"git checkout --to\" from the manpage\n worktree: fix incorrect references to file \"locked\"\n worktree: the \"locked\" mechanism is already implemented\n config.txt: refer to \"git worktree prune\", not \"prune --worktree\"\n worktree: wordsmith worktree-related manpages\n\n Documentation/config.txt | 10 +++++-----\n Documentation/git-worktree.txt | 20 +++++++++-----------\n Documentation/gitrepository-layout.txt | 22 +++++++++++-----------\n 3 files changed, 25 insertions(+), 27 deletions(-)\n\n-- \n2.1.4\n"},{"id":"266354","messageId":"097393a343401706ac8041eb01a9e61949a31d2d.1437271363.git.mhagger@alum.mit.edu","threadId":"39889","inReplyTo":"cover.1437271363.git.mhagger@alum.mit.edu","subject":"[PATCH 1/6] worktree: consistently use term \"linked working tree\" in manpages","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2015-07-19T02:10:53Z","receivedAt":"2015-07-19T02:10:53Z","isPatch":true,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"Sometimes linked working trees were called \"linked working\ndirectories\" or \"linked worktrees\". Always refer to them as \"linked\nworking trees\" for consistency.\n\nSigned-off-by: Michael Haggerty \n---\n Documentation/git-worktree.txt | 10 +++++-----\n Documentation/gitrepository-layout.txt | 8 ++++----\n 2 files changed, 9 insertions(+), 9 deletions(-)\n\ndiff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\nindex da71f50..c8dd0e5 100644\n--- a/Documentation/git-worktree.txt\n+++ b/Documentation/git-worktree.txt\n@@ -31,9 +31,9 @@ The working tree's administrative files in the repository (see\n `git worktree prune` in the main or any linked working tree to\n clean up any stale administrative files.\n \n-If you move a linked working directory to another file system, or\n+If you move a linked working tree to another file system, or\n within a file system that does not support hard links, you need to run\n-at least one git command inside the linked working directory\n+at least one git command inside the linked working tree\n (e.g. `git status`) in order to update its administrative files in the\n repository so that they do not get automatically pruned.\n \n@@ -141,7 +141,7 @@ demands that you fix something immediately. You might typically use\n linkgit:git-stash[1] to store your changes away temporarily, however, your\n worktree is in such a state of disarray (with new, moved, and removed files,\n and other bits and pieces strewn around) that you don't want to risk\n-disturbing any of it. Instead, you create a temporary linked worktree to\n+disturbing any of it. Instead, you create a temporary linked working tree to\n make the emergency fix, remove it when done, and then resume your earlier\n refactoring session.\n \n@@ -163,10 +163,10 @@ recommended to make multiple checkouts of a superproject.\n git-worktree could provide more automation for tasks currently\n performed manually, such as:\n \n-- `remove` to remove a linked worktree and its administrative files (and\n+- `remove` to remove a linked working tree and its administrative files (and\n warn if the worktree is dirty)\n - `mv` to move or rename a worktree and update its administrative files\n-- `list` to list linked worktrees\n+- `list` to list linked working trees\n - `lock` to prevent automatic pruning of administrative files (for instance,\n for a worktree on a portable device)\n \ndiff --git a/Documentation/gitrepository-layout.txt b/Documentation/gitrepository-layout.txt\nindex 7173b38..fe7fa96 100644\n--- a/Documentation/gitrepository-layout.txt\n+++ b/Documentation/gitrepository-layout.txt\n@@ -252,8 +252,8 @@ modules::\n \n worktrees::\n \tContains worktree specific information of linked\n-\tcheckouts. Each subdirectory contains the worktree-related\n-\tpart of a linked checkout. This directory is ignored if\n+\tworking trees. Each subdirectory contains the worktree-related\n+\tpart of a linked working tree. This directory is ignored if\n \t$GIT_COMMON_DIR is set and \"$GIT_COMMON_DIR/worktrees\" will be\n \tused instead.\n \n@@ -265,9 +265,9 @@ worktrees//gitdir::\n \tupdated every time the linked repository is accessed.\n \n worktrees//locked::\n-\tIf this file exists, the linked repository may be on a\n+\tIf this file exists, the linked working tree may be on a\n \tportable device and not available. It does not mean that the\n-\tlinked repository is gone and `worktrees/` could be\n+\tlinked working tree is gone and `worktrees/` could be\n \tremoved. The file's content contains a reason string on why\n \tthe repository is locked.\n \n-- \n2.1.4\n"},{"id":"266350","messageId":"58c91f8907d5f3ef91607e85a96adcf0b1278e14.1437271363.git.mhagger@alum.mit.edu","threadId":"39889","inReplyTo":"cover.1437271363.git.mhagger@alum.mit.edu","subject":"[PATCH 2/6] worktree: remove references to \"git checkout --to\" from the manpage","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2015-07-19T02:10:54Z","receivedAt":"2015-07-19T02:10:54Z","isPatch":true,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"Signed-off-by: Michael Haggerty \n---\n Documentation/git-worktree.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\nindex c8dd0e5..707dfd0 100644\n--- a/Documentation/git-worktree.txt\n+++ b/Documentation/git-worktree.txt\n@@ -18,7 +18,7 @@ DESCRIPTION\n Manage multiple worktrees attached to the same repository.\n \n A git repository can support multiple working trees, allowing you to check\n-out more than one branch at a time. With `git checkout --to` a new working\n+out more than one branch at a time. With `git worktree add`, a new working\n tree is associated with the repository. This new working tree is called a\n \"linked working tree\" as opposed to the \"main working tree\" prepared by \"git\n init\" or \"git clone\". A repository has one main working tree (if it's not a\n@@ -99,7 +99,7 @@ Each linked working tree has a private sub-directory in the repository's\n $GIT_DIR/worktrees directory. The private sub-directory's name is usually\n the base name of the linked working tree's path, possibly appended with a\n number to make it unique. For example, when `$GIT_DIR=/path/main/.git` the\n-command `git checkout --to /path/other/test-next next` creates the linked\n+command `git worktree add /path/other/test-next next` creates the linked\n working tree in `/path/other/test-next` and also creates a\n `$GIT_DIR/worktrees/test-next` directory (or `$GIT_DIR/worktrees/test-next1`\n if `test-next` is already taken).\n-- \n2.1.4\n"},{"id":"266351","messageId":"dfaad2ac35a59b93f1f650d3398c861abd84f289.1437271363.git.mhagger@alum.mit.edu","threadId":"39889","inReplyTo":"cover.1437271363.git.mhagger@alum.mit.edu","subject":"[PATCH 3/6] worktree: fix incorrect references to file \"locked\"","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2015-07-19T02:10:55Z","receivedAt":"2015-07-19T02:10:55Z","isPatch":true,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"The manpage referred to file \"lock\" in a couple of places. The file is\nactually called \"locked\".\n\nSigned-off-by: Michael Haggerty \n---\n Documentation/git-worktree.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\nindex 707dfd0..6cb3877 100644\n--- a/Documentation/git-worktree.txt\n+++ b/Documentation/git-worktree.txt\n@@ -39,7 +39,7 @@ repository so that they do not get automatically pruned.\n \n If a linked working tree is stored on a portable device or network share\n which is not always mounted, you can prevent its administrative files from\n-being pruned by creating a file named 'lock' alongside the other\n+being pruned by creating a file named 'locked' alongside the other\n administrative files, optionally containing a plain text reason that\n pruning should be suppressed. See section \"DETAILS\" for more information.\n \n@@ -167,7 +167,7 @@ performed manually, such as:\n warn if the worktree is dirty)\n - `mv` to move or rename a worktree and update its administrative files\n - `list` to list linked working trees\n-- `lock` to prevent automatic pruning of administrative files (for instance,\n+- `locked` to prevent automatic pruning of administrative files (for instance,\n for a worktree on a portable device)\n \n GIT\n-- \n2.1.4\n"},{"id":"266356","messageId":"afdc2580349017ae26aade63c9e5ea2b4a986beb.1437271363.git.mhagger@alum.mit.edu","threadId":"39889","inReplyTo":"cover.1437271363.git.mhagger@alum.mit.edu","subject":"[PATCH 4/6] worktree: the \"locked\" mechanism is already implemented","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2015-07-19T02:10:56Z","receivedAt":"2015-07-19T02:10:56Z","isPatch":true,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"So remove it from the \"BUGS\" section.\n\nSigned-off-by: Michael Haggerty \n---\n Documentation/git-worktree.txt | 2 --\n 1 file changed, 2 deletions(-)\n\ndiff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\nindex 6cb3877..d5aeda0 100644\n--- a/Documentation/git-worktree.txt\n+++ b/Documentation/git-worktree.txt\n@@ -167,8 +167,6 @@ performed manually, such as:\n warn if the worktree is dirty)\n - `mv` to move or rename a worktree and update its administrative files\n - `list` to list linked working trees\n-- `locked` to prevent automatic pruning of administrative files (for instance,\n- for a worktree on a portable device)\n \n GIT\n ---\n-- \n2.1.4\n"},{"id":"266353","messageId":"47bb6cfeed994694bf333facb5956583da9b934b.1437271363.git.mhagger@alum.mit.edu","threadId":"39889","inReplyTo":"cover.1437271363.git.mhagger@alum.mit.edu","subject":"[PATCH 5/6] config.txt: refer to \"git worktree prune\", not \"prune --worktree\"","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2015-07-19T02:10:57Z","receivedAt":"2015-07-19T02:10:57Z","isPatch":true,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"The latter doesn't exist.\n\nSigned-off-by: Michael Haggerty \n---\n Documentation/config.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex 3e37b93..7f65dbb 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -1293,7 +1293,7 @@ gc.pruneExpire::\n \n gc.pruneWorktreesExpire::\n \tWhen 'git gc' is run, it will call\n-\t'prune --worktrees --expire 3.months.ago'.\n+\t'git worktree prune --expire 3.months.ago'.\n \tOverride the grace period with this config variable. The value\n \t\"now\" may be used to disable the grace period and prune\n \t$GIT_DIR/worktrees immediately.\n-- \n2.1.4\n"},{"id":"266352","messageId":"087f0fbeffa8fb30529ba69498e427d11396eddf.1437271363.git.mhagger@alum.mit.edu","threadId":"39889","inReplyTo":"cover.1437271363.git.mhagger@alum.mit.edu","subject":"[PATCH 6/6] worktree: wordsmith worktree-related manpages","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2015-07-19T02:10:58Z","receivedAt":"2015-07-19T02:10:58Z","isPatch":true,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"Signed-off-by: Michael Haggerty \n---\n Documentation/config.txt | 8 ++++----\n Documentation/git-worktree.txt | 2 +-\n Documentation/gitrepository-layout.txt | 18 +++++++++---------\n 3 files changed, 14 insertions(+), 14 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex 7f65dbb..10bc351 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -1292,11 +1292,11 @@ gc.pruneExpire::\n \tunreachable objects immediately.\n \n gc.pruneWorktreesExpire::\n-\tWhen 'git gc' is run, it will call\n+\tWhen 'git gc' is run, it calls\n \t'git worktree prune --expire 3.months.ago'.\n-\tOverride the grace period with this config variable. The value\n-\t\"now\" may be used to disable the grace period and prune\n-\t$GIT_DIR/worktrees immediately.\n+\tThis config variable can be used to set a different grace\n+\tperiod. The value \"now\" may be used to disable the grace\n+\tperiod and prune $GIT_DIR/worktrees immediately.\n \n gc.reflogExpire::\n gc..reflogExpire::\ndiff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\nindex d5aeda0..cecb44c 100644\n--- a/Documentation/git-worktree.txt\n+++ b/Documentation/git-worktree.txt\n@@ -138,7 +138,7 @@ EXAMPLES\n --------\n You are in the middle of a refactoring session and your boss comes in and\n demands that you fix something immediately. You might typically use\n-linkgit:git-stash[1] to store your changes away temporarily, however, your\n+linkgit:git-stash[1] to store your changes away temporarily; however, your\n worktree is in such a state of disarray (with new, moved, and removed files,\n and other bits and pieces strewn around) that you don't want to risk\n disturbing any of it. Instead, you create a temporary linked working tree to\ndiff --git a/Documentation/gitrepository-layout.txt b/Documentation/gitrepository-layout.txt\nindex fe7fa96..fe7d4bb 100644\n--- a/Documentation/gitrepository-layout.txt\n+++ b/Documentation/gitrepository-layout.txt\n@@ -251,25 +251,25 @@ modules::\n \tContains the git-repositories of the submodules.\n \n worktrees::\n-\tContains worktree specific information of linked\n-\tworking trees. Each subdirectory contains the worktree-related\n+\tContains worktree-specific information for linked working\n+\ttrees. Each subdirectory contains the worktree-related\n \tpart of a linked working tree. This directory is ignored if\n-\t$GIT_COMMON_DIR is set and \"$GIT_COMMON_DIR/worktrees\" will be\n-\tused instead.\n+\t$GIT_COMMON_DIR is set, in which case\n+\t\"$GIT_COMMON_DIR/worktrees\" will be used instead.\n \n worktrees//gitdir::\n \tA text file containing the absolute path back to the .git file\n \tthat points to here. This is used to check if the linked\n \trepository has been manually removed and there is no need to\n-\tkeep this directory any more. mtime of this file should be\n+\tkeep this directory any more. The mtime of this file should be\n \tupdated every time the linked repository is accessed.\n \n worktrees//locked::\n \tIf this file exists, the linked working tree may be on a\n-\tportable device and not available. It does not mean that the\n-\tlinked working tree is gone and `worktrees/` could be\n-\tremoved. The file's content contains a reason string on why\n-\tthe repository is locked.\n+\tportable device and not available. The presence of this file\n+\tprevents `worktrees/` from being pruned by `git worktree\n+\tprune`. The file may contain a string explaining why the\n+\trepository is locked.\n \n worktrees//link::\n \tIf this file exists, it is a hard link to the linked .git\n-- \n2.1.4\n"},{"id":"266357","messageId":"CAPig+cTY4+e3rbDuRRhKajVP+xrBRVr3nZOsQCFiBtYyiQdvqg@mail.gmail.com","threadId":"39889","inReplyTo":"cover.1437271363.git.mhagger@alum.mit.edu","subject":"Re: [PATCH 0/6] Various tweaks to 'worktree'-related manpages","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2015-07-19T02:50:31Z","receivedAt":"2015-07-19T02:50:31Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Sat, Jul 18, 2015 at 10:10 PM, Michael Haggerty wrote:\n> I haven't been following the discussion of this feature on the mailing\n> list, so I apologize if these changes overlap with any in-flight\n> patches, and these changes should definitely be checked over by\n> somebody more familiar with the feature. These patches mostly fix what\n> seem like inconsistencies in the documentation. The last patch\n> contains some wordsmithing. Feel free to squash them together if you\n> prefer.\n\nThanks for the patches. The changes mostly look fine, but see my\nresponses to the individual patches for some comments.\n"},{"id":"266358","messageId":"CAPig+cSYO5FOXpbbA6sTSTFX0TB9mXaNu+1K7nvYHDncoOwJ6g@mail.gmail.com","threadId":"39889","inReplyTo":"097393a343401706ac8041eb01a9e61949a31d2d.1437271363.git.mhagger@alum.mit.edu","subject":"Re: [PATCH 1/6] worktree: consistently use term \"linked working tree\" in manpages","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2015-07-19T02:54:29Z","receivedAt":"2015-07-19T02:54:29Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Sat, Jul 18, 2015 at 10:10 PM, Michael Haggerty wrote:\n> Sometimes linked working trees were called \"linked working\n> directories\" or \"linked worktrees\". Always refer to them as \"linked\n> working trees\" for consistency.\n>\n> Signed-off-by: Michael Haggerty \n> ---\n> diff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\n> index da71f50..c8dd0e5 100644\n> --- a/Documentation/git-worktree.txt\n> +++ b/Documentation/git-worktree.txt\n> @@ -141,7 +141,7 @@ demands that you fix something immediately. You might typically use\n> linkgit:git-stash[1] to store your changes away temporarily, however, your\n> worktree is in such a state of disarray (with new, moved, and removed files,\n\nWas your intention to replace all instances of \"worktree\" with\n\"working tree\" or just some? If only some, what is the criteria by\nwhich you decide? I ask because several instances of \"worktree\" (such\nas the one above) remain even after this patch.\n\n> and other bits and pieces strewn around) that you don't want to risk\n> -disturbing any of it. Instead, you create a temporary linked worktree to\n> +disturbing any of it. Instead, you create a temporary linked working tree to\n> make the emergency fix, remove it when done, and then resume your earlier\n> refactoring session.\n"},{"id":"266359","messageId":"CAPig+cSmRDNCY-k-1cVj7QzSJPP0NiKO+s2i2gLsacQBH8OxuA@mail.gmail.com","threadId":"39889","inReplyTo":"58c91f8907d5f3ef91607e85a96adcf0b1278e14.1437271363.git.mhagger@alum.mit.edu","subject":"Re: [PATCH 2/6] worktree: remove references to \"git checkout --to\" from the manpage","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2015-07-19T02:56:03Z","receivedAt":"2015-07-19T02:56:03Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Sat, Jul 18, 2015 at 10:10 PM, Michael Haggerty wrote:\n> Signed-off-by: Michael Haggerty \n> ---\n> diff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\n> index c8dd0e5..707dfd0 100644\n> --- a/Documentation/git-worktree.txt\n> +++ b/Documentation/git-worktree.txt\n> @@ -18,7 +18,7 @@ DESCRIPTION\n> Manage multiple worktrees attached to the same repository.\n>\n> A git repository can support multiple working trees, allowing you to check\n> -out more than one branch at a time. With `git checkout --to` a new working\n> +out more than one branch at a time. With `git worktree add`, a new working\n> tree is associated with the repository. This new working tree is called a\n> \"linked working tree\" as opposed to the \"main working tree\" prepared by \"git\n> init\" or \"git clone\". A repository has one main working tree (if it's not a\n> @@ -99,7 +99,7 @@ Each linked working tree has a private sub-directory in the repository's\n> $GIT_DIR/worktrees directory. The private sub-directory's name is usually\n> the base name of the linked working tree's path, possibly appended with a\n> number to make it unique. For example, when `$GIT_DIR=/path/main/.git` the\n> -command `git checkout --to /path/other/test-next next` creates the linked\n> +command `git worktree add /path/other/test-next next` creates the linked\n> working tree in `/path/other/test-next` and also creates a\n> `$GIT_DIR/worktrees/test-next` directory (or `$GIT_DIR/worktrees/test-next1`\n> if `test-next` is already taken).\n\nThese fixes are already in 'master': 4d5a3c5\n(Documentation/git-worktree: fix stale \"git checkout --to\" references,\n2015-07-16)\n"},{"id":"266360","messageId":"CAPig+cS4CC8D9JKxw3KSKMZdb_3sWR0C91LL9qcKrF-+RAbfEQ@mail.gmail.com","threadId":"39889","inReplyTo":"dfaad2ac35a59b93f1f650d3398c861abd84f289.1437271363.git.mhagger@alum.mit.edu","subject":"Re: [PATCH 3/6] worktree: fix incorrect references to file \"locked\"","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2015-07-19T03:04:12Z","receivedAt":"2015-07-19T03:04:12Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Sat, Jul 18, 2015 at 10:10 PM, Michael Haggerty wrote:\n> The manpage referred to file \"lock\" in a couple of places. The file is\n> actually called \"locked\".\n>\n> Signed-off-by: Michael Haggerty \n> ---\n> diff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\n> index 707dfd0..6cb3877 100644\n> --- a/Documentation/git-worktree.txt\n> +++ b/Documentation/git-worktree.txt\n> @@ -39,7 +39,7 @@ repository so that they do not get automatically pruned.\n>\n> If a linked working tree is stored on a portable device or network share\n> which is not always mounted, you can prevent its administrative files from\n> -being pruned by creating a file named 'lock' alongside the other\n> +being pruned by creating a file named 'locked' alongside the other\n\nThis change is good.\n\n> administrative files, optionally containing a plain text reason that\n> pruning should be suppressed. See section \"DETAILS\" for more information.\n>\n> @@ -167,7 +167,7 @@ performed manually, such as:\n> warn if the worktree is dirty)\n> - `mv` to move or rename a worktree and update its administrative files\n> - `list` to list linked working trees\n> -- `lock` to prevent automatic pruning of administrative files (for instance,\n> +- `locked` to prevent automatic pruning of administrative files (for instance,\n> for a worktree on a portable device)\n\nThis one is not. This 'lock' is referring to a proposed user-interface\ncommand (\"git worktree lock\") for manipulating the underlying 'locked'\nfile so that the user doesn't have to have detailed implementation\nknowledge or muck with the underlying mechanics manually.\n"},{"id":"266361","messageId":"CAPig+cRSWWnWyV2HJVjS9MkAxv_gLk=KauSbNvw_3k2N_rkuVg@mail.gmail.com","threadId":"39889","inReplyTo":"afdc2580349017ae26aade63c9e5ea2b4a986beb.1437271363.git.mhagger@alum.mit.edu","subject":"Re: [PATCH 4/6] worktree: the \"locked\" mechanism is already implemented","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2015-07-19T03:06:49Z","receivedAt":"2015-07-19T03:06:49Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Sat, Jul 18, 2015 at 10:10 PM, Michael Haggerty wrote:\n> So remove it from the \"BUGS\" section.\n>\n> Signed-off-by: Michael Haggerty \n> ---\n> Documentation/git-worktree.txt | 2 --\n> 1 file changed, 2 deletions(-)\n>\n> diff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\n> index 6cb3877..d5aeda0 100644\n> --- a/Documentation/git-worktree.txt\n> +++ b/Documentation/git-worktree.txt\n> @@ -167,8 +167,6 @@ performed manually, such as:\n> warn if the worktree is dirty)\n> - `mv` to move or rename a worktree and update its administrative files\n> - `list` to list linked working trees\n> -- `locked` to prevent automatic pruning of administrative files (for instance,\n> - for a worktree on a portable device)\n\nThis patch should be dropped. This BUGS item is not referring to the\nlow-level implementation of locking, but rather to the proposed user\ninterface command (\"git worktree lock\") for manipulating the 'locked'\nfile so that the user doesn't need detailed knowledge about the\nunderlying implementation.\n"},{"id":"266362","messageId":"CAPig+cQGT1=97uN1EiAWqw4yaxQJJE-UsvxOQCjLkO2wXCu4xQ@mail.gmail.com","threadId":"39889","inReplyTo":"47bb6cfeed994694bf333facb5956583da9b934b.1437271363.git.mhagger@alum.mit.edu","subject":"Re: [PATCH 5/6] config.txt: refer to \"git worktree prune\", not \"prune --worktree\"","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2015-07-19T03:08:50Z","receivedAt":"2015-07-19T03:08:50Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Sat, Jul 18, 2015 at 10:10 PM, Michael Haggerty wrote:\n> The latter doesn't exist.\n>\n> Signed-off-by: Michael Haggerty \n> ---\n> Documentation/config.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/Documentation/config.txt b/Documentation/config.txt\n> index 3e37b93..7f65dbb 100644\n> --- a/Documentation/config.txt\n> +++ b/Documentation/config.txt\n> @@ -1293,7 +1293,7 @@ gc.pruneExpire::\n>\n> gc.pruneWorktreesExpire::\n> When 'git gc' is run, it will call\n> - 'prune --worktrees --expire 3.months.ago'.\n> + 'git worktree prune --expire 3.months.ago'.\n\nLooks good. This appears to be the last reference to \"prune --worktrees\".\n\n> Override the grace period with this config variable. The value\n> \"now\" may be used to disable the grace period and prune\n> $GIT_DIR/worktrees immediately.\n"},{"id":"266363","messageId":"CAPig+cShJjmDAnwBJ0gzgHb0KwAbNZLLOCfJ2tiGQJZsCQf1gA@mail.gmail.com","threadId":"39889","inReplyTo":"087f0fbeffa8fb30529ba69498e427d11396eddf.1437271363.git.mhagger@alum.mit.edu","subject":"Re: [PATCH 6/6] worktree: wordsmith worktree-related manpages","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2015-07-19T03:21:39Z","receivedAt":"2015-07-19T03:21:39Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Sat, Jul 18, 2015 at 10:10 PM, Michael Haggerty wrote:\n> Signed-off-by: Michael Haggerty \n> ---\n> diff --git a/Documentation/config.txt b/Documentation/config.txt\n> index 7f65dbb..10bc351 100644\n> --- a/Documentation/config.txt\n> +++ b/Documentation/config.txt\n> @@ -1292,11 +1292,11 @@ gc.pruneExpire::\n> unreachable objects immediately.\n>\n> gc.pruneWorktreesExpire::\n> - When 'git gc' is run, it will call\n> + When 'git gc' is run, it calls\n> 'git worktree prune --expire 3.months.ago'.\n\nThis is a tangent, but I wonder if this config variable should be\nrenamed to gc.worktreePruneExpire to reflect the rename of \"prune\n--wortkrees\" to \"worktree prune\".\n\n> - Override the grace period with this config variable. The value\n> - \"now\" may be used to disable the grace period and prune\n> - $GIT_DIR/worktrees immediately.\n> + This config variable can be used to set a different grace\n> + period. The value \"now\" may be used to disable the grace\n> + period and prune $GIT_DIR/worktrees immediately.\n>\n> gc.reflogExpire::\n> gc..reflogExpire::\n> diff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt\n> index d5aeda0..cecb44c 100644\n> --- a/Documentation/git-worktree.txt\n> +++ b/Documentation/git-worktree.txt\n> @@ -138,7 +138,7 @@ EXAMPLES\n> worktrees//locked::\n> If this file exists, the linked working tree may be on a\n> - portable device and not available. It does not mean that the\n> - linked working tree is gone and `worktrees/` could be\n> - removed. The file's content contains a reason string on why\n> - the repository is locked.\n> + portable device and not available. The presence of this file\n> + prevents `worktrees/` from being pruned by `git worktree\n> + prune`. The file may contain a string explaining why the\n> + repository is locked.\n\nWhen I read the new text, I kind of get the impression that pruning\nonly happens manually when the user invokes \"git worktree prune\". I\nwonder if it can be reworded to mention that pruning can happen\nautomatically too. Or, maybe don't mention \"git worktree prune\" at\nall, and instead just say, generally, that it prevents the\nlinked-worktree administrative files from being pruned.\n"}]}