{"thread":{"id":"120","subject":"[PATCH] provide better committer information to commit-tree.c","startedAt":"2005-04-19T00:11:27Z","lastAt":"2005-04-19T08:18:33Z","messageCount":10,"participants":["Greg KH","Linus Torvalds","David Woodhouse","Petr Baudis","Chris Wedgwood","Russell King"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"736","messageId":"20050419001126.GB21170@kroah.com","threadId":"120","inReplyTo":null,"subject":"[PATCH] provide better committer information to commit-tree.c","fromName":"Greg KH","fromEmail":"greg@kroah.com","sentAt":"2005-04-19T00:11:27Z","receivedAt":"2005-04-19T00:11:27Z","isPatch":true,"sender":{"key":"greg@kroah.com","avatar":"https://gravatar.com/avatar/5bb5aa0cc2e01c00ec899d11130c07796bc186e465bae57bc34873b13b72c7c8?d=mp&s=160"},"body":"Here's a small patch to commit-tree.c that does two things:\n\t- allows the committer email address and name to be overridden\n\t  by environment variables (if you don't like the environment\n\t  variable names I've used (COMMIT_AUTHOR_NAME,\n\t  COMMIT_AUTHOR_EMAIL), feel free to change them.)\n\t- provide the proper domainname to the author/committer email\n\t  address (otherwise, my address was only showing up as from the\n\t  hostname.)\n\nThis allows people to set sane values for the commit names and email\naddresses, preventing odd, private hostnames and domains from being\nexposed to the world.\n\nSigned-off-by: Greg Kroah-Hartman <gregkh@suse.de>\n\n--- a/commit-tree.c.orig\t2005-04-18 17:02:50.000000000 -0700\n+++ b/commit-tree.c\t2005-04-18 17:03:19.000000000 -0700\n@@ -284,8 +284,8 @@\n \tunsigned char tree_sha1[20];\n \tunsigned char parent_sha1[MAXPARENT][20];\n \tunsigned char commit_sha1[20];\n-\tchar *gecos, *realgecos;\n-\tchar *email, realemail[1000];\n+\tchar *gecos, *realgecos, *commitgecos;\n+\tchar *email, *commitemail, realemail[1000];\n \tchar date[20], realdate[20];\n \tchar *audate;\n \tchar comment[1000];\n@@ -317,20 +317,24 @@\n \tmemcpy(realemail, pw->pw_name, len);\n \trealemail[len] = '@';\n \tgethostname(realemail+len+1, sizeof(realemail)-len-1);\n+\tstrcat(realemail, \".\");\n+\tgetdomainname(realemail+strlen(realemail), sizeof(realemail)-strlen(realemail)-1);\n \ttime(&now);\n \ttm = localtime(&now);\n \n \tstrftime(realdate, sizeof(realdate), \"%s %z\", tm);\n \tstrcpy(date, realdate);\n \n+\tcommitgecos = getenv(\"COMMIT_AUTHOR_NAME\") ? : realgecos;\n+\tcommitemail = getenv(\"COMMIT_AUTHOR_EMAIL\") ? : realemail;\n \tgecos = getenv(\"AUTHOR_NAME\") ? : realgecos;\n \temail = getenv(\"AUTHOR_EMAIL\") ? : realemail;\n \taudate = getenv(\"AUTHOR_DATE\");\n \tif (audate)\n \t\tparse_rfc2822_date(audate, date, sizeof(date));\n \n-\tremove_special(gecos); remove_special(realgecos);\n-\tremove_special(email); remove_special(realemail);\n+\tremove_special(gecos); remove_special(realgecos); remove_special(commitgecos);\n+\tremove_special(email); remove_special(realemail); remove_special(commitemail);\n \n \tinit_buffer(&buffer, &size);\n \tadd_buffer(&buffer, &size, \"tree %s\\n\", sha1_to_hex(tree_sha1));\n@@ -345,7 +349,7 @@\n \n \t/* Person/date information */\n \tadd_buffer(&buffer, &size, \"author %s <%s> %s\\n\", gecos, email, date);\n-\tadd_buffer(&buffer, &size, \"committer %s <%s> %s\\n\\n\", realgecos, realemail, realdate);\n+\tadd_buffer(&buffer, &size, \"committer %s <%s> %s\\n\\n\", commitgecos, commitemail, realdate);\n \n \t/* And add the comment */\n \twhile (fgets(comment, sizeof(comment), stdin) != NULL)\n"},{"id":"740","messageId":"Pine.LNX.4.58.0504181730130.15725@ppc970.osdl.org","threadId":"120","inReplyTo":"20050419001126.GB21170@kroah.com","subject":"Re: [PATCH] provide better committer information to commit-tree.c","fromName":"Linus Torvalds","fromEmail":"torvalds@osdl.org","sentAt":"2005-04-19T00:31:16Z","receivedAt":"2005-04-19T00:31:16Z","isPatch":true,"sender":{"key":"torvalds@linux-foundation.org","avatar":"https://avatars.githubusercontent.com/u/1024025?v=4"},"body":"\n\nOn Mon, 18 Apr 2005, Greg KH wrote:\n>\n> Here's a small patch to commit-tree.c that does two things:\n\nGaah, I really was hoping that people wouldn't feel like they have to lie \nabout their committer information.\n\nI guess we don't have much choice, but I'm not happy about it.\n\n\t\tLinus\n"},{"id":"743","messageId":"20050419004548.GA21623@kroah.com","threadId":"120","inReplyTo":"Pine.LNX.4.58.0504181730130.15725@ppc970.osdl.org","subject":"Re: [PATCH] provide better committer information to commit-tree.c","fromName":"Greg KH","fromEmail":"greg@kroah.com","sentAt":"2005-04-19T00:45:48Z","receivedAt":"2005-04-19T00:45:48Z","isPatch":true,"sender":{"key":"greg@kroah.com","avatar":"https://gravatar.com/avatar/5bb5aa0cc2e01c00ec899d11130c07796bc186e465bae57bc34873b13b72c7c8?d=mp&s=160"},"body":"On Mon, Apr 18, 2005 at 05:31:16PM -0700, Linus Torvalds wrote:\n> \n> \n> On Mon, 18 Apr 2005, Greg KH wrote:\n> >\n> > Here's a small patch to commit-tree.c that does two things:\n> \n> Gaah, I really was hoping that people wouldn't feel like they have to lie \n> about their committer information.\n> \n> I guess we don't have much choice, but I'm not happy about it.\n\nWell Russell has stated that he has to for EU Privacy reasons.  And I'd\nlike to do it as I don't have a local suse.de hostname for my laptop and\nmy employer probably doesn't really want my greg@dunce.kroah.org address\nshowing up :)\n\nBut if you really don't like it, and you don't want anyone trying to\nhide anything, at least allow for a proper domainname.  On my boxes, the\ndomainname doesn't show up at all without that patch (just the\ngetdomainname() part).  I'll split it out if you want.\n\nthanks,\n\ngreg k-h\n"},{"id":"746","messageId":"1113871951.3579.25.camel@localhost.localdomain","threadId":"120","inReplyTo":"20050419004548.GA21623@kroah.com","subject":"Re: [PATCH] provide better committer information to commit-tree.c","fromName":"David Woodhouse","fromEmail":"dwmw2@infradead.org","sentAt":"2005-04-19T00:52:30Z","receivedAt":"2005-04-19T00:52:30Z","isPatch":true,"sender":{"key":"dwmw2@infradead.org","avatar":"https://gravatar.com/avatar/7afd4f07e0cf7d7e046ae2d23678296b37777c96488e6f3451e78a5514154ebd?d=mp&s=160"},"body":"On Mon, 2005-04-18 at 17:45 -0700, Greg KH wrote:\n> Well Russell has stated that he has to for EU Privacy reasons.  And I'd\n> like to do it as I don't have a local suse.de hostname for my laptop and\n> my employer probably doesn't really want my greg@dunce.kroah.org address\n> showing up :)\n\nWhy not? Do they complain that we see 'greg@dunce.kroah.org' when you\nconnect to an IRC server? This _isn't_ an email address, and doesn't\nreally need to be treated as such. \n\n-- \ndwmw2\n\n"},{"id":"751","messageId":"20050419011236.GA22091@kroah.com","threadId":"120","inReplyTo":"1113871951.3579.25.camel@localhost.localdomain","subject":"Re: [PATCH] provide better committer information to commit-tree.c","fromName":"Greg KH","fromEmail":"greg@kroah.com","sentAt":"2005-04-19T01:12:37Z","receivedAt":"2005-04-19T01:12:37Z","isPatch":true,"sender":{"key":"greg@kroah.com","avatar":"https://gravatar.com/avatar/5bb5aa0cc2e01c00ec899d11130c07796bc186e465bae57bc34873b13b72c7c8?d=mp&s=160"},"body":"On Tue, Apr 19, 2005 at 10:52:30AM +1000, David Woodhouse wrote:\n> On Mon, 2005-04-18 at 17:45 -0700, Greg KH wrote:\n> > Well Russell has stated that he has to for EU Privacy reasons.  And I'd\n> > like to do it as I don't have a local suse.de hostname for my laptop and\n> > my employer probably doesn't really want my greg@dunce.kroah.org address\n> > showing up :)\n> \n> Why not? Do they complain that we see 'greg@dunce.kroah.org' when you\n> connect to an IRC server?\n\nNo, this employer seems quite sane :)\nHowever, irc connections are as \"real\" as a kernel changelog message to\na lot of people.\n\n> This _isn't_ an email address, and doesn't really need to be treated\n> as such. \n\nOk, then why display it as one?  Seriously, it doesn't really bother me,\njust added it as I thought others would need it.  I know the \"shortlog\"\nprogram can pick the real names out of the Signed-off-by: lines.\n\nBut I'll wait for Russell to wake up and start quoting the proper EU\nprivacy laws that he feels causes him to be forced to obfuscate his\nemail addresses in the changelog commits (as he did for the bk ones.)\n\nthanks,\n\ngreg k-h\n"},{"id":"752","messageId":"20050419011542.GV5554@pasky.ji.cz","threadId":"120","inReplyTo":"20050419011236.GA22091@kroah.com","subject":"Re: [PATCH] provide better committer information to commit-tree.c","fromName":"Petr Baudis","fromEmail":"pasky@ucw.cz","sentAt":"2005-04-19T01:15:42Z","receivedAt":"2005-04-19T01:15:42Z","isPatch":true,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"Dear diary, on Tue, Apr 19, 2005 at 03:12:37AM CEST, I got a letter\nwhere Greg KH <greg@kroah.com> told me that...\n> But I'll wait for Russell to wake up and start quoting the proper EU\n> privacy laws that he feels causes him to be forced to obfuscate his\n> email addresses in the changelog commits (as he did for the bk ones.)\n\nI'd be actually pretty interested in knowing about such laws; I think I\nmissed it on LKML when it went through. And after all, being an EU\ncitizen myself...\n\n-- \n\t\t\t\tPetr \"Pasky\" Baudis\nStuff: http://pasky.or.cz/\nC++: an octopus made by nailing extra legs onto a dog. -- Steve Taylor\n"},{"id":"754","messageId":"1113873634.3579.32.camel@localhost.localdomain","threadId":"120","inReplyTo":"20050419011236.GA22091@kroah.com","subject":"Re: [PATCH] provide better committer information to commit-tree.c","fromName":"David Woodhouse","fromEmail":"dwmw2@infradead.org","sentAt":"2005-04-19T01:20:32Z","receivedAt":"2005-04-19T01:20:32Z","isPatch":true,"sender":{"key":"dwmw2@infradead.org","avatar":"https://gravatar.com/avatar/7afd4f07e0cf7d7e046ae2d23678296b37777c96488e6f3451e78a5514154ebd?d=mp&s=160"},"body":"On Mon, 2005-04-18 at 18:12 -0700, Greg KH wrote:\n> Ok, then why display it as one? \n\nNobody ever displays it as one as far as I'm aware. That would be\nsomething like \"mailto:$COMMITTER\"\n\n> But I'll wait for Russell to wake up and start quoting the proper EU\n> privacy laws that he feels causes him to be forced to obfuscate his\n> email addresses in the changelog commits (as he did for the bk ones.)\n\nHe's talking about his own interpretation of the UK's Data Protection\nAct, which requires you to be registered and fulfil certain other\nrequirements if you keep personal information about people in a\ndatabase. Email addresses have been ruled to be 'personal information'\nin this context, but this _isn't_ an email address -- and there are\nother get-out clauses for noncommercial situations such as this anyway,\nI believe. \n\nBesides, he can still obscure the author information as he unfortunately\ninsists on doing; it's the _committer_ information which we're\ndiscussing here -- and that's always going to be himself in this case.\n\n-- \ndwmw2\n\n"},{"id":"759","messageId":"20050419014738.GA13602@taniwha.stupidest.org","threadId":"120","inReplyTo":"20050419004548.GA21623@kroah.com","subject":"Re: [PATCH] provide better committer information to commit-tree.c","fromName":"Chris Wedgwood","fromEmail":"cw@f00f.org","sentAt":"2005-04-19T01:47:38Z","receivedAt":"2005-04-19T01:47:38Z","isPatch":true,"sender":{"key":"cw@f00f.org","avatar":null},"body":"On Mon, Apr 18, 2005 at 05:45:48PM -0700, Greg KH wrote:\n\n> But if you really don't like it, and you don't want anyone trying to\n> hide anything, at least allow for a proper domainname.  On my boxes,\n> the domainname doesn't show up at all without that patch (just the\n> getdomainname() part).  I'll split it out if you want.\n\nthere are plenty of times you need this... internal domain names\nusually have the MTA rewrite addresses as needed, i don't see how this\nis all the different\n\nwe have internal/private state that's not globally useful and need to\nreplace it with something that is, how else could we do this?\n"},{"id":"773","messageId":"Pine.LNX.4.58.0504181949540.15725@ppc970.osdl.org","threadId":"120","inReplyTo":"20050419014738.GA13602@taniwha.stupidest.org","subject":"Re: [PATCH] provide better committer information to commit-tree.c","fromName":"Linus Torvalds","fromEmail":"torvalds@osdl.org","sentAt":"2005-04-19T02:51:46Z","receivedAt":"2005-04-19T02:51:46Z","isPatch":true,"sender":{"key":"torvalds@linux-foundation.org","avatar":"https://avatars.githubusercontent.com/u/1024025?v=4"},"body":"\n\nOn Mon, 18 Apr 2005, Chris Wedgwood wrote:\n>\n> On Mon, Apr 18, 2005 at 05:45:48PM -0700, Greg KH wrote:\n> \n> > But if you really don't like it, and you don't want anyone trying to\n> > hide anything, at least allow for a proper domainname.  On my boxes,\n> > the domainname doesn't show up at all without that patch (just the\n> > getdomainname() part).  I'll split it out if you want.\n> \n> there are plenty of times you need this... internal domain names\n> usually have the MTA rewrite addresses as needed, i don't see how this\n> is all the different\n> \n> we have internal/private state that's not globally useful and need to\n> replace it with something that is, how else could we do this?\n\nGuys, I ended up applying Greg's patch, since it's clear that many people \nwant to do this.  So ..\n\n\t\tLinus\n"},{"id":"791","messageId":"20050419091833.C13488@flint.arm.linux.org.uk","threadId":"120","inReplyTo":"20050419004548.GA21623@kroah.com","subject":"Re: [PATCH] provide better committer information to commit-tree.c","fromName":"Russell King","fromEmail":"rmk@arm.linux.org.uk","sentAt":"2005-04-19T08:18:33Z","receivedAt":"2005-04-19T08:18:33Z","isPatch":true,"sender":{"key":"rmk@arm.linux.org.uk","avatar":null},"body":"On Mon, Apr 18, 2005 at 05:45:48PM -0700, Greg KH wrote:\n> On Mon, Apr 18, 2005 at 05:31:16PM -0700, Linus Torvalds wrote:\n> > \n> > \n> > On Mon, 18 Apr 2005, Greg KH wrote:\n> > >\n> > > Here's a small patch to commit-tree.c that does two things:\n> > \n> > Gaah, I really was hoping that people wouldn't feel like they have to lie \n> > about their committer information.\n> > \n> > I guess we don't have much choice, but I'm not happy about it.\n> \n> Well Russell has stated that he has to for EU Privacy reasons.\n\nThat's author information, not committer information.  For my committing\npurposes, I'm going to be the one doing the commit, and I'm unlikely to\nissue a suit on myself for spreading my own personal information.\n\n-- \nRussell King\n\n"}]}