Re: [U-Boot] [PATCH 2/3] OMAP3: Rename interrupts.c to timer.c
Dear Dirk, In message 4a047b71.6080...@googlemail.com you wrote: This results then in something cpu/sa1100/{interrupts.c = timer.c} |5 +- rename cpu/arm925t/{interrupts.c = timer.c} (99%) (from [1]). But this is only what normally 'diffstat' generates and what is totally ignored by 'patch' . It can be applied with git am (well, not the diffstat, but the special rename diff hunk that git also generates). Ok, thanks. But not with other tools, like plain 'patch', correct? Indeed. But why would you use such a tool in combination with a git repository? It makes no sense. 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 You can't evaluate a man by logic alone. -- McCoy, I, Mudd, stardate 4513.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] OMAP3: Rename interrupts.c to timer.c
Dear Jean-Christophe, Jean-Christophe PLAGNIOL-VILLARD wrote: On 17:41 Tue 05 May , Dirk Behme wrote: After removal of dublicated interrupt code, rename file to what it really does now. I prefer to rename all files at the same time Then, please don't complain that you have to do everything on your own and nobody helps you ;) I would prefer that changes are done by the board maintainers and not by the custodian, though. Btw please use git to show that you only rename the file without change I'm not sure that this works for non-custodians. Custodians can do git-rename and then send a patch to the mailing list. When a custodian sends such a patch, it is mainly for review only, and not to be applied somewhere (because it is already in git). This results then in something cpu/sa1100/{interrupts.c = timer.c} |5 +- rename cpu/arm925t/{interrupts.c = timer.c} (99%) (from [1]). But this is only what normally 'diffstat' generates and what is totally ignored by 'patch' . I.e. in the part of the patch which is handled by 'patch' this info is totally missing. As far as I know 'patch' itself can't deal with file rename any other way than remove one file and create the other one with --- u-boot-main.orig/cpu/arm_cortexa8/omap3/interrupts.c +++ /dev/null --- /dev/null +++ u-boot-main/cpu/arm_cortexa8/omap3/timer.c So it's my understanding that non-custodians have to send a patch which can be applied everywhere by everyone by 'patch' utility, i.e. the part 'patch' deals with contains all information necessary. Do I miss anything? Best regards Dirk [1] http://lists.denx.de/pipermail/u-boot/2009-May/051921.html ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] OMAP3: Rename interrupts.c to timer.c
On Fri, May 08, 2009 at 05:47:27PM +0200, Dirk Behme wrote: I'm not sure that this works for non-custodians. Custodians can do git-rename and then send a patch to the mailing list. When a custodian sends such a patch, it is mainly for review only, and not to be applied somewhere (because it is already in git). This results then in something cpu/sa1100/{interrupts.c = timer.c} |5 +- rename cpu/arm925t/{interrupts.c = timer.c} (99%) (from [1]). But this is only what normally 'diffstat' generates and what is totally ignored by 'patch' . It can be applied with git am (well, not the diffstat, but the special rename diff hunk that git also generates). -Scott ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] OMAP3: Rename interrupts.c to timer.c
Scott Wood wrote: On Fri, May 08, 2009 at 05:47:27PM +0200, Dirk Behme wrote: I'm not sure that this works for non-custodians. Custodians can do git-rename and then send a patch to the mailing list. When a custodian sends such a patch, it is mainly for review only, and not to be applied somewhere (because it is already in git). This results then in something cpu/sa1100/{interrupts.c = timer.c} |5 +- rename cpu/arm925t/{interrupts.c = timer.c} (99%) (from [1]). But this is only what normally 'diffstat' generates and what is totally ignored by 'patch' . It can be applied with git am (well, not the diffstat, but the special rename diff hunk that git also generates). Ok, thanks. But not with other tools, like plain 'patch', correct? Best regards Dirk ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] OMAP3: Rename interrupts.c to timer.c
Dirk Behme wrote: Scott Wood wrote: It can be applied with git am (well, not the diffstat, but the special rename diff hunk that git also generates). Ok, thanks. But not with other tools, like plain 'patch', correct? Right, until someone fixes that. :-) Which I suppose could be a problem if git am refuses to accept the patch and the normal patch command needs to be used to resolve the conflict (I really wish git would treat it more like a merge from a UI perspective, even if it doesn't look that way internally). -Scott ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] OMAP3: Rename interrupts.c to timer.c
Dear Dirk, In message 4a04540f.10...@googlemail.com you wrote: Btw please use git to show that you only rename the file without change I'm not sure that this works for non-custodians. Custodians can do git-rename and then send a patch to the mailing list. When a custodian sends such a patch, it is mainly for review only, and not to be applied somewhere (because it is already in git). In which way is this rtestricted to custodians? everybody can do the same in his own git repositories and use this to provide patches and statistics. (from [1]). But this is only what normally 'diffstat' generates and what is totally ignored by 'patch' . I.e. in the part of the patch We don't use plain old patch, we use git tools (like git am). And these do understand about rename patches. which is handled by 'patch' this info is totally missing. As far as I know 'patch' itself can't deal with file rename any other way than remove one file and create the other one with That's why we use git. So it's my understanding that non-custodians have to send a patch which can be applied everywhere by everyone by 'patch' utility, i.e. the part 'patch' deals with contains all information necessary. Do I miss anything? Yes, you are missing that we use git these days, and git is more powerful. 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 In the beginning, there was nothing, which exploded. - Terry Pratchett, _Lords and Ladies_ ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] OMAP3: Rename interrupts.c to timer.c
On 17:41 Tue 05 May , Dirk Behme wrote: After removal of dublicated interrupt code, rename file to what it really does now. I prefer to rename all files at the same time Btw please use git to show that you only rename the file without change Best Regards, J. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot