Hi Vojta,


>> so, after some modificatios[1] based on trial and error during the configure 
>> and make phases the compilation gets to
>> this point
>>
>>     /usr/local/cross/ia32/bin/i686-pc-linux-gnu-gcc -DHAVE_CONFIG_H -I.   
>> -imacros 
>> /home/esteban/helenos/coastline/build/libpng/libpng-1.6.10/helenos_config.h  
>> -D__LE__ -I/home/esteban/helenos/coastline/helenos/include/posix 
>> -I/home/esteban/helenos/coastline/helenos/include/  -O3 -imacros 
>> /home/esteban/helenos/coastline/helenos/include/system_config.h 
>> -fexec-charset=UTF-8 -fwide-exec-charset=UTF-32LE -finput-charset=UTF-8 
>> -ffreestanding -fno-builtin -nostdlib -nostdinc -Wall -Wextra -Wno-clobbered 
>> -Wno-unused-parameter -Wmissing-prototypes -std=gnu99 -Wwrite-strings -pipe 
>> -ggdb -D__LE__ -march=pentium -fno-omit-frame-pointer 
>> -I/home/esteban/helenos/coastline/helenos/include/posix 
>> -I/home/esteban/helenos/coastline/helenos/include/  
>> -Wl,-L/home/esteban/helenos/coastline/helenos/lib -Wl,-n -Wl,-T 
>> -Wl,/home/esteban/helenos/coastline/helenos/link.ld -Wl,--whole-archive 
>> -Wl,--start-group -Wl,-lposixaslibc -Wl,-lsoftint -Wl,--end-group 
>> -Wl,--no-whole-archive -Wl,-lc4posix -Wl,-L/home/est
 eban/helenos/coastline/libs 
-Wl,-L/home/esteban/helenos/coastline/build/libpng/libpng-1.6.10/fixedlibs 
-Wl,--whole-archive -Wl,-lfdm -Wl,--no-whole-archive -MT contrib/tools/pngfix.o 
-MD -MP -MF $depbase.Tpo -c -o contrib/tools/pngfix.o contrib/tools/pngfix.c &&\
>>     mv -f $depbase.Tpo $depbase.Po
>> contrib/tools/pngfix.c:868:4: error: unknown type name 'posix_fpos_t'
>>     posix_fpos_t         data_pos;      /* Position of first byte of chunk 
>> data */
>>     ^
>
> This is rather weird. Because inside the file there is fpos_t and I do not 
> understand how is it possible that it reports posix_fpos_t. At this phase it 
> shall not see the posix_ prefix at all. What happens if you run only the 
> preprocessor?

My bad here, I replaced beforehand the occurrences of fpos_t with
posix_fpos_t after grepping in HelenOS headers. I forgot to
take that out to reproduce the original error. I'm sorry.
This is actually the original error:

contrib/tools/pngfix.c:868:19: error: field 'data_pos' has incomplete type
    fpos_t         data_pos;      /* Position of first byte of chunk data */
                   ^
contrib/tools/pngfix.c:1550:19: error: field 'chunk_data_pos' has
incomplete type
    fpos_t         chunk_data_pos;    /* Position of first byte of chunk data */
                   ^
contrib/tools/pngfix.c: In function 'zlib_check':
contrib/tools/pngfix.c:2615:11: error: storage size of 'start_pos' isn't known
    fpos_t start_pos;
           ^
I don't really get what is wrong, my guess is some header is missing
but I'm not quite sure, stdio.h is included in pngfix.

> Actually, it is there:
> typedef struct _posix_fpos __POSIX_DEF__(fpos_t);
> and
> struct _posix_fpos {
>     off64_t offset;
> };
> the problem is somewhere around this. But I do not see it now.

Yes, I saw that and I replaced fpos_t with _posix_fpos since I thought
the problem was that it couldn't find a declaration
but it looks that there's something else to be done.

> I have a suggestion. How about forking the coastline repo and committing the 
> harbour there (even if it does not work)? It would help me quite a lot when 
> reproducing this. Regarding the patch to math.h - keep it there as well with 
> comment that it has to be applied to mainline. How about that?

I totatlly agree. I already forked it and commited the harbour, the
patch and some notes.

Cheers,
Esteban

_______________________________________________
HelenOS-devel mailing list
[email protected]
http://lists.modry.cz/listinfo/helenos-devel

Reply via email to