# [PATCH] doc: move git-rev-parse from porcelain to plumbing 4 messages from 2018-10-10 to 2018-10-10. Participants: Daniels Umanovskis, Junio C Hamano. Thread: https://gitlist.dev/t/49532 ## Daniels Umanovskis, 2018-10-10 21:37 Subject: [PATCH] doc: move git-rev-parse from porcelain to plumbing Message-ID: <20181010213726.26111-1-daniels@umanovskis.se> URL: https://gitlist.dev/e/20181010213726.26111-1-daniels%40umanovskis.se ``` git-rev-parse mostly seems like plumbing, and is more usd in scripts than in regular use. Online it's often mentioned as a plumbing command. Nonetheless it's listed under porcelain interrogators in `man git`. It seems appropriate to formally move git-rev-parse to plumbing interrogators. Signed-off-by: Daniels Umanovskis --- command-list.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/command-list.txt b/command-list.txt index c36ea3c18..e6364e167 100644 --- a/command-list.txt +++ b/command-list.txt @@ -153,7 +153,7 @@ git-rerere ancillaryinterrogators git-reset mainporcelain worktree git-revert mainporcelain git-rev-list plumbinginterrogators -git-rev-parse ancillaryinterrogators +git-rev-parse plumbinginterrogators git-rm mainporcelain worktree git-send-email foreignscminterface complete git-send-pack synchingrepositories -- 2.19.1.330.g93276587c.dirty ``` ## Junio C Hamano, 2018-10-10 22:26 Subject: Re: [PATCH] doc: move git-rev-parse from porcelain to plumbing Message-ID: URL: https://gitlist.dev/e/xmqqy3b5jv5p.fsf%40gitster-ct.c.googlers.com In-Reply-To: <20181010213726.26111-1-daniels@umanovskis.se> ``` Daniels Umanovskis writes: > git-rev-parse mostly seems like plumbing, and is more usd in > scripts than in regular use. Online it's often mentioned as > a plumbing command. Nonetheless it's listed under porcelain > interrogators in `man git`. It seems appropriate to formally > move git-rev-parse to plumbing interrogators. Correct. "ancillary" category ended up with full of Porcelain, it seems, but there still are plumbing commands there, and this is a prime example that should not be mixed up with Porcelain commands. Among the remaining ones in the list, cherry and get-tar-commit-id are probably better classified as plumbing. I do not know why cherry is marked for completion; perhaps some crazy people use that on the command line? ``` ## Daniels Umanovskis, 2018-10-10 22:56 Subject: Re: [PATCH] doc: move git-rev-parse from porcelain to plumbing Message-ID: <7c08fd68-9a0a-ab2a-0667-f51726fa691b@umanovskis.se> URL: https://gitlist.dev/e/7c08fd68-9a0a-ab2a-0667-f51726fa691b%40umanovskis.se In-Reply-To: ``` On 10/11/18 12:26 AM, Junio C Hamano wrote: > Among the remaining ones in the list, cherry and get-tar-commit-id > are probably better classified as plumbing. I do not know why > cherry is marked for completion; perhaps some crazy people use that > on the command line? I think cherry could go either way, get-tar-commit-id is definitely plumbing. Would you like me to fix those two on the same patch then? ``` ## Junio C Hamano, 2018-10-10 23:40 Subject: Re: [PATCH] doc: move git-rev-parse from porcelain to plumbing Message-ID: URL: https://gitlist.dev/e/xmqqftxdjrph.fsf%40gitster-ct.c.googlers.com In-Reply-To: <7c08fd68-9a0a-ab2a-0667-f51726fa691b@umanovskis.se> ``` Daniels Umanovskis writes: > On 10/11/18 12:26 AM, Junio C Hamano wrote: >> Among the remaining ones in the list, cherry and get-tar-commit-id >> are probably better classified as plumbing. I do not know why >> cherry is marked for completion; perhaps some crazy people use that >> on the command line? > > I think cherry could go either way, get-tar-commit-id is definitely > plumbing. Would you like me to fix those two on the same patch then? No, what you sent for rev-parse is already good. A separate patch that addresses other ones can be discussed as an orthogonal matter. It even may deserve to make them two separate patches, as I anticipate that some people would resist marking "cherry" as plumbing, so that only one can be applied while dropping the other as/if needed. Thanks. ```