Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

Set rust frameworks' database property to None type to see if that broke travis CI#1650

Merged
msmith-techempower merged 1 commit intoTechEmpower:masterfrom
waiteb3:rust_repairs
Jun 10, 2015
Merged

Set rust frameworks' database property to None type to see if that broke travis CI#1650
msmith-techempower merged 1 commit intoTechEmpower:masterfrom
waiteb3:rust_repairs

Conversation

@waiteb3
Copy link
Copy Markdown
Contributor

@waiteb3 waiteb3 commented Jun 5, 2015

Travis CI broke on Rust, which may be caused by using "" rather than "None" as the database config.

@waiteb3 waiteb3 changed the title Set database to None type to see if that broke travis CI Set rust frameworks' database property to None type to see if that broke travis CI Jun 5, 2015
@zane-techempower
Copy link
Copy Markdown
Contributor

Looks like I made the same fix in #1654 with the addition of making the frameworks "versus": "rust" as well. However I see that the tests are still failing due to lack of cargo:

Setup iron: + cargo build --release
Setup iron: setup.sh: line 6: cargo: command not found

Lack of cargo also appears to be the latest development in #1636 so I will leave it to the rust crew to settle this.

@msmith-techempower
Copy link
Copy Markdown
Member

Looks good, but cargo missing is part of the travis installation issue (SSL handshake is failing for some reason); LGTM!

msmith-techempower added a commit that referenced this pull request Jun 10, 2015
Set rust frameworks' database property to None type to see if that broke travis CI
@msmith-techempower msmith-techempower merged commit 8bce921 into TechEmpower:master Jun 10, 2015
@waiteb3 waiteb3 deleted the rust_repairs branch June 10, 2015 18:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants