{"thread":{"id":"11651","subject":"git gui and commit-msg hook","startedAt":"2008-01-16T23:13:03Z","lastAt":"2008-01-21T04:39:47Z","messageCount":5,"participants":["Blucher, Guy","Shawn O. Pearce","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"65626","messageId":"054F21930D24A0428E5B4588462C7AEDC5F8DC@ednex512.dsto.defence.gov.au","threadId":"11651","inReplyTo":"054F21930D24A0428E5B4588462C7AEDC5F8DB@ednex512.dsto.defence.gov.au","subject":"git gui and commit-msg hook","fromName":"Blucher, Guy","fromEmail":"guy.blucher@dsto.defence.gov.au","sentAt":"2008-01-16T23:13:03Z","receivedAt":"2008-01-16T23:13:03Z","isPatch":false,"sender":{"key":"guy.blucher@dsto.defence.gov.au","avatar":null},"body":"Hi,\n\nIs the commit-msg hook supposed to work when using git gui to do a\ncommit?\n\nI've just setup a commit-msg hook in my local repository. It works as\nadvertised from the command-line when using git commit, but if I use git\ngui to do the commit, then the hook is ignored.\n\nI also have a pre-commit hook in my repository which runs fine when\nusing 'git gui' or 'git commit'. \n\nAll I want to do is verify that the text of a commit message follows\nsome \"corporate standard\", perhaps I can do this using the pre-commit\nhook instead, but commit-msg seems to be the best fit? \n\nRegards,\nGuy.\n\nIMPORTANT: This email remains the property of the Australian Defence Organisation and is subject to the jurisdiction of section 70 of the CRIMES ACT 1914. If you have received this email in error, you are requested to contact the sender and delete the email.\n"},{"id":"65820","messageId":"20080118041230.GA24004@spearce.org","threadId":"11651","inReplyTo":"054F21930D24A0428E5B4588462C7AEDC5F8DC@ednex512.dsto.defence.gov.au","subject":"Re: git gui and commit-msg hook","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-01-18T04:12:30Z","receivedAt":"2008-01-18T04:12:30Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"\"Blucher, Guy\" wrote:\n> Is the commit-msg hook supposed to work when using git gui to do a\n> commit?\n\nUhhh. Whoops. ;-)\n\ngit-gui doesn't implement the commit-msg hook. Because I forgot\nto implement it. So yes, its supposed to work, but no, it doesn't,\nbecause it was never coded. :-)\n \n> All I want to do is verify that the text of a commit message follows\n> some \"corporate standard\", perhaps I can do this using the pre-commit\n> hook instead, but commit-msg seems to be the best fit? \n\nYes, commit-msg is the proper hook to use here. But git-gui doesn't\nhonor it. I'll try to work out a fix after dinner this evening.\n\nThanks for the bug report. Definately appreciated.\n\n-- \nShawn.\n"},{"id":"66070","messageId":"20080121040819.GE24004@spearce.org","threadId":"11651","inReplyTo":"054F21930D24A0428E5B4588462C7AEDC5F8DC@ednex512.dsto.defence.gov.au","subject":"Re: git gui and commit-msg hook","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-01-21T04:08:19Z","receivedAt":"2008-01-21T04:08:19Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"\"Blucher, Guy\" wrote:\n> I've just setup a commit-msg hook in my local repository. It works as\n> advertised from the command-line when using git commit, but if I use git\n> gui to do the commit, then the hook is ignored.\n\nFixed in my latest git-gui master, which will be gitgui-0.9.2 or .3.\n\nI apparently missed git 1.5.4-rc4, but will have to ask Junio nicely\nif this can be included into 1.5.4 final.\n\n-- \nShawn.\n"},{"id":"66072","messageId":"7vmyqz3ijb.fsf@gitster.siamese.dyndns.org","threadId":"11651","inReplyTo":"20080121040819.GE24004@spearce.org","subject":"Re: git gui and commit-msg hook","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-01-21T04:32:40Z","receivedAt":"2008-01-21T04:32:40Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Shawn O. Pearce\" writes:\n\n> \"Blucher, Guy\" wrote:\n>> I've just setup a commit-msg hook in my local repository. It works as\n>> advertised from the command-line when using git commit, but if I use git\n>> gui to do the commit, then the hook is ignored.\n>\n> Fixed in my latest git-gui master, which will be gitgui-0.9.2 or .3.\n>\n> I apparently missed git 1.5.4-rc4, but will have to ask Junio nicely\n> if this can be included into 1.5.4 final.\n\nAbsolutely. That's a consistency-and-usability bugfix, isn't\nit?\n"},{"id":"66075","messageId":"20080121043947.GG24004@spearce.org","threadId":"11651","inReplyTo":"7vmyqz3ijb.fsf@gitster.siamese.dyndns.org","subject":"Re: git gui and commit-msg hook","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2008-01-21T04:39:47Z","receivedAt":"2008-01-21T04:39:47Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Junio C Hamano wrote:\n> \"Shawn O. Pearce\" writes:\n> \n> > \"Blucher, Guy\" wrote:\n> >> I've just setup a commit-msg hook in my local repository. It works as\n> >> advertised from the command-line when using git commit, but if I use git\n> >> gui to do the commit, then the hook is ignored.\n> >\n> > Fixed in my latest git-gui master, which will be gitgui-0.9.2 or .3.\n> >\n> > I apparently missed git 1.5.4-rc4, but will have to ask Junio nicely\n> > if this can be included into 1.5.4 final.\n> \n> Absolutely. That's a consistency-and-usability bugfix, isn't\n> it?\n\nI consider it a bug fix for git-gui, but some may say its a new\nfeature since git-gui didn't have a commit-msg hook before this\nchange. :)\n\nAnyway, its now published on repo.or.cz if you want to include\nit into your next published git.git master to try and get it\nsome more testing before 1.5.4 goes final.\n\nI'm pretty happy with what's in my master right now. I'm actually\ntempted to tag it gitgui-0.9.2 and say that's the final version\nwe're including in 1.5.4.\n\n-- \nShawn.\n"}]}