[SR-Dev] commit message subject

Jan Janak jan at iptel.org
Fri Nov 21 16:31:19 CET 2008


On 21-11 16:15, Andrei Pelinescu-Onciul wrote:
> On Nov 21, 2008 at 16:10, Jan Janak <jan at iptel.org> wrote:
> > On 21-11 15:55, Andrei Pelinescu-Onciul wrote:
> > > On Nov 21, 2008 at 15:20, Jan Janak <jan at iptel.org> wrote:
> > > > Andrei,
> > > > 
> > > > did you setup a different commit mailer for sip-router git repository than the
> > > > one that we had for the ser git repository?
> > > 
> > > No, it's the same as for ser, with --first-parent added to git rev-list.
> > 
> >   OK, my bad, I thought I added the branch name in the subject line, but it
> >   was the repository name in fact.
> > 
> > > > If so, do you think you could modify it to include the name of the branch
> > > > in the subject of the message?
> > > 
> > > Current format:
> > > 
> > > $subject = "git" . ":" . $repos_name . ": " . ${$info{"log"}}[0]
> > > 
> > > I think we could remove the repo_name, or replace it somehing shorter.
> > > My preffered format would be:
> > > 
> > > $subject = "git" . ":" . ${$info{"log"}}[0]$ . "(" . $ref ")"
> > > 
> > > (the most visible part == commit summary rather then branch).
> > > 
> > > What do you think?
> > 
> >   $subject = "git" . ":" . $ref . ":" . ${$info{"log"}}[0]$ . "(" . $ref ")"
> > 
> >   This way we would replace the repository with the name of the branch.
> > 
> >   Having the branch name in subject would be IMO very useful, because it
> >   would allow us to filter out commits specific for a particular branch, i.e.
> >   when I scan through the list of commit logs on the mailing list, I could
> >   easily filter out commits in andrei/* if I need.
> 
> Ok done:
> 
> for tags:
> 
> $subject = "git" . ":" . $ref . ": " . "Tag " . $info{"tag"} . ": " .
> ${$info{"log"}}[0];
> 
> for normal commits:
> 
> $subject = "git" . ":" . $ref . ": " . ${$info{"log"}}[0];

  Sounds good, thanks.

     Jan.



More information about the sr-dev mailing list