Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
RE: [cdt-dev] commit auto comment in bugzilla

+1!


> -----Original Message-----
> From: cdt-dev-bounces@xxxxxxxxxxx 
> [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Alena Laskavaia
> Sent: Thursday, May 27, 2010 8:47 AM
> To: CDT General developers list.
> Subject: Re: [cdt-dev] commit auto comment in bugzilla
> 
> Bugzilla does not allow HTML, if we want link it has to be visible
> (see http://www.bugzilla.org/docs/2.16/html/hintsandtips.html)
> 
> On Thu, May 27, 2010 at 8:09 AM, John Cortell 
> <rat042@xxxxxxxxxxxxx> wrote:
> > But even with DSF, often there are additional tweaks that 
> happen after the
> > major fix and we sometimes don't bother creating a patch 
> file for them. A
> > big +1 from me on this one. It adds clutter to the bug 
> reports, but the
> > cross-referencing benefits far outweigh the eyesore, IMO. 
> And it removes the
> > human elements from the process (oversight, laziness, 
> errors). Not to
> > mention, it shows what was actually committed, rather than 
> what someone
> > intends to commit, so it is much more reliable. Patches 
> would still be
> > necessary for commits that require code reviews.
> >
> > What would really help cut down the clutter is if the 
> content was listed as
> > follows:
> >
> > *** cdt cvs genie ***
> >
> > [ ] PerProjectSICollector.java, 1.25
> > [+] EFSExtensionManager.java, 1.1
> > [-] FileSystemUtilityManager.java
> > [ ] ErrorParserManager.java, 1.65
> >
> > * Only one [cdt cvs genie] header
> > * Have a marker indicate the nature of the delta: changed, 
> new, deleted
> > * Show only the simple file name
> > * Show the revision
> > * The file names would be links. You could see the full URL 
> by hovering, and
> > go to the diff by clicking it.
> >
> > That would make this mechanism perfect in my eyes, but I 
> think it's worth
> > having as-is.
> >
> > John
> >
> > At 06:38 AM 5/27/2010, Alena Laskavaia wrote:
> >
> > If you want I can exclude you from the list (i.e if you 
> made change it
> > won't do it)
> > Maybe in dsf it is normal to attach a patch but in other projects it
> > is only done on occasion.
> >
> > On Thu, May 27, 2010 at 3:43 AM, Schorn, Markus
> > <Markus.Schorn@xxxxxxxxxxxxx> wrote:
> >> I don't like them, a patch attached to a bug is much more useful.
> >> Markus.
> >>
> >>> -----Original Message-----
> >>> From: cdt-dev-bounces@xxxxxxxxxxx
> >>> [ mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Alena Laskavaia
> >>> Sent: Wednesday, May 26, 2010 8:55 PM
> >>> To: CDT General developers list.
> >>> Subject: [cdt-dev] commit auto comment in bugzilla
> >>> Importance: Low
> >>>
> >>> I finished my script that adds comments to bugzilla when
> >>> commit performed.
> >>> See example https://bugs.eclipse.org/bugs/show_bug.cgi?id=291815
> >>> If you like what it does I will run it in daemon mode.
> >>> _______________________________________________
> >>> 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
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
> 

Back to the top