-
Notifications
You must be signed in to change notification settings - Fork 135
Does not seem to work on Heroku-16 #43
Comments
I was facing the issue as well on I'm using it together with
Rolling back to |
Same issue for me. I am not an expert in GDAL or GIS libraries, in fact I'm fairly new to these areas, but I dove a little into this today and tried to reproduce the problem that I'm getting on the stack-trace directly on the server:
I don't see libjasper anywhere in the geo libraries, so maybe this is what is missing? |
Similar issue for me. The stack is Heroku-16, Django 1.10, Python 3.5.2 I tried changing to cedar-14 but it would not solve it. Do you have any idea what could be wrong ? |
@dschep Can you make a branch without the changes to the readme and make a PR to this repo? I feel like a decent number of people use this repo as a buildpack and it would help a LOT of people to fix it :-) |
Can do. I'll make a PR this evening. |
Well, before I do that, would @cyberdelia have any interest in merging such a PR? They haven't responded to this issue. |
@dschep There is no guarantee this path is stable across two stack version, it would be better to vendor it this buildpack. I'm also unsure if adding the full file path will not break non-python libraries. |
I know that the copying of the libjasper file works across both stacks - I just submitted a PR to the official Python buildpack to that effect. |
Ah and I had no issues doing something like this in my GDAL_LIBRARY_PATH = '/app/.heroku/vendor/lib/libgdal.so' # or os.getenv('...') and overriding it in Heroku app configuration This could remove the need for changing the default GDAL_LIBRARY_PATH in the buildpack |
How exactly do we use the fix by @dschep? I've tried Using the buildpacks will throw heroku-16, django 1.11.8 Relevant settings:
|
Not sure. I'm not using the project I initially built this for and i'm having trouble with the current project. I've got Django running, but fails when trying to save a geometry in django admin with an if os.getenv('DYNO'):
GDAL_LIBRARY_PATH = os.path.expandvars(os.getenv('GDAL_LIBRARY_PATH'))
GEOS_LIBRARY_PATH = os.path.expandvars(os.getenv('GEOS_LIBRARY_PATH')) Hmm looking at what you tried i'll give it one last shot and update PROJ4's path too. (this is using Edit: nevermind about proj4 path. That's nota ctually a setting in django: https://github.com/django/django/search?utf8=%E2%9C%93&q=PROJ4_LIBRARY_PATH&type= |
Since heroku/heroku-buildpack-python@f767a73 landed in November, we've successfully been using heroku-16 and the plain heroku/python buildpack together with PostGIS. See https://devcenter.heroku.com/articles/postgis#geodjango-setup for what settings to define in your Django project. |
per @jodal's comment, I tried it out using only the original heroku/python buildpack, and settings as defined only this page here: https://devcenter.heroku.com/articles/postgis#geodjango-setup. And interestingly, it'll build up to the same point and throw the same error of So now I'm wondering whether following the info-box(on this page: https://devcenter.heroku.com/articles/postgis#geodjango-setup) which ultimately points to this miniconda on heroku example app, is an absolute necessity in order to make Geodjango on heroku work. Has this been the case for you @jodal? |
Ooh.. I didn't realize heroku now could just use docker images. that should make things easier. |
Woohoo. Tested out Heroku's Docker support. Soooo much easier!! I'm officially abandoning my fork of this buildpack. Buildpacks suck. This replaces the use of
|
@dschep Hey, question about your dockerized django proj on Heroku: did you end up needing a heroku.yml? Seems no matter what I do(even copying your Dockerfile), my Mind sharing a bit on how you got your image on Heroku? |
Hey @simonsterj. Nope, I don't have a This is the documentation I followed to get it working: https://devcenter.heroku.com/articles/container-registry-and-runtime |
@goinnn Worked for me as well. Thanks |
I'm currently trying out heroku-18. Have folks abandoned this build pack for docker? or would a PR for @goinnn changes be accepted? oh it seems like the generic python heroku build pack has the geospatial libs? heroku/heroku-buildpack-python#752 I'll be trying this out |
@morenoh149 I've been having good luck using the official buildpack for a while. Someone normally pings me when the geospatial libs break on that buildpack and I normally have it fixed on the next weekend :D edit: it normally doesn't break unless someone upgrades the distro version, which normally just requires a reshuffling of includes/lib paths |
@zachmccormick yes, I didn't report back but I now have geodjango running on heroku with the default python build pack. A related SO answer I made https://stackoverflow.com/a/53509910/630752 |
This helped me out when my rails server kept failing to deploy https://www.diowa.com/blog/heroku/2017/08/01/using-rgeo-with-geos-on-heroku-with-apt-get I think it was moving these two to the top of my Apt file libproj-dev And moving the "https://github.com/heroku/heroku-buildpack-apt" to be the first build pack Now I can deploy again.. |
We have two reports of this buildpack not working with Heroku-16, the next version of Heroku's stack based on Ubuntu 16.04:
https://devcenter.heroku.com/articles/heroku-16-stack
The text was updated successfully, but these errors were encountered: