在 Sat, 23 Jan 2010 18:03:40 +0800,Paul Menzel <[email protected]> 写道:

Am Samstag, den 23.01.2010, 17:53 +0800 schrieb Guo Hongruan:
the building of linux-storcenter_2.6.27.7 failed due to lack of this patch

I think Holger meant, what the patch is doing. Maybe it is not needed
and you can remove it from `SRC_URI`.

I just copied it from linux-orion directory. This patch just adds space between '-m' and 'mode'. (see the patch) On my ubuntu-8.04, 'install -m0644' is the same as 'install -m 0644', so this patch can be removed from SRC_URI. But I don't know whether or not it is an issue on other linux operation systems.

By the way, I found it in the following recipes, all of them are for ppc arch. linux-linkstationppc_2.6.28.4.bb: file://fw-and-powerpc-install.patch;patch=1 \ linux-orion_2.6.28.4.bb: file://fw-and-powerpc-install.patch;patch=1 \ linux-storcenter_2.6.27.7.bb: file://fw-and-powerpc-install.patch;patch=1 \
So, I just copy it into linux-storcenter.


That said, firstly Holger probably did not see that the patch is copied
from somewhere else in the repository. The question is if it is
recommended to note in the commit message from where it is copied or if
Git should be used to determine this.

Yes, I understand. I should tell where I copied it from


The second thing is, that the patch from `SRC_URI` is missing a header
with the description of its purpose [1]. But since it is copied it has
been so before.


Thanks,

Paul


[1] http://wiki.openembedded.net/index.php/Push_patches_upstream


--
Guo Hongruan, Embedded Linux Consultant
Skype: camelguo
Twitter: camelguo
http://www.gulessoft.com

_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

Reply via email to