On 04/01/2013 04:48 PM, Eric Blake wrote:
> On 04/01/2013 04:36 PM, LRN wrote:
>>>> Apparently, the MSVCRT implementation  of fseek() is used.
>>>>
>>>>> If not, then we should fix the m4 tests to filter out the
>>>>> buggy W32 fseek and install the gnulib replacement.
>>>>
>>>> I'm unsure of how they work.
>>
>>> What does 'grep _FSEEK config.status' show?
>> $ grep _FSEEK config.status
>> S["REPLACE_FSEEKO"]="0"
>> S["REPLACE_FSEEK"]="0"
> 
> Well, apparently the m4 files don't think they need to replace fseek...
> 
>> S["HAVE_FSEEKO"]="1"
>> S["HAVE_DECL_FSEEKO"]="1"
>> S["GNULIB_FSEEKO"]="1"
>> S["GNULIB_FSEEK"]="1"
> 
> ...even though the gnulib modules are in use...
> 
>> D["HAVE_FSEEKO"]=" 1"
>> D["HAVE_DECL_FSEEKO"]=" 1"
>> D["GNULIB_TEST_FSEEK"]=" 1"
>> D["GNULIB_TEST_FSEEKO"]=" 1"
>> D["HAVE_RAW_DECL_FSEEKO"]=" 1"
>>
>>> How about 'grep LSEEK_PIPE config.h'?
>> $ grep LSEEK_PIPE config.status
>> D["LSEEK_PIPE_BROKEN"]=" 1"
> 
> ...and even though we know seek is broken on the platform.  Looks like
> I'll have to figure out what went wrong; I'll probably have a patch in
> the next 24 hours or so.

I'm still stumped on how to reproduce this; on latest gnulib.git
installed in a Cygwin setup, I did:

./gnulib-tool --create-testdir --dir=testdir0 fseeko

then in that directory did a configure with a cross compiler:

./configure --host=i686-pc-mingw32 --build=i686-pc-cygwin

and the resulting build passed all tests.  Looking in config.status, I see:

S["REPLACE_FSEEKO"]="0"
S["REPLACE_FSEEK"]="1"
S["HAVE_FSEEKO"]="0"
S["HAVE_DECL_FSEEKO"]="0"

which differs from your setup.  Maybe I should repeat my tests with
mingw64 instead of the older mingw32?

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to