Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does not compile on Mac OS X Mavericks (clang) #3

Closed
thhomas opened this issue Oct 3, 2014 · 4 comments
Closed

Does not compile on Mac OS X Mavericks (clang) #3

thhomas opened this issue Oct 3, 2014 · 4 comments

Comments

@thhomas
Copy link

thhomas commented Oct 3, 2014

Many errors when trying to build on mavericks:

  • does not like use of ostringstream like ctb-extents.cpp:148, ctb-tile.cpp:184, ctb-tile.cpp:194, ctb-tile.cpp:207, ctb-tile.cpp:216
    error:
    cesium-terrain-builder-master/tools/ctb-extents.cpp:148:66: error: cannot take the address of an rvalue of type 'std::__1::basic_ostringstream<char, std::__1::char_traits, std::__1::allocator >'
    const string filename = dirname + static_cast<ostringstream*>( &(ostringstream() << zoom << ".geojson") )->str();
  • problem in instantiation of function template specialization package_task:
    ctb-tile.cpp:460:48: note: in instantiation of function template specialization
    'std::__1::packaged_task<int (TerrainBuild *, ctb::Grid *)>::packaged_task<int (&)(TerrainBuild *, ctb::Grid *), void>' requested here
    packaged_task<int(TerrainBuild *, Grid *)> task($runTiler); // wrap the function
@homme
Copy link
Member

homme commented Oct 8, 2014

Thanks for testing and the heads up. Unfortunately I don't work with Mac OS X so can't help out. I'll link this issue from the README, however, and it can be used to track progress: hopefully someone more knowledgeable will be able to help...

@tmizu23
Copy link
Contributor

tmizu23 commented May 16, 2015

I success the compile changing the code.
for example, I change the error code of tools/ctb-extents.cpp like this.

const string filename = dirname + (ostringstream() << zoom << ".geojson").str();

the error of ctb-tile.cpp is also the same.

@homme
Copy link
Member

homme commented May 20, 2015

@tmizu23 Thanks you. Are you able to provide a pull request for your changes?

@homme
Copy link
Member

homme commented Aug 6, 2015

Feel free to re-open this issue if additional improvements are required.

@homme homme closed this as completed Aug 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants