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 notes to make it obvious how to install Sqlite support #591

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mikeschinkel
Copy link

@mikeschinkel mikeschinkel commented Jun 7, 2021

After spending over an hour trying to figure out why sqlite was not available as a --db-type with the buffalo command I finally realized that the section that said "Custom installation with SQLite3 Support" meant "to get Sqlite support you MUST do a custom installation" instead of what I initially thought it meant which was "when doing a custom installation this is the way to do it with Sqlite support vs. the next section which shows how to do it without Sqlite support."

This pull request is my attempt to help the next person who comes along needing Sqlite support and is as confused by the install docs as I was.

After spending over an hour trying to figure out why `sqlite` was not available as
a --db-type with the `buffalo` command I finally realized that the section that said "Custom installation with SQLite3 Support" meant "to get Sqlite support you MUST do a custom installation" instead of what I initially thought it meant which was "when doing a custom installation this is the way to do it with Sqlite support vs. the next section which shows how to do it without Sqlite support." This pull request is my attempt to help the next person who comes along and is as confused by the docs as I was.
@mikeschinkel mikeschinkel requested a review from a team June 7, 2021 03:19
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.

1 participant