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

diesel_cli does not compile in travis container #238

Closed
jgillich opened this Issue Mar 18, 2016 · 7 comments

Comments

Projects
None yet
3 participants
@jgillich
Contributor

jgillich commented Mar 18, 2016

Installing diesel_cli with cargo on nightly and nightly-2016-03-11:

undefined reference to `sqlite3_errstr'

https://travis-ci.org/jgillich/mp/jobs/116882725

I've tried installing libsqlite3-dev, does not help. No idea what is going on here, it works witth sudo: required for some reason...

@mcasper

This comment has been minimized.

Collaborator

mcasper commented Mar 18, 2016

It looks like you're using postgres, so you should be able to work around this with

cargo install diesel_cli --no-default-features --features postgresql

Unfortunately diesel_cli by default compiles all the different connection types, which requires those backends to be installed. Still working on a way around this so that cargo install diesel_cli will always just work.

@jgillich

This comment has been minimized.

Contributor

jgillich commented Mar 18, 2016

What exactly would I have to install? I've added libsqlite3-dev, is there anything that would be needed?

I do plan on using both in the future actually.

@mcasper

This comment has been minimized.

Collaborator

mcasper commented Mar 18, 2016

libsqlite3-dev should be plenty, I think the sudo: required was the real ticket. (Vaguely) According to Travis docs, it looks like that's required to install any packages: Allows use of sudo (e.g., for installing apt packages).

However, it looks like we don't even install sqlite in our .travis.yml, we just have sudo: required, so that may be all you need.

@jgillich

This comment has been minimized.

Contributor

jgillich commented Mar 18, 2016

No, you can set addons.apt.packages when you're using containers. Check the build log (Installing APT Packages), it was definitely installed. So yes, the issue is that diesel_cli only builds in VMs.

@mcasper

This comment has been minimized.

Collaborator

mcasper commented Mar 18, 2016

Ah, good point. I'll investigate this tonight

@sgrif

This comment has been minimized.

Member

sgrif commented Mar 20, 2016

Travis uses an ancient version of SQLite and you have to go through contortions to get it to work (see our travis.yml's history). However, the "update SQLite" issue on the travis package whitelist repo was recently closed, so they may have fixed it.

@sgrif

This comment has been minimized.

Member

sgrif commented Mar 20, 2016

What exactly would I have to install?

Any version of libsqlite that is more recent than 2012

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