CesiumGS / cdb-to-3dtiles

Convert CDB to 3D Tiles
Apache License 2.0
75 stars 28 forks source link

CDBConverter does not compile #53

Open Greg13006 opened 3 years ago

Greg13006 commented 3 years ago

Hi,

i have followed the Getting Start below in a clean Ubuntu 20.10 machine: https://github.com/CesiumGS/cdb-to-3dtiles#getting-started

When i try to compile i have several errors such as:


[ 99%] Building CXX object CDBTo3DTiles/CMakeFiles/CDBTo3DTiles.dir/src/CDBGeoCell.cpp.o
In file included from /home/diginext/CDB_To_3DTiles/cdb-to-3dtiles/CDBTo3DTiles/src/CDBGeoCell.h:3,
                 from /home/diginext/CDB_To_3DTiles/cdb-to-3dtiles/CDBTo3DTiles/src/CDBGeoCell.cpp:1:
/home/diginext/CDB_To_3DTiles/cdb-to-3dtiles/CDBTo3DTiles/include/Utility.h:15:13: error: ‘string’ in namespace ‘std’ does not name a type
   15 | inline std::string toStringWithZeroPadding(size_t numOfZeroes, T num)
      |             ^~~~~~

and:


[ 99%] Building CXX object CDBTo3DTiles/CMakeFiles/CDBTo3DTiles.dir/src/CDBTile.cpp.o
/home/diginext/CDB_To_3DTiles/cdb-to-3dtiles/CDBTo3DTiles/src/CDBGeoCell.cpp: In member function ‘std::string CDBTo3DTiles::CDBGeoCell::getLongitudeDirectoryName() const’:
/home/diginext/CDB_To_3DTiles/cdb-to-3dtiles/CDBTo3DTiles/src/CDBGeoCell.cpp:95:22: error: ‘toStringWithZeroPadding’ was not declared in this scope
   95 |         return "W" + toStringWithZeroPadding(3, -m_longitude);
      |                      ^~~~~~~~~~~~~~~~~~~~~~~

How to fix this ? Is there a prebuild version somewhere ?

Regards,

lilleyse commented 3 years ago

@Greg13006 Good timing, I'm about to open a PR that fixes builds for clang 10 which may fix the errors you're getting. What OS and compiler are you using?

lilleyse commented 3 years ago

@Greg13006 can you try again now? https://github.com/CesiumGS/cdb-to-3dtiles/pull/54 was merged.

Greg13006 commented 3 years ago

Thanks for your quick action. i will not be able to try again before monday.