Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-30 Thread Pierre FICHEUX
Hi, So do you want a new patch with LDFLAGS only? regards -- Pierre FICHEUX -/- CTO Smile-ECS, France -\- pierre.fich...@smile.fr http://www.smile.fr http://ingenierie.openwide.fr I would love to change the world, but they

Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-26 Thread Einar Vading
On Mon, Jul 17, 2017 at 4:48 PM, Leonardo Sandoval < leonardo.sandoval.gonza...@linux.intel.com> wrote: > On Mon, 2017-07-17 at 15:30 +0100, Burton, Ross wrote: > > > > On 17 July 2017 at 15:04, Leonardo Sandoval > > wrote: > > master has the

Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-17 Thread Pierre FICHEUX
My first proposed patch didn't include CPPFLAGS/CFLAGS but Khem Raj did prefer the new way :) -- Pierre FICHEUX -/- CTO Smile-ECS, France -\- pierre.fich...@smile.fr http://www.smile.fr http://ingenierie.openwide.fr I would

Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-17 Thread Thomas A. F. Thorne MEng AUS MIET
On 17/07/17 11:46, Burton, Ross wrote: > > On 17 July 2017 at 11:25, Thomas A. F. Thorne MEng AUS MIET > > wrote: > > helloworld.c is a C file, not a C++ file. ${CC} should give you a C > Compiler. CPPFLAGS should not be used

Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-17 Thread Leonardo Sandoval
On Mon, 2017-07-17 at 15:30 +0100, Burton, Ross wrote: > > On 17 July 2017 at 15:04, Leonardo Sandoval > wrote: > master has the second line (LDFLAGS) but not the first one > (CPPFLAGS and > CFLAGS) so the problem may also be

Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-17 Thread Burton, Ross
On 17 July 2017 at 15:04, Leonardo Sandoval < leonardo.sandoval.gonza...@linux.intel.com> wrote: > master has the second line (LDFLAGS) but not the first one (CPPFLAGS and > CFLAGS) so the problem may also be present on master. Would you mind > checking this and sent patches to the poky mailing

Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-17 Thread Leonardo Sandoval
On Mon, 2017-07-17 at 10:00 +0200, Pierre FICHEUX wrote: > This problem causes the following error : > > ERROR: example-0.1-r0 do_package_qa: QA Issue: No GNU_HASH in the elf binary > > > Signed-off-by: Pierre FICHEUX > --- >

Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-17 Thread Burton, Ross
On 17 July 2017 at 11:25, Thomas A. F. Thorne MEng AUS MIET < thomas.tho...@net2edge.com> wrote: > helloworld.c is a C file, not a C++ file. ${CC} should give you a C > Compiler. CPPFLAGS should not be used or necessary for a C compilation. > CPPFLAGS is C Preprocesor, you're thinking of

Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-17 Thread Thomas A. F. Thorne MEng AUS MIET
I am not sure about this change: On 17/07/17 09:00, Pierre FICHEUX wrote: > This problem causes the following error : > > ERROR: example-0.1-r0 do_package_qa: QA Issue: No GNU_HASH in the elf binary > > > Signed-off-by: Pierre FICHEUX > --- >

Re: [yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-17 Thread Burton, Ross
On 17 July 2017 at 09:00, Pierre FICHEUX wrote: > This problem causes the following error : > > ERROR: example-0.1-r0 do_package_qa: QA Issue: No GNU_HASH in the elf > binary > This is essentially a backport of aa3e99e24c3234b6f3f19169939d4cd498d0ce36, can you mark it

[yocto] [PATCH] Added missing CPPFLAGS/CFLAGS/LDFLAGS in sample recipe for morty branch

2017-07-17 Thread Pierre FICHEUX
This problem causes the following error : ERROR: example-0.1-r0 do_package_qa: QA Issue: No GNU_HASH in the elf binary Signed-off-by: Pierre FICHEUX --- .../target/arch/layer/recipes-example/example/example-recipe-0.1.bb| 3 ++- 1 file changed, 2 insertions(+), 1