Skip to contentSkip to conversation

Mail after commit

DISCUSSION Claudio Scordino 4 participants 6 messages Jun 15, 2016

lore
Share & tools
lore
#0 lore raw

Hi all.

To have an email sent after every commit, I've put the following command in .git/hooks/post-commit:

rm -f /tmp/git-commit.patch ; git-format-patch --stdout -s git log --pretty=oneline | awk '{print $1}' | sed -n '2p' > /tmp/git-commit.patch ; git-send-email --no-signed-off-by-cc --to XXX --from YYY /tmp/git-commit.patch ; rm -f /tmp/git-commit.patch

and it works, indeed.

However, it's quite tricky.

Doesn't exist any way to make it simpler ?

Regards,

Claudio

#1 lore raw
Quoted 1 line · Claudio Scordino

Doesn't exist any way to make it simpler ?

I think most people setup gitweb, and follow its rss feed. Not exacty the same, I know, but quite useful.

cheers,

martin

#2 lore raw
Quoted 5 lines · Martin Langhoff
Quoted 1 line · Claudio Scordino

Doesn't exist any way to make it simpler ?

I think most people setup gitweb, and follow its rss feed. Not exacty the same, I know, but quite useful.

Erm, no, not really. Emailing the patches is *quite* useful when there's a dedicated review team.

MfG, JBG

Signature · 4 lines
      Jan-Benedict Glaw      [email protected]              +49-172-7608481
Signature of:               The real problem with C++ for kernel modules is:
the second  :                                 the language just sucks.
                                                   -- Linus Torvalds
#3 lore raw
Quoted 5 lines · Jan-Benedict Glaw
Quoted 2 lines

I think most people setup gitweb, and follow its rss feed. Not exacty the same, I know, but quite useful.

Erm, no, not really. Emailing the patches is *quite* useful when there's a dedicated review team.

As an aside - I find it better to send announce emails on git-push rather than git-commit. If you send an email on git-commit, then your working repository becomes a whole lot less useful.

For example I often do:

git commit -a -m "Shelve what I'm doing right now" git checkout -b temporary-branch HEAD^^^ vim fix-bug-that-I-never-want-anyone-to-see git checkout master
git rebase temporary-branch
git branch -d temporary-branch
git reset HEAD^

Obviously this only works for commits I haven't pushed yet, but it's very useful to be able to do - usually I need it because I missed a git-add of a file out of a commit and didn't notice for a while.

Once I'm happy with my local history, then I git-push to the shared repository and an email is generated to let other members of the team know I've done something - they can then git-fetch and review if they feel like it.

To me, sending an email every commit would be like sending an email every time I pressed "save" in the editor.

Andy

Signature · 2 lines
Dr Andy Parkins, M Eng (hons), MIET
[email protected]
#4 lore raw
In reply to Jan-Benedict Glaw
Quoted 9 lines · Jan-Benedict Glaw
Quoted 5 lines · Martin Langhoff
Quoted 1 line · Claudio Scordino

Doesn't exist any way to make it simpler ?

I think most people setup gitweb, and follow its rss feed. Not exacty the same, I know, but quite useful.

Erm, no, not really. Emailing the patches is *quite* useful when there's a dedicated review team.

I'm sure the review team can read the RSS feed ;-) I've also seen RSS-to-IMAP gateways that make it easy to "reply" to the patch from your MUA.

Not _exactly_ the same, but quite useful.

martin

#5 lore raw
Quoted 8 lines · Martin Langhoff
Quoted 7 lines · Jan-Benedict Glaw
Quoted 5 lines · On Wed, 2007-06-13 08:35:57 +1200, Martin Langhoff >…
Quoted 1 line · Claudio Scordino

Doesn't exist any way to make it simpler ?

I think most people setup gitweb, and follow its rss feed. Not exacty the same, I know, but quite useful.

Quoted 6 lines
Quoted 2 lines

Erm, no, not really. Emailing the patches is *quite* useful when there's a dedicated review team.

I'm sure the review team can read the RSS feed ;-) I've also seen RSS-to-IMAP gateways that make it easy to "reply" to the patch from your MUA.

Well, why import it to the mail platform via RSS when there's a direct way:) Almost all interesting patches require review (and commenting), and we're working with the email client all the day. So it's just straight forward ;-)

MfG, JBG

Signature · 4 lines
      Jan-Benedict Glaw      [email protected]              +49-172-7608481
Signature of:         "really soon now":      an unspecified period of time, likly to
the second  :                                 be greater than any reasonable definition
                                              of "soon".