PiRSquared17 / alembic

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

bootsrap failed on SlowLeopard #173

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
bootstrapping fails when compiling Alembic on SnowLeopard when try-compiling 
zlib, with the following error.

error: ‘unlink’ was not declared in this scope

Here's a patch to fix it.

diff -r 64de83fd8e91 build/bootstrap/zlib_trycompile/zlib_trycompile.cpp
--- a/build/bootstrap/zlib_trycompile/zlib_trycompile.cpp   Wed May 25 16:45:42 
2011 -0700
+++ b/build/bootstrap/zlib_trycompile/zlib_trycompile.cpp   Wed Jun 08 01:38:24 
2011 +0900
@@ -22,6 +22,9 @@
 #ifdef PLATFORM_WINDOWS
 #  include <unistd111.h>
 #endif
+#ifdef __APPLE__
+#  include <unistd.h>
+#endif
 #include <string.h>
 #include <assert.h>
 #include "zlib.h"

Original issue reported on code.google.com by syoyofuj...@gmail.com on 7 Jun 2011 at 4:39

GoogleCodeExporter commented 9 years ago
Hello,

Thanks so much for your submission to fix bootstrap.

For us to incorporate your change and if you wouldn't mind, we'd like to have 
you fill out one of the following Contributor's Agreements. If you are 
representing the efforts of a company or corporation, use the corporate 
version. If this is just on your behalf as part of your own private efforts, 
you can use the individual version.

Corp
http://code.google.com/p/alembic/downloads/detail?name=Alembic%20Contributor%20L
icense%20Agreement%20-%20Corporate%20.doc&can=2&q=

Individual
http://code.google.com/p/alembic/downloads/detail?name=Alembic%20Contributor%20L
icense%20Agreement%20-%20Individual.doc&can=2&q=

Original comment by scottmmo...@gmail.com on 8 Jun 2011 at 3:12

GoogleCodeExporter commented 9 years ago
Hi there, rather than use your proposed fix, I did the following:

http://code.google.com/r/ardent-embic/source/detail?r=cac336afc1557634fd7a77d7f1
ed2dd4dfaeed46&name=default

Basically, "if it's not Windows, then #include unistd.h".

This should go into the next official release.

Original comment by ard...@gmail.com on 20 Jun 2011 at 11:02