Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 01 Nov 2013 13:55:27 -0400
From:      "Jason E. Hale" <jhale@freebsd.org>
To:        marino@freebsd.org
Cc:        svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org
Subject:   Re: svn commit: r332099 - head/devel/cssc/files
Message-ID:  <5558324.uLoFuByiHu@mocha.verizon.net>
In-Reply-To: <5272FBC0.2050306@marino.st>
References:  <201310301246.r9UCkiWu069471@svn.freebsd.org> <5271A160.8050309@marino.st> <5272FBC0.2050306@marino.st>

next in thread | previous in thread | raw e-mail | index | archive | help
On Friday, November 01, 2013 01:54:24 John Marino wrote:
> 
> Well, the maintainer patch does NOT contain <unistd.h>, contrary to what
> you said earlier so now it's broken again with gcc47+.
> 
> Can you please retrieve the contents of my patch and integrate it into
> the maintainer patch?
> 
I'm not sure why it would break.  The following is in the patched file:

#elif defined(__FreeBSD__)
#define GTEST_OS_FREEBSD 1
#define GTEST_USE_OWN_TR1_TUPLE 1
#endif  // __CYGWIN__

#if GTEST_OS_CYGWIN || GTEST_OS_LINUX || GTEST_OS_MAC || GTEST_OS_SYMBIAN || \
    GTEST_OS_SOLARIS || GTEST_OS_AIX || GTEST_OS_FREEBSD

// On some platforms, <regex.h> needs someone to define size_t, and
// won't compile otherwise.  We can #include it here as we already
// included <stdlib.h>, which is guaranteed to define size_t through
// <stddef.h>.
#include <regex.h>  // NOLINT
#include <strings.h>  // NOLINT
#include <sys/types.h>  // NOLINT
#include <time.h>  // NOLINT
#include <unistd.h>  // NOLINT

So, <unistd.h> is included.  I just ran a build test with gcc48 and it built 
fine.
-- 
Jason E. Hale - jhale@
FreeBSD Ports Committer
KDE/FreeBSD Team



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5558324.uLoFuByiHu>