{"thread":{"id":"32335","subject":"[PATCH] Documentation/git-clean: Document --force --force","startedAt":"2012-12-14T01:46:55Z","lastAt":"2012-12-21T17:09:47Z","messageCount":4,"participants":["Soren Brinkmann","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"204864","messageId":"b48ad5f0-25f7-45c3-b2dc-c0c01645a247@CO9EHSMHS031.ehs.local","threadId":"32335","inReplyTo":null,"subject":"[PATCH] Documentation/git-clean: Document --force --force","fromName":"Soren Brinkmann","fromEmail":"soren.brinkmann@xilinx.com","sentAt":"2012-12-14T01:46:55Z","receivedAt":"2012-12-14T01:46:55Z","isPatch":true,"sender":{"key":"soren.brinkmann@xilinx.com","avatar":null},"body":"This patch documents the behavior of 'git clean' when\nencountering nested git repositories.\nSuch repositories are only deleted if '-f' is passed twice\nto 'git clean'.\n\nSigned-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>\n---\n Documentation/git-clean.txt | 5 +++++\n 1 file changed, 5 insertions(+)\n\ndiff --git a/Documentation/git-clean.txt b/Documentation/git-clean.txt\nindex 9f42c0d..0b31454 100644\n--- a/Documentation/git-clean.txt\n+++ b/Documentation/git-clean.txt\n@@ -23,6 +23,9 @@ example, be useful to remove all build products.\n If any optional `<path>...` arguments are given, only those paths\n are affected.\n \n+Nested git repositories are not removed unless the '-f' option is\n+passed to 'git clean' twice.\n+\n OPTIONS\n -------\n -d::\n@@ -35,6 +38,8 @@ OPTIONS\n --force::\n \tIf the git configuration variable clean.requireForce is not set\n \tto false, 'git clean' will refuse to run unless given -f or -n.\n+\tPass this option twice to 'git clean' in order to also remove\n+\tnested git repositories.\n \n -n::\n --dry-run::\n-- \n1.8.0.2\n"},{"id":"205302","messageId":"335802a0-38b5-4cbc-9e52-92c92083119e@VA3EHSMHS005.ehs.local","threadId":"32335","inReplyTo":"b48ad5f0-25f7-45c3-b2dc-c0c01645a247@CO9EHSMHS031.ehs.local","subject":"Re: [PATCH] Documentation/git-clean: Document --force --force","fromName":"Soren Brinkmann","fromEmail":"soren.brinkmann@xilinx.com","sentAt":"2012-12-20T23:37:27Z","receivedAt":"2012-12-20T23:37:27Z","isPatch":true,"sender":{"key":"soren.brinkmann@xilinx.com","avatar":null},"body":"Ping?\n\nOn Thu, Dec 13, 2012 at 05:46:55PM -0800, Soren Brinkmann wrote:\n> This patch documents the behavior of 'git clean' when\n> encountering nested git repositories.\n> Such repositories are only deleted if '-f' is passed twice\n> to 'git clean'.\n> \n> Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>\n> ---\n>  Documentation/git-clean.txt | 5 +++++\n>  1 file changed, 5 insertions(+)\n> \n> diff --git a/Documentation/git-clean.txt b/Documentation/git-clean.txt\n> index 9f42c0d..0b31454 100644\n> --- a/Documentation/git-clean.txt\n> +++ b/Documentation/git-clean.txt\n> @@ -23,6 +23,9 @@ example, be useful to remove all build products.\n>  If any optional `<path>...` arguments are given, only those paths\n>  are affected.\n>  \n> +Nested git repositories are not removed unless the '-f' option is\n> +passed to 'git clean' twice.\n> +\n>  OPTIONS\n>  -------\n>  -d::\n> @@ -35,6 +38,8 @@ OPTIONS\n>  --force::\n>  \tIf the git configuration variable clean.requireForce is not set\n>  \tto false, 'git clean' will refuse to run unless given -f or -n.\n> +\tPass this option twice to 'git clean' in order to also remove\n> +\tnested git repositories.\n>  \n>  -n::\n>  --dry-run::\n> -- \n> 1.8.0.2\n> \n> \n> --\n> To unsubscribe from this list: send the line \"unsubscribe git\" in\n> the body of a message to majordomo@vger.kernel.org\n> More majordomo info at  http://vger.kernel.org/majordomo-info.html\n> \n"},{"id":"205307","messageId":"7vr4mk2jmy.fsf@alter.siamese.dyndns.org","threadId":"32335","inReplyTo":"335802a0-38b5-4cbc-9e52-92c92083119e@VA3EHSMHS005.ehs.local","subject":"Re: [PATCH] Documentation/git-clean: Document --force --force","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-12-21T03:01:41Z","receivedAt":"2012-12-21T03:01:41Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Soren Brinkmann <soren.brinkmann@xilinx.com> writes:\n\n> Ping?\n\nI *think* it is a mistake for the command to remove a separate\nproject repository within, with any number of \"-f\", so I'd rather\nsee a patch to fix it, instead of casting such a misbehaviour as a\nfeature in stone by documenting it.\n\nI dunno.\n\n> On Thu, Dec 13, 2012 at 05:46:55PM -0800, Soren Brinkmann wrote:\n>> This patch documents the behavior of 'git clean' when\n>> encountering nested git repositories.\n>> Such repositories are only deleted if '-f' is passed twice\n>> to 'git clean'.\n>> \n>> Signed-off-by: Soren Brinkmann <soren.brinkmann@xilinx.com>\n>> ---\n>>  Documentation/git-clean.txt | 5 +++++\n>>  1 file changed, 5 insertions(+)\n>> \n>> diff --git a/Documentation/git-clean.txt b/Documentation/git-clean.txt\n>> index 9f42c0d..0b31454 100644\n>> --- a/Documentation/git-clean.txt\n>> +++ b/Documentation/git-clean.txt\n>> @@ -23,6 +23,9 @@ example, be useful to remove all build products.\n>>  If any optional `<path>...` arguments are given, only those paths\n>>  are affected.\n>>  \n>> +Nested git repositories are not removed unless the '-f' option is\n>> +passed to 'git clean' twice.\n>> +\n>>  OPTIONS\n>>  -------\n>>  -d::\n>> @@ -35,6 +38,8 @@ OPTIONS\n>>  --force::\n>>  \tIf the git configuration variable clean.requireForce is not set\n>>  \tto false, 'git clean' will refuse to run unless given -f or -n.\n>> +\tPass this option twice to 'git clean' in order to also remove\n>> +\tnested git repositories.\n>>  \n>>  -n::\n>>  --dry-run::\n>> -- \n>> 1.8.0.2\n>> \n>> \n>> --\n>> To unsubscribe from this list: send the line \"unsubscribe git\" in\n>> the body of a message to majordomo@vger.kernel.org\n>> More majordomo info at  http://vger.kernel.org/majordomo-info.html\n>> \n"},{"id":"205350","messageId":"3f0bbc3d-c605-4741-990f-99ba3c377e6b@CH1EHSMHS025.ehs.local","threadId":"32335","inReplyTo":"7vr4mk2jmy.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Documentation/git-clean: Document --force --force","fromName":"Soren Brinkmann","fromEmail":"soren.brinkmann@xilinx.com","sentAt":"2012-12-21T17:09:47Z","receivedAt":"2012-12-21T17:09:47Z","isPatch":true,"sender":{"key":"soren.brinkmann@xilinx.com","avatar":null},"body":"On Thu, Dec 20, 2012 at 07:01:41PM -0800, Junio C Hamano wrote:\n> Soren Brinkmann <soren.brinkmann@xilinx.com> writes:\n> \n> > Ping?\n> \n> I *think* it is a mistake for the command to remove a separate\n> project repository within, with any number of \"-f\", so I'd rather\n> see a patch to fix it, instead of casting such a misbehaviour as a\n> feature in stone by documenting it.\n> \n> I dunno.\n\nSince I use this feature, I naturally have to disagree. But fair point. And with\nthe patch rectifying git-clean output it's probably tolerable.\n\nThough, what would be your way of cleaning files/dirs from a repository where\ngit-clean will report to have some left undeleted? Manually calling rm -rf?\n\n\tSoren\n"}]}