matrakim / opencollada

Automatically exported from code.google.com/p/opencollada
0 stars 0 forks source link

Small fix to compile with gcc 4.2 while keeping gcc 4.4 and greater working #133

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Issue was already fixed once:

Problen is hash_fun.h was moved. Between gcc 4,3 and 4.4

Index: COLLADAFramework/include/COLLADAFWHashFunctions.h
===================================================================
--- COLLADAFramework/include/COLLADAFWHashFunctions.h   (revision 779)
+++ COLLADAFramework/include/COLLADAFWHashFunctions.h   (working copy)
@@ -15,7 +15,11 @@
 #include "COLLADABUPlatform.h"

 #ifdef COLLADABU_OS_LINUX
-#  include <backward/hash_fun.h>
+#if ((__GNUC__ < 4) || (__GNUC__ == 4 && __GNUC_MINOR__ <= 3))
+#  include <ext/hash_fun.h>
+#else
+#   include <backward/hash_fun.h>
+#endif
 #   include "COLLADAFWUniqueId.h"
 #elif (defined COLLADABU_OS_MAC)
 #  include <ext/hash_fun.h>

Original issue reported on code.google.com by martijn....@gmail.com on 8 Nov 2010 at 10:03

GoogleCodeExporter commented 8 years ago
Also please see issue 152, which affects this issue.

Original comment by carsten....@cafu.de on 16 Jan 2011 at 1:09

GoogleCodeExporter commented 8 years ago
Could you please check if this problem still exists or if the patch provided in 
issue 152 and applied in r798 has fixed this problem for current versions of 
gcc?

Original comment by opencollada2@googlemail.com on 18 Jan 2011 at 3:48

GoogleCodeExporter commented 8 years ago
Comment 1 by martijn....@gmail.com, Jan 16 (2 days ago)
Fixes issue 133 for me. 

That is building with gcc 4.2.4 ubuntu 8.04 on both x86 and amd64.

Yes patch in issue 152 fixed all issues as reported. And its a much nicer way 
of solving them

Original comment by martijn....@gmail.com on 19 Jan 2011 at 3:20

GoogleCodeExporter commented 8 years ago
Thanks for the verification.

Original comment by opencollada2@googlemail.com on 19 Jan 2011 at 3:43