Re: [OE-core] [PATCH] eglibc: Fix to egrep 'yes$' instead 'yes' at configure

2014-08-27 Thread Benjamin Esquivel
Ok, I will re-do the patch and send it over On Mon, 2014-08-25 at 16:31 -0700, Khem Raj wrote: > On 14-08-25 17:57:47, Benjamin Esquivel wrote: > > To Khem and Andreas, > > > > I would say that the proposed patch is based on the existing design, a > > corner case was found and it deserves a fix

Re: [OE-core] [PATCH] eglibc: Fix to egrep 'yes$' instead 'yes' at configure

2014-08-25 Thread Khem Raj
On 14-08-25 17:57:47, Benjamin Esquivel wrote: > To Khem and Andreas, > > I would say that the proposed patch is based on the existing design, a > corner case was found and it deserves a fix but not so sure about > redesigning it > > By redesign I mean to modify also the part where the string is

Re: [OE-core] [PATCH] eglibc: Fix to egrep 'yes$' instead 'yes' at configure

2014-08-25 Thread Benjamin Esquivel
To Khem and Andreas, I would say that the proposed patch is based on the existing design, a corner case was found and it deserves a fix but not so sure about redesigning it By redesign I mean to modify also the part where the string is thrown into the conftest.c Please let me know your thoughts

Re: [OE-core] [PATCH] eglibc: Fix to egrep 'yes$' instead 'yes' at configure

2014-08-25 Thread Khem Raj
On 14-08-24 21:58:11, Benjamin Esquivel wrote: > The pre-processed output of conftest.c contains an include dir > and if the path of it contains a 'yes' it will cause some variables > to be wrongly set to yes because of the way it greps for it. > Fix is to egrep for yes$ instead. > This has to be d

[OE-core] [PATCH] eglibc: Fix to egrep 'yes$' instead 'yes' at configure

2014-08-25 Thread Benjamin Esquivel
The pre-processed output of conftest.c contains an include dir and if the path of it contains a 'yes' it will cause some variables to be wrongly set to yes because of the way it greps for it. Fix is to egrep for yes$ instead. This has to be done at the configure and the configure.ac files given the