[Tile-serving] [openstreetmap/osm2pgsql] Appveyor enhancement (#820)

Alireza Dabiri Nejad notifications at github.com
Mon Feb 12 21:01:08 UTC 2018


> Could you elaborate about the odd Postgres setup? Is there a particular reason why not to build against Postgers 9.6 as well? Would we run into issues with backwards compatibility?

Well, during my tries to make a successful build, I've used Postgres 9.6 64bit (without running the unit tests), and when used the .exe to run an import, an error showed up (can't remember the error now, will try to build again to reproduce error), so decided to use 9.4.7 for building. I just want to update appveyor config for a more mature auto builds, as a starting point, so can make a pull request and discuss about other things here. About the backwards compatibility, as far as I know, there is no issue. But I'll search to get a 100% trust-able answer.

> Do you think wingetopt is small enough that we could simply build it as part of the appveyor script as well? 
Yes. It's tiny. Just 1 cpp file. And also cmake compatible.

> It would be nice to have all dependencies available via automatic builds.
You mean that we build every single one during the build process? Well need to consider more things I think. 
List of dependencies that needed:
- expat
- proj
- bzip2
- zlib
- Lua
- wingetopt
- PostgreSQL
- Boost libraries

Well Postgres and Boost are huge libraries. Also Boost is available in appveyor vms. So i think, we could create a dependencies zip file from dependencies except Postgres and Boost, for x86 and x64, host it somewhere, and download and use it during build process (All the files needed to build for both arch, except Postgres and Boost, are zipped and saved as an artifact). By using this approach, updating the dependencies become a trivial task (Just change the content of zip file without even touching the cmake or appveyor config files).

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/820#issuecomment-365060388
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20180212/3c7f4a5a/attachment.html>


More information about the Tile-serving mailing list