Re: [Bug-wget] [PATCH] Regression since wget 1.10: no_prefix function is *bad*

[Tim Rühsen]

Hi Alex,

yes, it is the _PC_NAME_MAX issue which is only valid for pathconf().

Attached is the little patch to fix it.

Since MinGW is based on gcc-4.6, C99 should be available.
As the Wiki states (well, the entry is 3 years old...), printf() is the main 
issue. But there may be some other functions that don't behave C99 compliant.
Sound not like a compiler, but a library issue.

Maybe some functions have to be provided by Wget. If we just had a list of 
issues/functions...

Regards, Tim

Am Montag, 13. Mai 2013 schrieb Bykov Aleksey:
> Greetings, Tim Rühsen.
> Possible that i'm understood wrong, but according to
> http://www.mingw.org/wiki/C99 MinGW doesn't support C99 at all. So i'm not
> sure about cross-compile.
> May be after implementation of C99 code it can be compiled only through
> CygWin (don't remember exactly, but two years ago Wget compiled at CygWin
> required on pure Windows only several libraries as dependencies. If need,
> i'll try to check this at week).
> 
> P.S. Currently Wget compiled without any problem in Windows MinGW and
> resulting binary work in pure Windows without any dependencies.
> 
> P.P.S. What "minor compile issue in url.c"? I get only error with
> undefined PC_NAME_MAX (just add it to compiler flags)?
> 
> P.P.P.S. Sorry for bad English.
> 
> Best regards, Alex.
> 
> > Thanks for the hint. I just installed Debians MinGW packages and it  
> > workedlike a charm (except a minor compile issue in url.c).
> >
> > For anyone who wants to try:
> > $ make distclean
> > $ ./configure --host=i686-w64-mingw32 --without-ssl --disable-ipv6
> > $ make
> > $ wine src/wget.exe http://www.example.com
> >
> > I have no real Windows installation around to test wget.exe.
> > But assuming, it works: Is there any need to stick at c89 code ?
> > Or in other words: do we still have to support native Windows  
> > compilation withMSVC (couldn't these people install and use mingw) ?
> >
> > Regards, Tim
>

0001-src-utils.c-cleanup.patch
Description: Text Data