Remove the bios emulator binary files from MAI board
Hi, Wolfgang,
I once have sent a patch to remove the bios emulator from MAI board. As the patch was too large, I put it in a internal git development tree and listed the url link in the mail. The url link I listed was the 'plain' of the patch. It's no harm to common patch, but when there're binary files in the patch, the 'plain' can not display those binaries. So the patch you get from the 'plain' link may not include the binary
file deletion.
I've tried to make another patch to delete those binary files. But this patch is also large than the requirment. This time I tried put the patch itself on the link below, Please refer to it. Thanks.
http://www.kernel.org/pub/linux/kernel/people/leo/patches/u-boot-remove-obsolete-binary.patch
Best Regards.
Jason
Quoted 7 lines · In message <[email protected]> you wrote:
As the patch was too large, I put it in a internal git development tree and listed the url link in the mail. The url link I listed was the 'plain' of the patch. It's no harm to common patch, but when there're binary files in the patch, the 'plain' can not display those binaries. So the patch you get from the 'plain' link may not include the binary
file deletion.
I'm afraid I don't really understand what that means.
Quoted 3 lines
I've tried to make another patch to delete those binary files. But this patch is also large than the requirment. This time I tried put the patch itself on the link below, Please refer to it. Thanks.
Umm... why is your patch that big? When I remove a file, I expect that the resulting "patch" takes only 3...4 lines per removed file, something like this:
diff --git a/<file> b/<file> deleted file mode 100644 index 5533346..0000000 Binary files <file> and /dev/null differ
Are you using git-diff to create the patch? If not, you should try that...
Best regards,
Wolfgang Denk
Signature · 5 lines
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 at denx.de
A dog always bit deepest on the veterinary hand.
- Terry Pratchett, _Wyrd Sisters_Quoted 16 lines
-----Original Message-----
From: wd at denx.de [mailto:wd at denx.de]
Sent: Thursday, August 23, 2007 12:20 AM
To: Jin Zhengxiong-R64188
Cc: u-boot-users at lists.sourceforge.net; Loeliger Jon-LOELIGER
Subject: Re: Remove the bios emulator binary files from MAI boardIn message
Quoted 5 lines · <[email protected]> you wrote:
As the patch was too large, I put it in a internal git development tree and listed the url link in the mail. The url link I listed was the 'plain' of the patch. It's no harm to common patch, but when there're binary files in the patch, the 'plain' can not
Quoted 6 lines
display those binaries.
Quoted 1 line
So the patch you get from the 'plain' link may not include
the binary
Quoted 1 line
file deletion.
I'm afraid I don't really understand what that means.
I mean that my orignal patch include removing those binary files. but the link can not display it.
Quoted 9 lines
Quoted 2 lines
I've tried to make another patch to delete those binary files. But
this patch is also large than the requirment. This time Itried put
Quoted 1 line
the patch itself on the link below, Please refer to it. Thanks.
Umm... why is your patch that big? When I remove a file, I expect that the resulting "patch" takes only 3...4 lines per removed file, something like this:
Quoted 7 lines
diff --git a/<file> b/<file> deleted file mode 100644 index 5533346..0000000 Binary files <file> and /dev/null differAre you using git-diff to create the patch? If not, you should try that...
Usually, I use git-format-patch to create the patch and use git-apply apply the patch.
Yes, git-diff can generate the patch like you descripted. but the patch can not apply to the source by git-apply, I tried to use cg-patch to apply this kind of patch, it can remove the binary files but can not remove the empty directory.
Best Regards.
Jason
Quoted 6 lines · On Wed, 2007-08-22 at 21:03, Jin Zhengxiong-R64188 wrote:
Usually, I use git-format-patch to create the patch and use git-apply apply the patch.
Yes, git-diff can generate the patch like you descripted. but the patch can not apply to the source by git-apply,
If there is a patch that git generates that git-apply or git-am can not then re-apply, file a bug report!
Quoted 2 lines
I tried to use cg-patch to
apply this kind of patch,
Ewww..... You should stop using cogito entirely. Really. It is totally deprecated and unsupported these days. Most all of the original cogito functionality has been incorporated into core git these days.
Quoted 2 lines
it can remove the binary files but can not
remove the empty directory.
Rest assured, git is not tracking the empty directories. :-)
Quoted 2 lines
Best Regards.
Jason
jdl
Quoted 29 lines
-----Original Message-----
From: Loeliger Jon-LOELIGER
Sent: Friday, August 24, 2007 12:17 AM
To: Jin Zhengxiong-R64188
Cc: Wolfgang Denk; u-boot-users
Subject: RE: Remove the bios emulator binary files from MAI boardQuoted 2 lines · On Wed, 2007-08-22 at 21:03, Jin Zhengxiong-R64188 wrote:
Usually, I use git-format-patch to create the patch and use
git-apply
Quoted 4 lines
apply the patch.
Yes, git-diff can generate the patch like you descripted. but the patch can not apply to the source by git-apply,
If there is a patch that git generates that git-apply or git-am can not then re-apply, file a bug report!
Quoted 2 lines
I tried to use cg-patch to
apply this kind of patch,Ewww..... You should stop using cogito entirely. Really. It is totally deprecated and unsupported these days. Most all of the original cogito functionality has been incorporated into core git these days.
Quoted 5 lines
Quoted 2 lines
it can remove the binary files but can not remove the empty
directory.Rest assured, git is not tracking the empty directories. :-)
I updated git to V1.5 and used git-rm with option '-r', then used git-format-
patch to generate the patch. The patch can apply by git-apply and can remove the directory, But as Wolfgang said, it's too large!
Jason
Quoted 5 lines · In message <CC27DED0F8F39E48A7E75FD768688B7A4D0736@zch01exm27.fsl.freescale.net…
I updated git to V1.5 and used git-rm with option '-r', then used git-format-
patch to generate the patch. The patch can apply by git-apply and can remove the directory, But as Wolfgang said, it's too large!
No, I didn't say this. I said that git-diff generates nice smallpatches.
Best regards,
Wolfgang Denk
Signature · 5 lines
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 at denx.de All a hacker needs is a tight PUSHJ, a loose pair of UUOs, and a warm place to shift.
Quoted 13 lines
-----Original Message-----
From: wd at denx.de [mailto:wd at denx.de]
Sent: Friday, August 24, 2007 2:47 PM
To: Jin Zhengxiong-R64188
Cc: Loeliger Jon-LOELIGER; u-boot-users
Subject: Re: Remove the bios emulator binary files from MAI boardIn message <[email protected]
Quoted 1 line · le.net> you wrote:
Quoted 9 lines
Quoted 3 lines
I updated git to V1.5 and used git-rm with option '-r', then used git-format-
patch to generate the patch. The patch can apply bygit-apply and can
Quoted 1 line
remove the directory, But as Wolfgang said, it's too large!
No, I didn't say this. I said that git-diff generates nice smallpatches.
I tried kinds of way to generate the patch, the only way I found to remove the binary files and the directories at the same time is the patch I sent dayes ago. If it's not acceptable, Could you please try to delete it directly in the tree? Thanks.
Best Regards,
Jason Jin
Wolfgang Denk
AppliedQuoted 11 lines · In message <[email protected]> you wrote:
Hi, Wolfgang,
I once have sent a patch to remove the bios emulator from MAI board. As the patch was too large, I put it in a internal git development tree and listed the url link in the mail. The url link I listed was the 'plain' of the patch. It's no harm to common patch, but when there're binary files in the patch, the 'plain' can not display those binaries. So the patch you get from the 'plain' link may not include the binary
file deletion.
Quoted 5 lines
I've tried to make another patch to delete those binary files. But this patch is also large than the requirment. This time I tried put the patch itself on the link below, Please refer to it. Thanks.
http://www.kernel.org/pub/linux/kernel/people/leo/patches/u-boot-remove-obsolete-binary.patch
Applied, thanks.
Best regards,
Wolfgang Denk
Signature · 5 lines
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 at denx.de Artificial Intelligence is the study of how to make real computers act like the ones in movies.
Dear Jason Jin,
can you please also fix the missing BUILD_DIR support in drivers/bios_emulator/Makefile ?
Please try for example:
$ BUILD_DIR=/tmp/foo ./MAKEALL MPC8641HPCN
to see what I mean:
Configuring for MPC8641HPCN board...
ati_radeon_fb.c: In function 'radeon_setmode':
ati_radeon_fb.c:323: warning: unused variable 'i'
ati_radeon_fb.c: In function 'radeon_probe':
ati_radeon_fb.c:383: warning: implicit declaration of function 'BootVideoCardBIOS' ati_radeon_fb.c: At top level:
ati_radeon_fb.c:307: warning: 'common_regs' defined but not used
ppc_4xx-ld: cannot find drivers/bios_emulator/atibios.o
make: *** [/work/wd/tmp/u-boot] Error 1
Thanks in advance.
Best regards,
Wolfgang Denk
Signature · 4 lines
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 at denx.de If it has syntax, it isn't user friendly.