{"thread":{"id":"3909","subject":"Next problem: empty ident not allowed","startedAt":"2006-04-18T20:25:25Z","lastAt":"2006-04-19T09:51:28Z","messageCount":6,"participants":["Jörn Engel","Junio C Hamano","Linus Torvalds","Petr Baudis"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"18826","messageId":"20060418202525.GD25688@wohnheim.fh-wedel.de","threadId":"3909","inReplyTo":null,"subject":"Next problem: empty ident not allowed","fromName":"Jörn Engel","fromEmail":"joern@wohnheim.fh-wedel.de","sentAt":"2006-04-18T20:25:25Z","receivedAt":"2006-04-18T20:25:25Z","isPatch":false,"sender":{"key":"joern@wohnheim.fh-wedel.de","avatar":null},"body":"After successfully using a central object store with git://... instead\nof rsync://... (thanks to Sam and Linus), I've run into the next\nproblem:\n\n$ env | grep GIT\nGIT_AUTHOR_NAME=Joern Engel\nGIT_OBJECT_DIRECTORY=/home/joern/.git\nGIT_AUTHOR_EMAIL=joern@wh.fh-wedel.de\n\n$ git pull git://git.infradead.org/mtd-2.6.git\nUnpacking 74 objects\n 100% (74/74) done\nTrying really trivial in-index merge...\nWonderful.\nfatal: empty ident not allowed\n\nAdding this to .git/config solved the problem:\n[user]\n\tname = \"Joern Engel\"\n\temail = \"joern@foo.bar\"\n\n\nAnd now I have some questions:\n1. Why didn't the environment variables work?\n2. Why is there a check for commit information when I pull from some\ntree?\n\nJörn\n\n-- \nI've never met a human being who would want to read 17,000 pages of\ndocumentation, and if there was, I'd kill him to get him out of the\ngene pool.\n-- Joseph Costello\n"},{"id":"18827","messageId":"7vu08qei2o.fsf@assigned-by-dhcp.cox.net","threadId":"3909","inReplyTo":"20060418202525.GD25688@wohnheim.fh-wedel.de","subject":"Re: Next problem: empty ident not allowed","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-04-18T20:37:19Z","receivedAt":"2006-04-18T20:37:19Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jörn Engel writes:\n\n> And now I have some questions:\n> 1. Why didn't the environment variables work?\n> 2. Why is there a check for commit information when I pull from some\n> tree?\n\nBecause \"pull\" means \"fetch and merge the local modifications if\nany\". When you merge (and you _did_ merge), you create a new\ncommit of your own, and the commit records who committed it.\n\nYou need GIT_COMMITTER_EMAIL.\n\nNow, the normal \"git log\" does not verbosely show committer and\nauthor,\n"},{"id":"18828","messageId":"7vr73uei1w.fsf@assigned-by-dhcp.cox.net","threadId":"3909","inReplyTo":"20060418202525.GD25688@wohnheim.fh-wedel.de","subject":"Re: Next problem: empty ident not allowed","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-04-18T20:37:47Z","receivedAt":"2006-04-18T20:37:47Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jörn Engel writes:\n\n> And now I have some questions:\n> 1. Why didn't the environment variables work?\n> 2. Why is there a check for commit information when I pull from some\n> tree?\n\nBecause \"pull\" means \"fetch and merge the local modifications if\nany\". When you merge (and you _did_ merge), you create a new\ncommit of your own, and the commit records who committed it.\n\nYou need GIT_COMMITTER_EMAIL.\n"},{"id":"18829","messageId":"20060418204018.GE25688@wohnheim.fh-wedel.de","threadId":"3909","inReplyTo":"7vu08qei2o.fsf@assigned-by-dhcp.cox.net","subject":"Re: Next problem: empty ident not allowed","fromName":"Jörn Engel","fromEmail":"joern@wohnheim.fh-wedel.de","sentAt":"2006-04-18T20:40:18Z","receivedAt":"2006-04-18T20:40:18Z","isPatch":false,"sender":{"key":"joern@wohnheim.fh-wedel.de","avatar":null},"body":"On Tue, 18 April 2006 13:37:19 -0700, Junio C Hamano wrote:\n> Jörn Engel writes:\n> \n> > And now I have some questions:\n> > 1. Why didn't the environment variables work?\n> > 2. Why is there a check for commit information when I pull from some\n> > tree?\n> \n> Because \"pull\" means \"fetch and merge the local modifications if\n> any\". When you merge (and you _did_ merge), you create a new\n> commit of your own, and the commit records who committed it.\n> \n> You need GIT_COMMITTER_EMAIL.\n\nOk. I was stupid in two accounts. Thanks!\n\nJörn\n\n-- \nTime? What's that? Time is only worth what you do with it.\n-- Theo de Raadt\n"},{"id":"18830","messageId":"Pine.LNX.4.64.0604181341000.3701@g5.osdl.org","threadId":"3909","inReplyTo":"20060418202525.GD25688@wohnheim.fh-wedel.de","subject":"Re: Next problem: empty ident not allowed","fromName":"Linus Torvalds","fromEmail":"torvalds@osdl.org","sentAt":"2006-04-18T20:47:23Z","receivedAt":"2006-04-18T20:47:23Z","isPatch":false,"sender":{"key":"torvalds@linux-foundation.org","avatar":"https://avatars.githubusercontent.com/u/1024025?v=4"},"body":"\n\nOn Tue, 18 Apr 2006, Jörn Engel wrote:\n> \n> And now I have some questions:\n> 1. Why didn't the environment variables work?\n\nThey do. But they just described the _author_. Git wanted the _committer_ \nto be described too.\n\nSo if you had used GIT_COMMITTER_NAME/EMAIL, everything would have been \nfine.\n\nNotice that if you don't give any author information, git will assume that \nthe committer is the author, so if you give committer information, you \ndon't actually need to give author information. It \"knows\" that the \ncommitter is you, and since you don't tell it otherwise, it will think \nthat you're the author too.\n\nSo the committer is always the \"default author\".\n\nBut the reverse is _not_ true: even if you tell git who the author is, it \nwill _not_ assume that the committer is the same one. It knows that the \ncommitter is you, but you didn't talk about yourself, you just talked \nabout the author.\n\nThat non-symmetric behaviour does make a lot of sense, if you think about \nit a while. The author is _not_ the \"default committer\". \n\nPutting your name and email in the git config file solves the problem, \nbecause then git knows who the committer is (and by extension, the \n\"default author\").\n\n> 2. Why is there a check for commit information when I pull from some\n> tree?\n\nBecause you're going to create a merge commit. It wants to know who to \nattribute that merge to. Why the committer is, in other words.\n\n\t\tLinus"},{"id":"18867","messageId":"20060419095128.GE27689@pasky.or.cz","threadId":"3909","inReplyTo":"7vr73uei1w.fsf@assigned-by-dhcp.cox.net","subject":"Re: Next problem: empty ident not allowed","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2006-04-19T09:51:28Z","receivedAt":"2006-04-19T09:51:28Z","isPatch":false,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"Dear diary, on Tue, Apr 18, 2006 at 10:37:47PM CEST, I got a letter\nwhere Junio C Hamano said that...\n> Jörn Engel writes:\n> \n> > And now I have some questions:\n> > 1. Why didn't the environment variables work?\n> > 2. Why is there a check for commit information when I pull from some\n> > tree?\n> \n> Because \"pull\" means \"fetch and merge the local modifications if\n> any\". When you merge (and you _did_ merge), you create a new\n> commit of your own, and the commit records who committed it.\n> \n> You need GIT_COMMITTER_EMAIL.\n\nYou mean GIT_COMMITTER_NAME. :-)\n\n-- \n\t\t\t\tPetr \"Pasky\" Baudis\nStuff: http://pasky.or.cz/\nRight now I am having amnesia and deja-vu at the same time. I think\nI have forgotten this before.\n"}]}