Re: wip/ncc version 2.7 -> 2.8

"OBATA Akio" <[email protected]>
Newsgroups gmane.os.netbsd.devel.pkgsrc.wip.review
Message-ID <[email protected]>
On Sat, 18 Oct 2008 08:24:40 +0900, Marko Schütz <[email protected]> wrote:

> At Fri, 17 Oct 2008 16:16:15 -0500,
> Larson, Timothy E  wrote:
>>
>> On Friday, October 17, 2008 3:48 PM, [email protected]
>> <mailto:[email protected]> scribbled:
>>
>> > At Fri, 17 Oct 2008 14:27:07 -0500,
>> > Larson, Timothy E  wrote:
>> >> This does not build on Solaris 10.  First guess is because preproc.C
>> >> does not include strings.h, where rindex is declared.
>> >
>> > added that include. Please check again.
>>
>> Now builds on Solaris 10.  Advise checking on other platforms to make sure
>> that doesn't result in conflicts (i.e. the include may have to be
>> platform-conditional or something).
>
> Thanks for checking it so quickly.
>
> Can someone else check on different platforms?

Or instead of include strings.h, replace rindex with strrchr.
It should be defined in string.h and more portable.

-- 
"Of course I love NetBSD":-)
OBATA Akio / [email protected]

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
pkgsrc-wip-review mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-review
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.