Re: [cdt-dev] Git's gurus - need you help

[Marc Khouzam <marc.khouzam@xxxxxxxxxxxx>]

I like the way git does it better than CVS.  Only a single link
to click on.  It was quite tedious with CVS before, for large commits.

The reason I like the list of files is that it gives a quick overview
of what has changed (how many files, which components).
But I guess clicking on the one link is not a big deal to get to that
information.

Don't spend time on that, unless other people also think it is worth
doing.

Thanks

> -----Original Message-----
> From: laskava@xxxxxxxxx [mailto:laskava@xxxxxxxxx] On Behalf 
> Of Alena Laskavaia
> Sent: Tuesday, July 19, 2011 2:58 PM
> To: Marc Khouzam; CDT General developers list.
> Subject: Re: [cdt-dev] Git's gurus - need you help
> 
> So should I add list of file or not, considering they will be listed
> when you click on url?
> 
> On Tue, Jul 19, 2011 at 2:52 PM, Marc Khouzam 
> <marc.khouzam@xxxxxxxxxxxx> wrote:
> > Great.  Thank you.
> >
> >> -----Original Message-----
> >> From: cdt-dev-bounces@xxxxxxxxxxx
> >> [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Alena Laskavaia
> >> Sent: Tuesday, July 19, 2011 2:38 PM
> >> To: CDT General developers list.
> >> Subject: Re: [cdt-dev] Git's gurus - need you help
> >>
> >> I cannot figure out branch from the message it sends. 
> Files are listed
> >> in the link itself, the previous version
> >> were listing them because CVS don't support sense of multi-file
> >> commit. But file list is in the message so I can add it.
> >> Scripts are running on my local machine at work, so if you need
> >> changes just ask me.
> >>
> >> On Tue, Jul 19, 2011 at 2:31 PM, Marc Khouzam
> >> <marc.khouzam@xxxxxxxxxxxx> wrote:
> >> > Thanks Elena, this is great!
> >> >
> >> > Where is the genie code?  Can we update it ourselves or do
> >> you prefer
> >> > that we suggest things to you?
> >> >
> >> > 1- I was thinking that it would be nice to have the list of
> >> file names as part of
> >> > the commit message like the cvs genie did.
> >> > 2- Also, maybe the branch on which the commit was made.
> >> >
> >> > Thanks
> >> >
> >> > Marc
> >> >
> >> >> -----Original Message-----
> >> >> From: cdt-dev-bounces@xxxxxxxxxxx
> >> >> [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Alena 
> Laskavaia
> >> >> Sent: Tuesday, July 12, 2011 9:32 PM
> >> >> To: Schaefer, Doug
> >> >> Cc: CDT General developers list.
> >> >> Subject: Re: [cdt-dev] Git's gurus - need you help
> >> >>
> >> >> So genie back to working state, see
> >> >> https://bugs.eclipse.org/bugs/show_bug.cgi?id=351898
> >> >> as example
> >> >>
> >> >> I did not add support for edc. I don't know yet what
> >> message would be
> >> >> generated for that repo.
> >> >>
> >> >>
> >> >> On Tue, Jul 12, 2011 at 8:35 PM, Alena Laskavaia
> >> >> <elaskavaia.cdt@xxxxxxxxx> wrote:
> >> >> > Perfect
> >> >> >
> >> >> > On Tue, Jul 12, 2011 at 6:43 PM, Schaefer, Doug
> >> >> > <Doug.Schaefer@xxxxxxxxxxxxx> wrote:
> >> >> >> Try this:
> >> >> >>
> >> >> >>
> >> >> http://git.eclipse.org/c/cdt/org.eclipse.cdt.git/commit/?id=99
> >> >> 70e94efa165f20e78a6db09968b707d83ab1d1
> >> >> >>
> >> >> >> From the "via".
> >> >> >>
> >> >> >> Doug.
> >> >> >>
> >> >> >>> -----Original Message-----
> >> >> >>> From: cdt-dev-bounces@xxxxxxxxxxx
> >> >> [mailto:cdt-dev-bounces@xxxxxxxxxxx]
> >> >> >>> On Behalf Of Alena Laskavaia
> >> >> >>> Sent: Tuesday, July 12, 2011 6:42 PM
> >> >> >>> To: CDT General developers list.
> >> >> >>> Subject: [cdt-dev] Git's gurus - need you help
> >> >> >>>
> >> >> >>> I asked admin to enable git hook to post to a forum after
> >> >> successfull commit,
> >> >> >>> and we do have posts now to cdt-commits mailing list.
> >> >> >>> However I cannot interpret this info at all. I need to
> >> >> generate something
> >> >> >>> similar to what old genie did - list of changes files
> >> >> with diffs or link to a
> >> >> >>> "commit" which would show diff for all files online.
> >> >> Anybody can help?
> >> >> >>>
> >> >> >>> This is sample post:
> >> >> >>>
> >> >> >>>
> >> >> >>> The branch, cdt_8_0 has been updated
> >> >> >>>       via  9970e94efa165f20e78a6db09968b707d83ab1d1 (commit)
> >> >> >>>      from  120d96abea40139e7515848ff0f2666c9eab2f2a (commit)
> >> >> >>>
> >> >> >>> Those revisions listed above that are new to this
> >> >> repository have not
> >> >> >>> appeared on any other notification email; so we list
> >> >> those revisions in full,
> >> >> >>> below.
> >> >> >>>
> >> >> >>> - Log
> >> >> 
> -----------------------------------------------------------------
> >> >> >>> commit 9970e94efa165f20e78a6db09968b707d83ab1d1
> >> >> >>> Author: Pawel Piech <pawel.piech@xxxxxxxxxxxxx>
> >> >> >>> Date:   Tue Jul 12 13:55:40 2011 -0700
> >> >> >>>
> >> >> >>>    Bug 351898 - [expressions] Invalid expressions do not
> >> >> show errors in the
> >> >> >>>    value column.
> >> >> >>>
> >> >> >>>
> >> >> --------------------------------------------------------------
> >> >> ---------
> >> >> >>>
> >> >> >>> Summary of changes:
> >> >> >>>  .../properties/PropertiesUpdateStatus.java         |   23
> >> >> >>> +++++++++++++++++++-
> >> >> >>>  1 files changed, 22 insertions(+), 1 deletions(-)
> >> >> >>> _______________________________________________
> >> >> >>> cdt-dev mailing list
> >> >> >>> cdt-dev@xxxxxxxxxxx
> >> >> >>> https://dev.eclipse.org/mailman/listinfo/cdt-dev
> >> >> >>
> >> >> >
> >> >> _______________________________________________
> >> >> cdt-dev mailing list
> >> >> cdt-dev@xxxxxxxxxxx
> >> >> https://dev.eclipse.org/mailman/listinfo/cdt-dev
> >> >>
> >> _______________________________________________
> >> cdt-dev mailing list
> >> cdt-dev@xxxxxxxxxxx
> >> https://dev.eclipse.org/mailman/listinfo/cdt-dev
> >>
>