Patchwork Add build instructions for coreinfo in readme

login
register
about
Submitter Hamo
Date 2011-03-30 00:19:52
Message ID <AANLkTimY-s4Y3-L21d9Z+iUSFfArBdMrM9XftXdrO+6z@mail.gmail.com>
Download mbox | patch
Permalink /patch/2819/
State Accepted
Headers show

Comments

Hamo - 2011-03-30 00:19:52
Add build instructions for coreinfo, specially pointing out installing
gcc-multilib on a 64bit system.

Signed-off-by: Yang Hamo Bai <hamo.by@gmail.com>

This is the very first time I send a patch to coreboot, any comment is
welcome. Thanks.
Hamo - 2011-04-01 00:15:49
Can someone comment on this?

On 3/30/11, Hamo <hamo.by@gmail.com> wrote:
> Add build instructions for coreinfo, specially pointing out installing
> gcc-multilib on a 64bit system.
>
> Signed-off-by: Yang Hamo Bai <hamo.by@gmail.com>
>
> This is the very first time I send a patch to coreboot, any comment is
> welcome. Thanks.
>
Stefan Reinauer - 2011-04-01 00:39:51
* Hamo <hamo.by@gmail.com> [110330 02:19]:
> Add build instructions for coreinfo, specially pointing out installing
> gcc-multilib on a 64bit system.
> 
> Signed-off-by: Yang Hamo Bai <hamo.by@gmail.com>
> 
> This is the very first time I send a patch to coreboot, any comment is
> welcome. Thanks.


Thank you Hamo. Committed as r6470 with a slight change.

Patch

Index: payloads/coreinfo/README
===================================================================
--- payloads/coreinfo/README	(revision 6469)
+++ payloads/coreinfo/README	(working copy)
@@ -1,3 +1,22 @@ 
 This is a silly little program that demonstrates how cool libpayload is
 and also serves a purpose.  Its fun and educational!
 
+Build
+-----
+
+You need libpayload to build coreinfo. So, first, you need follow the README of 
+libpayload to build it but install libpayload into its own directory by doing 
+this:
+
+ $ make DESTDIR=/path/to/libpayload/install install
+
+don't forget to install gcc-multilib if you are on a 64bit system.
+
+Then you can build coreinfo now:
+
+ $ cd coreinfo
+
+ $ make menuconfig
+ 
+ $ make
+