diff mbox

[1/2] Prevent zconf.gperf from throwing an error on OSX

Message ID 1407078413-21972-2-git-send-email-delta.tatham@me.com
State New
Headers show

Commit Message

Delta Tatham Aug. 3, 2014, 3:06 p.m. UTC
From: Daniel Zauner <delta.tatham@me.com>

Grabbed this one from the homebrew formula. Might need further adaption for non-OSX
Signed-off-by: Daniel Zauner aka Delta Tatham <delta.tatham@me.com>
---
 kconfig/zconf.gperf | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Bryan Hundven Aug. 5, 2014, 5:20 a.m. UTC | #1
Daniel, List,

On Sun, Aug 3, 2014 at 8:06 AM, Delta Tatham <delta.tatham@me.com> wrote:
> From: Daniel Zauner <delta.tatham@me.com>
>
> Grabbed this one from the homebrew formula. Might need further adaption for non-OSX
> Signed-off-by: Daniel Zauner aka Delta Tatham <delta.tatham@me.com>
> ---
>  kconfig/zconf.gperf | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/kconfig/zconf.gperf b/kconfig/zconf.gperf
> index c9e690e..21e79e4 100644
> --- a/kconfig/zconf.gperf
> +++ b/kconfig/zconf.gperf
> @@ -7,6 +7,10 @@
>  %pic
>  %struct-type

Can we ifdef on Darwin/BSD here? If it works on other platforms, then
only the ones mentioned need stddef.h included here.

> +%{
> +#include <stddef.h>
> +%}
> +
>  struct kconf_id;
>
>  static struct kconf_id *kconf_id_lookup(register const char *str, register unsigned int len);
> --
> 1.9.3 (Apple Git-50)
>
>
> --
> For unsubscribe information see http://sourceware.org/lists.html#faq
>

Otherwise: +1

-Bryan

--
For unsubscribe information see http://sourceware.org/lists.html#faq
Fernando Ortiz (e2k) Aug. 5, 2014, 5:42 a.m. UTC | #2
I can confirm this work as is using Mac OS X, ubuntu and debian.

On Aug 5, 2014, at 12:20 AM, Bryan Hundven <bryanhundven@gmail.com> wrote:

> Daniel, List,
> 
> On Sun, Aug 3, 2014 at 8:06 AM, Delta Tatham <delta.tatham@me.com> wrote:
>> From: Daniel Zauner <delta.tatham@me.com>
>> 
>> Grabbed this one from the homebrew formula. Might need further adaption for non-OSX
>> Signed-off-by: Daniel Zauner aka Delta Tatham <delta.tatham@me.com>
>> ---
>> kconfig/zconf.gperf | 4 ++++
>> 1 file changed, 4 insertions(+)
>> 
>> diff --git a/kconfig/zconf.gperf b/kconfig/zconf.gperf
>> index c9e690e..21e79e4 100644
>> --- a/kconfig/zconf.gperf
>> +++ b/kconfig/zconf.gperf
>> @@ -7,6 +7,10 @@
>> %pic
>> %struct-type
> 
> Can we ifdef on Darwin/BSD here? If it works on other platforms, then
> only the ones mentioned need stddef.h included here.
> 
>> +%{
>> +#include <stddef.h>
>> +%}
>> +
>> struct kconf_id;
>> 
>> static struct kconf_id *kconf_id_lookup(register const char *str, register unsigned int len);
>> --
>> 1.9.3 (Apple Git-50)
>> 
>> 
>> --
>> For unsubscribe information see http://sourceware.org/lists.html#faq
>> 
> 
> Otherwise: +1
> 
> -Bryan
> 
> --
> For unsubscribe information see http://sourceware.org/lists.html#faq
> 


--
For unsubscribe information see http://sourceware.org/lists.html#faq
Bryan Hundven Aug. 5, 2014, 7:38 a.m. UTC | #3
Fernando, all,

On Mon, Aug 4, 2014 at 10:42 PM, Fernando Ortiz (e2k)
<eratos2000@gmail.com> wrote:
> I can confirm this work as is using Mac OS X, ubuntu and debian.

Was it a failing issue on ubuntu or debian? Did it add a warning? I'm
not around my computer, or I'd just test it myself.

> On Aug 5, 2014, at 12:20 AM, Bryan Hundven <bryanhundven@gmail.com> wrote:
>
>> Daniel, List,
>>
>> On Sun, Aug 3, 2014 at 8:06 AM, Delta Tatham <delta.tatham@me.com> wrote:
>>> From: Daniel Zauner <delta.tatham@me.com>
>>>
>>> Grabbed this one from the homebrew formula. Might need further adaption for non-OSX
>>> Signed-off-by: Daniel Zauner aka Delta Tatham <delta.tatham@me.com>
>>> ---
>>> kconfig/zconf.gperf | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/kconfig/zconf.gperf b/kconfig/zconf.gperf
>>> index c9e690e..21e79e4 100644
>>> --- a/kconfig/zconf.gperf
>>> +++ b/kconfig/zconf.gperf
>>> @@ -7,6 +7,10 @@
>>> %pic
>>> %struct-type
>>
>> Can we ifdef on Darwin/BSD here? If it works on other platforms, then
>> only the ones mentioned need stddef.h included here.
>>
>>> +%{
>>> +#include <stddef.h>
>>> +%}
>>> +
>>> struct kconf_id;
>>>
>>> static struct kconf_id *kconf_id_lookup(register const char *str, register unsigned int len);
>>> --
>>> 1.9.3 (Apple Git-50)
>>>
>>>
>>> --
>>> For unsubscribe information see http://sourceware.org/lists.html#faq
>>>
>>
>> Otherwise: +1
>>
>> -Bryan
>>
>> --
>> For unsubscribe information see http://sourceware.org/lists.html#faq
>>
>

Thanks,

-Bryan

--
For unsubscribe information see http://sourceware.org/lists.html#faq
Fernando Ortiz (e2k) Aug. 6, 2014, 4:52 p.m. UTC | #4
Bryan, all,

On 08/05/2014 02:38 AM, Bryan Hundven wrote:
> Fernando, all,
>
> On Mon, Aug 4, 2014 at 10:42 PM, Fernando Ortiz (e2k)
> <eratos2000@gmail.com> wrote:
>> I can confirm this work as is using Mac OS X, ubuntu and debian.
> Was it a failing issue on ubuntu or debian? Did it add a warning? I'm
> not around my computer, or I'd just test it myself.

The computer I'm using was built recently so not all the packages were 
there when I first tested, I know it fails the first time, but cannot 
recreate the problem now tested again on trusty and works without this 
patch.  I had installed many packages since I first tested that it may 
have fixed the original problem.

However it still fails on Mac OS X with: 
https://gist.github.com/nandub/fd91fcfcc751149fe7ba
>
>> On Aug 5, 2014, at 12:20 AM, Bryan Hundven <bryanhundven@gmail.com> wrote:
>>
>>> Daniel, List,
>>>
>>> On Sun, Aug 3, 2014 at 8:06 AM, Delta Tatham <delta.tatham@me.com> wrote:
>>>> From: Daniel Zauner <delta.tatham@me.com>
>>>>
>>>> Grabbed this one from the homebrew formula. Might need further adaption for non-OSX
>>>> Signed-off-by: Daniel Zauner aka Delta Tatham <delta.tatham@me.com>
>>>> ---
>>>> kconfig/zconf.gperf | 4 ++++
>>>> 1 file changed, 4 insertions(+)
>>>>
>>>> diff --git a/kconfig/zconf.gperf b/kconfig/zconf.gperf
>>>> index c9e690e..21e79e4 100644
>>>> --- a/kconfig/zconf.gperf
>>>> +++ b/kconfig/zconf.gperf
>>>> @@ -7,6 +7,10 @@
>>>> %pic
>>>> %struct-type
>>> Can we ifdef on Darwin/BSD here? If it works on other platforms, then
>>> only the ones mentioned need stddef.h included here.
>>>
>>>> +%{
>>>> +#include <stddef.h>
>>>> +%}
>>>> +
>>>> struct kconf_id;
>>>>
>>>> static struct kconf_id *kconf_id_lookup(register const char *str, register unsigned int len);
>>>> --
>>>> 1.9.3 (Apple Git-50)
>>>>
>>>>
>>>> --
>>>> For unsubscribe information see http://sourceware.org/lists.html#faq
>>>>
>>> Otherwise: +1
>>>
>>> -Bryan
>>>
>>> --
>>> For unsubscribe information see http://sourceware.org/lists.html#faq
>>>
> Thanks,
>
> -Bryan


--
For unsubscribe information see http://sourceware.org/lists.html#faq
diff mbox

Patch

diff --git a/kconfig/zconf.gperf b/kconfig/zconf.gperf
index c9e690e..21e79e4 100644
--- a/kconfig/zconf.gperf
+++ b/kconfig/zconf.gperf
@@ -7,6 +7,10 @@ 
 %pic
 %struct-type
 
+%{
+#include <stddef.h>
+%}
+
 struct kconf_id;
 
 static struct kconf_id *kconf_id_lookup(register const char *str, register unsigned int len);