/diesel

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

cargo install diesel_cli does not work if you haven't configured sqlite #205

Closed
Shiney opened this Issue Feb 11, 2016 · 2 comments

Comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@Shiney @sgrif
@Shiney

Shiney commented Feb 11, 2016

If you haven't installed sqlite then (on windows) the out of the box cargo install diesel_cli will not work, due to not finidng the SQLITE3_LIB_DIR.

I'm not sure if this matters but it means that if you are just targeting PostGres you have to install an additional dependency.

sgrif added a commit that referenced this issue Feb 11, 2016

@sgrif
Remove the hard dependencies on SQLite and Postgres from CLI 
Right now it's impossible to install Diesel CLI on your system if you
don't have both of these dependencies installed. This continues to
support both by default, but allows you to opt out of one of them if you
specifically need to.

This differs from our other crates where we only target Postgres by
default. However, I want `cargo install diesel_cli` to "just work", and
this case doesn't affect the binary size or memory usage of your actual
application.

Fixes #205
7e05402

@sgrif sgrif referenced this issue Feb 11, 2016

Merged

Remove the hard dependencies on SQLite and Postgres from CLI #207

@sgrif sgrif closed this in #207 Feb 11, 2016

@sgrif

This comment has been minimized.

Sign in to view
Member

sgrif commented Feb 11, 2016

I've released version 0.5.1 to address this. You can install with cargo install diesel_cli --no-default-features --features postgres now.
@Shiney

This comment has been minimized.

Sign in to view

Shiney commented Feb 11, 2016

Thanks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment