Re: [IAEP] [Sugar-devel] [ANNOUNCE] Sucrose 0.83.4 Development Release

2009-01-24 Thread Martin Langhoff
On Sat, Jan 24, 2009 at 10:42 AM, Bernie Innocenti wrote: > Well, it's customary to introduce an additional state where the bug is > fixed in the developer's intentions, but not yet QA'd: > > NEW -> ASSIGNED -> FIXED -> CLOSED Bernie, 99% of my commits are "working on it, mate" ;-) It's not ab

Re: [IAEP] [Sugar-devel] [ANNOUNCE] Sucrose 0.83.4 Development Release

2009-01-24 Thread Bernie Innocenti
Martin Langhoff wrote: > On Sat, Jan 24, 2009 at 6:39 AM, Bernie Innocenti wrote: >> I meant it should have been optional, but if we switch to using the >> "Closes: header in the body, where we have no size constraints, then >> we could has well use the prefix consistently. > > One important note

Re: [IAEP] [Sugar-devel] [ANNOUNCE] Sucrose 0.83.4 Development Release

2009-01-23 Thread Martin Langhoff
On Sat, Jan 24, 2009 at 6:39 AM, Bernie Innocenti wrote: > I meant it should have been optional, but if we switch to using the > "Closes: header in the body, where we have no size constraints, then > we could has well use the prefix consistently. One important note WRT 'Closes'... Code hits git

Re: [IAEP] [Sugar-devel] [ANNOUNCE] Sucrose 0.83.4 Development Release

2009-01-23 Thread Bernie Innocenti
Jonas Smedegaard wrote: >> - Given the above, the word "Closes: " steals precious characters, >> and is rather easy to deduce, therefore I'd opt it out. > > It really makes better sense to me to not squeeze bug hints into that > first line at all, but instead include them in a later line of the

Re: [IAEP] [Sugar-devel] [ANNOUNCE] Sucrose 0.83.4 Development Release

2009-01-23 Thread Tomeu Vizoso
On Fri, Jan 23, 2009 at 13:22, Simon Schampijer wrote: > Marco Pesenti Gritti wrote: >> On Fri, Jan 23, 2009 at 10:45 AM, Bernie Innocenti >> wrote: >>> Some thoughts: >>> >>> - Because the commit message summary appears in the shortlog, >>> it should be kept below 74 characters to avoid ugly

Re: [IAEP] [Sugar-devel] [ANNOUNCE] Sucrose 0.83.4 Development Release

2009-01-23 Thread Simon Schampijer
Marco Pesenti Gritti wrote: > On Fri, Jan 23, 2009 at 10:45 AM, Bernie Innocenti wrote: >> Some thoughts: >> >> - Because the commit message summary appears in the shortlog, >> it should be kept below 74 characters to avoid ugly wrapping. >> >> - Given the above, the word "Closes: " steals pre