Re: [PATCH] cygwin: define PATH_MAX for host tools build

[Spencer Oliver <spen at spen-soft dot co dot uk>]

On 12/05/2010 16:06, John Dallaway wrote:
> Hi Spencer
>
> Jonathan Larmour wrote:
>
> > Spencer Oliver wrote:
> >
> > > cygwin: define PATH_MAX for host tools build
> >
> > Wouldn't this be better in ecutils.cpp? It already has an ifdef
> > __CYGWIN__ block for cygwin_conv_to_*_path() which is the same place
> > PATH_MAX is used.
>
> I've been unable to find a build issue for which this patch might be
> required. Is there a specific eCos host tools source file which is
> failing to build for you? Can you provide the relevant part of your
> build log and the version of Cygwin that you are using please?

using cygwin 1.7.1 gcc 4.4.3
issue is when building configtool

c++ -O2 -c -ITCLDIR_use_system/include 
-I/cygdrive/c/ecos/configtool/build/tools/include 
-I/cygdrive/c/ecos/ecos.hg/host/tools/configtool/common/common 
-I/cygdrive/c/ecos/ecos.hg/host/tools/Utils/common 
-I/cygdrive/c/ecos/ecos.hg/host/tools/ecostest/common 
-DecUSE_EXPERIMENTAL_CODE=1 
`/cygdrive/c/ecos/configtool/build/wxlib/bin/wx-config --cppflags` 
-D_WIN32 -D__WIN32__ -DSTRICT -D__USE_W32_SOCKETS -o 
/cygdrive/c/ecos/configtool/build/configtool-build/eCosTestPlatform.o 
/cygdrive/c/ecos/ecos.hg/host/tools/ecostest/common/eCosTestPlatform.cpp
/cygdrive/c/ecos/ecos.hg/host/tools/ecostest/common/eCosTestPlatform.cpp: In 
static member function ‘static bool CeCosTestPlatform::LoadFromDir(const 
char*)’:
/cygdrive/c/ecos/ecos.hg/host/tools/ecostest/common/eCosTestPlatform.cpp:61: 
error: ‘PATH_MAX’ was not declared in this scope
/cygdrive/c/ecos/ecos.hg/host/tools/ecostest/common/eCosTestPlatform.cpp:62: 
error: ‘szOrigDir’ was not declared in this scope
make: *** 
[/cygdrive/c/ecos/configtool/build/configtool-build/eCosTestPlatform.o] 
Error 1

Cheers
Spen