search

jube / mapmaker

a flexible heightmap generator
ISC License
27 stars 4 forks source link

Fixing some issues #3

Closed kdridi closed 10 years ago

kdridi commented 10 years ago

Here are some quick fixes in order to compile this interesting project properly.

jube commented 10 years ago

Why do you include Boost?

kdridi commented 10 years ago

Because it's a YAML dependency

In file included from /Users/kdridi/git/github.com/kdridi/mapmaker.git/src/bin/mapmaker/finalizers.cc:16: In file included from /Users/kdridi/git/github.com/kdridi/mapmaker.git/src/bin/mapmaker/finalizers.h:22: In file included from /usr/local/Cellar/yaml-cpp/0.5.1/include/yaml-cpp/yaml.h:13: In file included from /usr/local/Cellar/yaml-cpp/0.5.1/include/yaml-cpp/node/node.h:10: /usr/local/Cellar/yaml-cpp/0.5.1/include/yaml-cpp/node/ptr.h:10:10: fatal error: 'boost/shared_ptr.hpp' file not found

include <boost/shared_ptr.hpp>

     ^
jube commented 10 years ago

I would say that this is a yaml-cpp issue. But I will merge anyway.

jube commented 10 years ago

I improved your fix for printf, using a cinttypes macro because I had a warning. Tell me if it works fine for you. And thanks for your patch !