[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