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

Add makefile error if diesel_cli is not installed #144

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

v0idpwn
Copy link
Collaborator

@v0idpwn v0idpwn commented Sep 10, 2023

diesel_postgres, diesel_mysql and diesel_sqlite will loop forever if the diesel_cli dependency isn't installed. This PR adds a step that verifies tha diesel_cli is installed and returns an error otherwise.

Comment on lines +52 to +53
ensure_diesel:
@command -v diesel >/dev/null 2>&1 || (echo "diesel_cli not installed, can't continue" && exit 1)
Copy link
Collaborator Author

@v0idpwn v0idpwn Sep 10, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think maybe we could require ensure_diesel for db. It isn't really required, but in the current setup, the containers are started even if diesel_cli isn't installed, and they aren't automatically killed on error. This could be avoided by depending on ensure_diesel for db.

@ayrat555 ayrat555 merged commit 4bb21c4 into ayrat555:master Sep 11, 2023
@v0idpwn v0idpwn deleted the fix/error-if-missing-diesel-cli branch September 11, 2023 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants