Message ID | 1294636905-30832-1-git-send-email-vapier@gentoo.org |
---|---|
State | Accepted |
Headers | show |
Dear Mike Frysinger, In message <1294636905-30832-1-git-send-email-vapier@gentoo.org> you wrote: > The following changes since commit 89c95f0cd3f8140f3b8a82a22a6a144c148d09c6: > > asm-offsets: generate bd_t size (2011-01-09 18:08:20 +0100) > > are available in the git repository at: > git://www.denx.de/git/u-boot-blackfin.git master ... > Mike Frysinger (27): > MAINTAINERS: sort Blackfin entries This was already applied. Why do you include it again? > Blackfin: BF50x: new processor port ... > Blackfin: BF52x: unify duplicated headers > Blackfin: BF537: unify duplicated headers I cannot find any trace of these commits on the mailing list nor in patchwork. Am I missing something? > Blackfin: bf506f-ezkit: new board port This commit was rejected because the MAINATINERS entry is missing, yet you include it here again? > Blackfin: ldrinfo: new command You posted a new version of the patch which added a README entry, yet here you include the old version Not pulled. Best regards, Wolfgang Denk
Dear Mike Frysinger, In message <1294636905-30832-1-git-send-email-vapier@gentoo.org> you wrote: > The following changes since commit 89c95f0cd3f8140f3b8a82a22a6a144c148d09c6: > > asm-offsets: generate bd_t size (2011-01-09 18:08:20 +0100) > > are available in the git repository at: > git://www.denx.de/git/u-boot-blackfin.git master See previous message. Not pulled either. Best regards, Wolfgang Denk
On Wednesday, January 12, 2011 17:43:37 Wolfgang Denk wrote: > Mike Frysinger wrote: > > The following changes since commit 89c95f0cd3f8140f3b8a82a22a6a144c148d09c6: > > asm-offsets: generate bd_t size (2011-01-09 18:08:20 +0100) > > > > are available in the git repository at: > > git://www.denx.de/git/u-boot-blackfin.git master > > ... > > > Mike Frysinger (27): > > MAINTAINERS: sort Blackfin entries > > This was already applied. Why do you include it again? i posted these patches and requests before you manually merged it. i dont see how this is surprising. > > Blackfin: BF50x: new processor port > ... > > Blackfin: BF52x: unify duplicated headers > > Blackfin: BF537: unify duplicated headers > > I cannot find any trace of these commits on the mailing list nor in > patchwork. Am I missing something? i mentioned in the original summary that these were too large for the list because i did quite a bit of clean up on them that isnt really splittable. -mike
On Wednesday, January 12, 2011 17:44:18 Wolfgang Denk wrote: > Mike Frysinger wrote: > > The following changes since commit 89c95f0cd3f8140f3b8a82a22a6a144c148d09c6: > > asm-offsets: generate bd_t size (2011-01-09 18:08:20 +0100) > > > > are available in the git repository at: > > git://www.denx.de/git/u-boot-blackfin.git master > > See previous message. Not pulled either. you replied to the same e-mail twice -mike
Dear Mike Frysinger, In message <201101121750.06963.vapier@gentoo.org> you wrote: > > > I cannot find any trace of these commits on the mailing list nor in > > patchwork. Am I missing something? > > i mentioned in the original summary that these were too large for the list > because i did quite a bit of clean up on them that isnt really splittable. You should know the rules. Please stick to them. I will not permit you to sneak in unreviewed code like that. Don't try that again. Best regards, Wolfgang Denk
Dear Mike Frysinger, In message <201101121750.37504.vapier@gentoo.org> you wrote: > > > See previous message. Not pulled either. > > you replied to the same e-mail twice Sorry, I somehow had a duplicated message in my folder. Best regards, Wolfgang Denk
On Wednesday, January 12, 2011 18:03:35 Wolfgang Denk wrote: > Mike Frysinger wrote: > > > I cannot find any trace of these commits on the mailing list nor in > > > patchwork. Am I missing something? > > > > i mentioned in the original summary that these were too large for the > > list because i did quite a bit of clean up on them that isnt really > > splittable. > > You should know the rules. Please stick to them. > > I will not permit you to sneak in unreviewed code like that. > > Don't try that again. stop making crap up. i'm not "sneaking" anything thing in. i explicitly called out these patches on the list and mentioned exactly where to find them. -mike
Dear Mike Frysinger, In message <201101121808.17659.vapier@gentoo.org> you wrote: > > stop making crap up. i'm not "sneaking" anything thing in. i explicitly > called out these patches on the list and mentioned exactly where to find them. Did you? In some summary message nobody is going to read? You are supposed to submit patches. Please (re-) read http://www.denx.de/wiki/U-Boot/Patches ; some link to some git repository is NOT sufficient (especially when it comes whithout any spoecific information like commit ID or at least branch name and Subject. As a custodian you are supposed to be especially careful and to set an example for others, less experienced developers. Best regards, Wolfgang Denk
Dear Mike Frysinger, In message <201101121808.17659.vapier@gentoo.org> you wrote: > > stop making crap up. i'm not "sneaking" anything thing in. i explicitly > called out these patches on the list and mentioned exactly where to find them. Actually I can see absolutely no good reason for not posting these patches. As far as I can tell, none of them exceeds the ML's hard size limit. 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: wd@denx.de The main thing is the play itself. I swear that greed for money has nothing to do with it, although heaven knows I am sorely in need of money. - Feodor Dostoyevsky
On Wednesday, January 12, 2011 18:18:25 Wolfgang Denk wrote: > Mike Frysinger wrote: > > stop making crap up. i'm not "sneaking" anything thing in. i explicitly > > called out these patches on the list and mentioned exactly where to find > > them. > > Did you? In some summary message nobody is going to read? that's crap. the whole point of the summary message is to *summarize* the patchset and give an overview of what's going on. -mike
On Wednesday, January 12, 2011 18:32:36 Wolfgang Denk wrote: > Mike Frysinger wrote: > > stop making crap up. i'm not "sneaking" anything thing in. i explicitly > > called out these patches on the list and mentioned exactly where to find > > them. > > Actually I can see absolutely no good reason for not posting these patches. > As far as I can tell, none of them exceeds the ML's hard size limit. i dont know what you're looking at, but you're mistaken. i explicitly got rejection notices about being too big. they're each 250+ KiB. -mike
Dear Mike Frysinger, In message <201101121837.43007.vapier@gentoo.org> you wrote: > > > Actually I can see absolutely no good reason for not posting these patches. > > As far as I can tell, none of them exceeds the ML's hard size limit. > > i dont know what you're looking at, but you're mistaken. i explicitly got > rejection notices about being too big. they're each 250+ KiB. I was looking at your repo. The ML soft limit is 100 KiB, the hard limit is 400 KiB. As I wrote: no reason at all not to post them (except that they need cleanup, for example because of exceeding max line length, or using incorrect multiline comments etc.). Best regards, Wolfgang Denk
Dear Mike Frysinger, In message <201101121836.14803.vapier@gentoo.org> you wrote: > > that's crap. the whole point of the summary message is to *summarize* the > patchset and give an overview of what's going on. Right. But (links to) patches are NOT posted in a summary message but sperately, with appropriate subject and full commit message. And a link to some git repo is NOT accepted. Please (re-) read http://www.denx.de/wiki/U-Boot/Patches Best regards, Wolfgang Denk
On Wednesday, January 12, 2011 18:47:53 Wolfgang Denk wrote: > Mike Frysinger wrote: > > that's crap. the whole point of the summary message is to *summarize* > > the patchset and give an overview of what's going on. > > Right. But (links to) patches are NOT posted in a summary message but > sperately, with appropriate subject and full commit message. And a > link to some git repo is NOT accepted. > > Please (re-) read http://www.denx.de/wiki/U-Boot/Patches you mean re-read after you just updated it to ban things i was doing -mike