On 24/11/14 07:20, Torsten Bögershausen wrote: > On 2014-11-24 00.15, Ramsay Jones wrote: >> On 23/11/14 18:53, Junio C Hamano wrote: >>> Ramsay Jones <ram...@ramsay1.demon.co.uk> writes: >>> >>>> On 23/11/14 14:16, Torsten Bögershausen wrote: >>>>> gcc under cygwin reports several warnings like this: >>>>> warning: implicit declaration of function 'memmem' >>>>> [-Wimplicit-function-declaration] >>>>> This has been observed under CYGWIN-32 with GCC 4.7.3 as well >>>>> as CYGWIN-64 with gcc v4.8.3-5 x86-64 >>>> >>>> Heh, thanks for looking into this. Your email came at a good time, >>>> since I was just about to boot my old laptop into windows XP to >>>> test my patch on 32-bit cygwin! (If I had not been watching the >>>> F1 Grand Prix on TV, I would already have done so! ;-) ). >>>> >>>> It's been a while since I updated my 32-bit cygwin installation >>>> (about 6 months) but I'm a little surprised you found this issue >>>> with gcc 4.7.3 (I'm _almost_ tempted to boot that laptop anyway >>>> just to see what versions of software it is running). >>> >>> So you have an old installation to check how well the patched >>> version is accepted by the old set of header files? >> >> ... I can, indeed, use this old installation to test this on >> 32-bit cygwin. sigh, I thought I had dodged that bullet! ;-) >> > > It depends what we mean with "old": > cygwin 1.5 is old, and I lost my test installation this summer:
I updated from cygwin 1.5 to cygwin 1.7 at the beginning of the year. Since it is no longer supported, I don't think we need to worry about version 1.5. When I said 'old installation' I meant my old version 1.7 32-bit installation. > One netbook was converted from XP to Linux, the other machine needs to be > re-installed and CYGWIN 1.5 is no longer available for download. > > I can confirm that Ramsays patch works with CYGWIN 1.7 32 Bit. Thanks! ATB, Ramsay Jones -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
