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

It is all because you don't use correct bug notation!
Just kidding. Fixed. Not kidding about bug notation, it should start
with capital B.

On Thu, May 27, 2010 at 10:30 AM, Andrew Gvozdev <angvoz.dev@xxxxxxxxx> wrote:
> Is there a bugzilla to report bugs? In bug 314253 I am getting:
>
>> *** cdt cvs genie on behalf of agvozdev ***
>> bug 314253: Linker warning not recognized
>>
>> [*] GLDErrorParserTests.java 1.3
>> http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.core.tests/misc/org/eclipse/cdt/core/internal/errorparsers/tests/GLDErrorParserTests.java?root=Tools_Project&r1=1.2&r2=1.3
>
> But second line of multiline comment is missing and there were more changes:
>
> Bug 314253: Linker warning not recognized
> Based on work from Axel Mueller
> * all/org.eclipse.cdt.core/plugin.xml 1.148 (changed +2 -2, diff)
> * all/org.eclipse.cdt.core/plugin.properties 1.57 (changed +2 -2, diff)
> *
> all/org.eclipse.cdt.core.tests/misc/org/eclipse/cdt/core/internal/errorparsers/tests/GLDErrorParserTests.java
> 1.3 (changed +25 -0, diff)
>
> Andrew
>
> On Thu, May 27, 2010 at 10:02 AM, Alena Laskavaia <elaskavaia.cdt@xxxxxxxxx>
> wrote:
>>
>> That is good idea to add commit comment there...
>> So I am changing [!] to [*]
>>
>> Also committers this is remainer that bug comment should look like
>> Bug 12345
>> in the BEGGING of commit comment. Capitalization matters too.
>>
>> So these are not valid comments !!! (ignoring the star *):
>>
>> * BUG 314055 - Updated build files to add EDC docs manual to CDT build
>> * further changes for  Bug 291815 - fix test failure on UNIX/Linux
>> * Bugzilla 12345
>> * bug 12345
>> * [12345]
>>
>>
>> On Thu, May 27, 2010 at 9:51 AM, Andrew Gvozdev <angvoz.dev@xxxxxxxxx>
>> wrote:
>> > I'd like to see a star for changed files [*] and actual commit message
>> > in
>> > the beginning of comment.
>> >
>> > Andrew
>> >
>> > On Thu, May 27, 2010 at 9:31 AM, Alena Laskavaia
>> > <elaskavaia.cdt@xxxxxxxxx>
>> > wrote:
>> >>
>> >> https://bugs.eclipse.org/bugs/show_bug.cgi?id=309126
>> >> How is that?
>> >> (Same file appear more than one because you did few check-ins of it I
>> >> think)
>> >>
>> >> 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
>> >
>> >
>> _______________________________________________
>> cdt-dev mailing list
>> cdt-dev@xxxxxxxxxxx
>> https://dev.eclipse.org/mailman/listinfo/cdt-dev
>
>


Back to the top