Hi Esteban. (The comments are again inline, I removed a lot of error messages to make it more readable.)
2014-03-11 16:22 GMT+01:00 Esteban Campostrini <[email protected]>: > 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/esteban/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? So, I'm having problems with fpos_t. It is not defined in stdio.h (which is > were it belongs) Does stdio.h should be extended > somehow to include this definitions? > 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. > FILENAME_MAX is also missing, is it OK to add it to stdio.h? > Yes. With some reasonable value :-). > [1]: + patch math.h to add declarations of modf, frexp, floor and pow, > since there was a file that was using them > Ok. Good. > + change the PNG_ZLIB_VERNUM so it maches ZLIB_VERNUM (from > zlib.h) otherwise it wouldn't compile. I'm > not sure about this change. > This is reasonable at least for now. 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 think you are doing well: you have already behind you the basic stuff and now comes the part where you actually extend HelenOS and see how to make old POSIX stuff work above new HelenOS API ;-). Cheers, - Vojta
_______________________________________________ HelenOS-devel mailing list [email protected] http://lists.modry.cz/listinfo/helenos-devel
