-
- Notifications
You must be signed in to change notification settings - Fork 212
chore: install pg_graphql extension #152
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
Conversation
| - name: pg_graphql - download libgraphqlparser | ||
| git: | ||
| repo: https://github.com/graphql/libgraphqlparser.git | ||
| dest: /tmp/libgraphqlparser | ||
| version: master | ||
| become: yes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@olirice do you have a recommendation/ preferred way to install libgraphqlparser? Their repository hasn't had any new releases since 2017 so the above is just building off from the master branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
preferred way to install libgraphqlparser
I've been pinning it to the last realease 0.7.0. Heres the install line in the dockerfile used for the quickstart
https://github.com/supabase/pg_graphql/blob/75565af8ef81f0d4e1cf281356d7270c2c6d5a1d/dockerfiles/db/Dockerfile#L13
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got it 👍
| pkg: | ||
| - clang-11 | ||
| - build-essential | ||
| - python |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
python2 is a dependency for installing libgraphqlparser. It shouldn't be required by pg_graphql. did it cause an issue for you?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, this is more of a catch all for all packages that would be used by the extension and any other dependencies.
| I've updated the steps to instead emulate the steps taken in the Dockerfile. One additional step is to run
|
| All good to go. @oli will let you approve and merge this one in once the final |
* install pg_graphql extension * install dependencies * update steps to reflect how the docker example is built * update pg_graphql to release v0.1.0 * omit ./bin/pgc build * add pg_graphql to list of extensions to be installed * bump pg_graphql version to v0.1.1 * update pg_graphql release from 0.1.1 to 0.1.2 Co-authored-by: Oliver Rice <github@oliverrice.com>
* install pg_graphql extension * install dependencies * update steps to reflect how the docker example is built * update pg_graphql to release v0.1.0 * omit ./bin/pgc build * add pg_graphql to list of extensions to be installed * bump pg_graphql version to v0.1.1 * update pg_graphql release from 0.1.1 to 0.1.2 Co-authored-by: Oliver Rice <github@oliverrice.com>
Installs the
pg_graphqlextension.