On Tue, Apr 6, 2010 at 07:29, Takahiro Itagaki <itagaki.takah...@oss.ntt.co.jp> wrote: > > Tom Lane <t...@sss.pgh.pa.us> wrote: > >> > - cygwin_conv_to_full_win32_path(cmdLine, buf); >> > + cygwin_conv_path(CCP_POSIX_TO_WIN_A, cmdLine, buf, sizeof(buf)); >> >> Buildfarm member brown_bat didn't like this. Seeing that that's the >> *only* active cygwin buildfarm member, that's not a good percentage. > > Hmmm, but avoiding deprecated APIs would be good on the lastest cygwin. > How about checking the version with #ifdef? > > #ifdef __CYGWIN__ > /* need to convert to windows path */ > +#if CYGWIN_VERSION_DLL_MAJOR >= 1007 > cygwin_conv_path(CCP_POSIX_TO_WIN_A, cmdLine, buf, sizeof(buf)); > +#else > + cygwin_conv_to_full_win32_path(cmdLine, buf); > +#endif > strcpy(cmdLine, buf); > #endif
That seems like the way to do it. Or if it's used in many places, use a #define from one to the other - we don't want those #ifdef's all over the place. Seems cygwin may have deprecated that API a bit early :-), but there's nothing we can do about that. If it's deprecated, they'll eventually delete it... -- Magnus Hagander Me: http://www.hagander.net/ Work: http://www.redpill-linpro.com/ -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
