Re: [U-Boot] [PATCH 00/17] i386 Updates (touches some common drivers)
please set sendemail.chainreplyto to false in your git settings. patch series should be like: [summary 0/xx] |- [patch 1/xx] |- [patch 2/xx] not: [summary 0/xx] `- [patch 1/xx] `- [patch 2/xx] `- the latter makes things a lot more painful for people to parse with their e- mail clients. you might have to update your git if it doesnt support this option (i see you're using 1.6.0 where as latest release is 1.6.4) ... -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 00/17] i386 Updates (touches some common drivers)
Dear Mike Frysinger, In message <200908230001.46558.vap...@gentoo.org> you wrote: > > please set sendemail.chainreplyto to false in your git settings. patch > series Why that? > should be like: > [summary 0/xx] > |- [patch 1/xx] > |- [patch 2/xx] > > > not: > [summary 0/xx] > `- [patch 1/xx] > `- [patch 2/xx] > `- On contrary. The second form is much more useful as it also indicates the order in which patches should be applied - too often people forget the -n. > the latter makes things a lot more painful for people to parse with their e- > mail clients. Huh? To me that makes no difference. Actually, I like the second form better. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Every little picofarad has a nanohenry all its own. - Don Vonada ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 00/17] i386 Updates (touches some common drivers)
On Sunday 23 August 2009 01:36:55 Wolfgang Denk wrote: > Mike Frysinger wrote: > > please set sendemail.chainreplyto to false in your git settings. patch > > series > > Why that? > > > should be like: > > [summary 0/xx] > > > > |- [patch 1/xx] > > |- [patch 2/xx] > > > > > > > > not: > > [summary 0/xx] > > `- [patch 1/xx] > > `- [patch 2/xx] > > `- > > On contrary. The second form is much more useful as it also indicates > the order in which patches should be applied - too often people forget > the -n. the numbering indicates the proper order and that is what -n controls. it's not directly related to threading as that is an independent option --chain- reply-to. > > the latter makes things a lot more painful for people to parse with their > > e- mail clients. > > Huh? To me that makes no difference. Actually, I like the second form > better. you must either not thread your e-mails properly, or your monitor is insanely wide and/or you use insanely small fonts. as soon as you hit 10+ depth on threads, subjects easily start getting truncated in the display. seeing a few to no characters in the e-mail reader for a subject is a pita. if it's all the same thread, that's one thing as the subject doesnt really change, but in this case they arent. sane example: http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/66481 insane example: http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/66597 -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 00/17] i386 Updates (touches some common drivers)
On Sun, Aug 23, 2009 at 4:14 PM, Mike Frysinger wrote: > On Sunday 23 August 2009 01:36:55 Wolfgang Denk wrote: >> Mike Frysinger wrote: >> > please set sendemail.chainreplyto to false in your git settings. patch >> > series >> >> Why that? >> > > sane example: > http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/66481 > > insane example: > http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/66597 > -mike I must say, I agree with Mike - It just looks 'odd' and adds nothing that [Patch x/y] does not already add ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 00/17] i386 Updates (touches some common drivers)
On Sun, Aug 23, 2009 at 07:36:55AM +0200, Wolfgang Denk wrote: > Dear Mike Frysinger, > > In message <200908230001.46558.vap...@gentoo.org> you wrote: > > > > please set sendemail.chainreplyto to false in your git settings. patch > > series > > Why that? > > > should be like: > > [summary 0/xx] > > |- [patch 1/xx] > > |- [patch 2/xx] > > > > > > not: > > [summary 0/xx] > > `- [patch 1/xx] > > `- [patch 2/xx] > > `- > > On contrary. The second form is much more useful as it also indicates > the order in which patches should be applied - too often people forget > the -n. > > > the latter makes things a lot more painful for people to parse with their e- > > mail clients. > > Huh? To me that makes no difference. Actually, I like the second form > better. I much prefer the first form. Besides being more like just about every other vertical list in the world, the second form tends to push the meaningful bit of the subject line off the right edge of the screen. The [U-Boot] tag doesn't help in that respect either... -Scott ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot