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

Heroku deployment (flex missing) #31

Closed
mme opened this issue Aug 26, 2015 · 1 comment
Closed

Heroku deployment (flex missing) #31

mme opened this issue Aug 26, 2015 · 1 comment

Comments

@mme
Copy link

mme commented Aug 26, 2015

Heroku deployment is a bit painful at the moment (needs a custom buildpack).

Would it be reasonable to drop the flex requirement by providing a pregenerated scan.c file together with the downloaded source code?

@lfittl
Copy link
Member

lfittl commented Aug 26, 2015

I think thats reasonable, it would be nice to use it without needing the custom build pack.

However this might take a bit, since in order to do it cleanly I'd want to do it after #24 is done. I've been meaning to change the current approach of using a forked Postgres (mostly for the JSON output functions), to using upstream postgres + patch series.

As part of that I can see adding pre-generated scan.c/gram.c files that are used if flex/bison are not installed. I'll see what I can do & update you once I've made progress :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants