Hi Tommi,
Thank you for applying the patch. I have tried, and noticed the following problems: 1. uuid does not compile due to new _WIN32 define. 2. Error that getpid() is not defined in uuid. This is why unistd.h has to be included. I'd expect that this is not specific to symbian, but that the problems occurs if cxxtools are disabled, which is the case for symbian builds. In difference to the first patch I moved it to the else branch of the _WIN32 clause, as it probably is not available on windows. (getpid is defined to some other function on windows). 3. content bld.inf and zimlib.mmp is appearantly doubled up.
Attached a patch which fixes these problems. The patch additionally removes bzip2 dependency.
Could you apply the patch?
Best regards, Christian
Am 24.03.2010 00:20, schrieb Tommi Mäkitalo:
Hi Christian,
after some changes in zimlib I decided to apply your patch. Unfortunately there are some problems.
I changed zim.h not to query _WIN32, so there is no need to undef it there. Can you check, if it needs to be undefined elsewhere? It is checked in src/uuid.cpp. Is it a problem? Maybe we need to undefine it there?
What about unistd.h? Why it is needed it uuid.cpp?
I added the other files into the repository.
Tommi _______________________________________________ dev-l mailing list dev-l@openzim.org https://intern.openzim.org/mailman/listinfo/dev-l