Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#432 closed defect (fixed)

Make error [2]

Reported by: Sylvain Owned by:
Priority: Should Have Milestone:
Component: Core engine Keywords:
Cc: Patch:

Description

I got an error while compilation with AtlasUi in "obj/AtlasUI_Debug/DLLInterface.o"

Debug/precompiled.h ../../../source/tools/atlas/AtlasUI/Misc/DLLInterface.cpp ../../../source/tools/atlas/AtlasUI/Misc/DLLInterface.cpp:48:2: error: #error libxml2 must have threading support enabled make[1]: * [obj/AtlasUI_Debug/DLLInterface.o] Erreur 1 make: * [AtlasUI] Erreur 2

I'm on Ubuntu Karmic 9.10 tell me if you need more information this is my first report and my english sucks !

Change History (12)

comment:1 by Sylvain, 14 years ago

Building AtlasUI

tex_bmp.cpp DLLInterface.cpp ia32.cpp ia32_asm.asm ia32_memcpy.asm topology.cpp x86_x64.cpp ldbg.cpp ../../../source/tools/atlas/AtlasUI/Misc/DLLInterface.cpp:48:2: error: #error libxml2 must have threading support enabled linux.cpp dir_watch_fam.cpp lcpu.cpp printf.cpp unix.cpp udbg.cpp x.cpp make[1]: * [obj/AtlasUI_Debug/DLLInterface.o] Erreur 1 make: * [AtlasUI] Erreur 2 make: * Attente des tâches non terminées.... gcc.cpp Linking lowlevel

comment:2 by Sylvain, 14 years ago

ok, first I made a copy of this file : /usr/include/libxml2/libxml/xmlversion.h and I uncomment the THREAD option in. The compilation seems to be good !

comment:3 by Sylvain, 14 years ago

should be heaven...

pyrogenesis-dbg doesn't appears, so i made ./update-workspace.sh and then another error :

"lot f error /usr/include/libxml2/libxml/hash.h:33: error: expected constructor, destructor, or type conversion before ‘extern’ In file included from /usr/include/libxml2/libxml/valid.h:15,

from /usr/include/libxml2/libxml/parser.h:19, from /usr/include/libxml2/libxml/globals.h:18, from /usr/include/libxml2/libxml/threads.h:35, from /usr/include/libxml2/libxml/xmlmemory.h:218, from /usr/include/libxml2/libxml/tree.h:1248, from FCollada/FUtils/FUtils.h:91, from FCollada/FCollada.h:49, from FCollada/StdAfx.h:13, from FCollada/FCollada.cpp:9:

/usr/include/libxml2/libxml/xmlerror.h:16: error: expected constructor, destructor, or type conversion before ‘extern’ FCollada/FCollada.cpp:170: error: expected ‘}’ at end of input make: * [output/debug/FCollada/FCollada.o] Erreur 1 ERROR: FCollada build failed"

comment:4 by Philip Taylor, 14 years ago

Sounds like the original error is the same as this forum post. You shouldn't edit xmlversion.h - that won't fix the real problem (that Ubuntu 9.10's libxml2 doesn't support threading) and it might result in weird random bugs and crashes when you run the game. Also, the new error sounds like it's probably caused by the edits to xmlversion.h.

As a temporary solution, you can restore xmlversion.h to the original unmodified version, then run update-workspaces.sh again and compile with "make pyrogenesis Collada" (instead of just "make"). That will stop it compiling the map editor, which is where the problems occur, so hopefully there won't be any errors.

We ought to find a better solution for this on Ubuntu, but I'm not sure yet what we should do. (Sorry for the trouble!)

comment:5 by Sylvain, 14 years ago

thank you very much, I did it and it's working now

in reply to:  5 comment:6 by Luciano, 14 years ago

Yes, the bug's listed here https://bugs.launchpad.net/ubuntu/+source/libxml2/+bug/309149

I suscribed for mail notification if the bug state changes, will keep you posted.

comment:7 by Luciano, 14 years ago

Resolution: fixed
Status: newclosed

Hi!

Good news: libxml2 was updated with thread support in Ubuntu backports. I updated and compilation with map editor was successful (notice https://bugs.launchpad.net/ubuntu/+source/libxml2/+bug/309149).

The package is as backport-proposed in Ubuntu so it will be included by default later (maybe in the next release). Must enable backports-proposed in Ubuntu repositories (https://wiki.ubuntu.com/Testing/EnableProposed) and "apt-get install libxml2-dev".

By the way if you are Ubuntu user you can test it keeping the package installed for some days and then give feedback telling your experience as requested in the bug report, so maybe it can be included faster by default.

Good luck,

comment:8 by Luciano, 14 years ago

Resolution: fixed
Status: closedreopened

comment:9 by Luciano, 14 years ago

Resolution: fixed
Status: reopenedclosed

comment:10 by Luciano, 14 years ago

Sorry by the status changes, I did a mistake ;) .

comment:11 by Philip Taylor, 14 years ago

Milestone: Pre-AlphaUnclassified

Milestone Pre-Alpha deleted

comment:12 by (none), 14 years ago

Milestone: Unclassified

Milestone Unclassified deleted

Note: See TracTickets for help on using tickets.