Skip to content

Fix a few minor build errors. - #45

Open
PuppyKevin wants to merge 3 commits into
iron-graveyard:masterfrom
PuppyKevin:master
Open

Fix a few minor build errors.#45
PuppyKevin wants to merge 3 commits into
iron-graveyard:masterfrom
PuppyKevin:master

Conversation

@PuppyKevin

Copy link
Copy Markdown

rust-crypto has changed names to just crypto, and Rust's extern crate syntax changed. Along with that, Travis now supports Rust.

@PuppyKevin PuppyKevin changed the title Fix a minor build error. Fix a few minor build errors. Sep 15, 2015
@reem

reem commented Sep 17, 2015

Copy link
Copy Markdown
Contributor

Given that this still doesn't compile at all, I'm not sure if I should merge this.

@PuppyKevin

Copy link
Copy Markdown
Author

I wouldn't yet - I started working on this before realizing that cookie needs to be brought up to date with Iron itself. I'm planning on getting around to that when I have a bit more free time.

@reem

reem commented Sep 18, 2015

Copy link
Copy Markdown
Contributor

Just let me know.

@SkylerLipthay

Copy link
Copy Markdown

@reem I don't know if you have any interest in internalizing this: oatmeal_raisin

@PuppyKevin I just checked and oatmeal_raisin doesn't compile with latest nightly, but it's practically modern and was built to replace this middleware. I'll be working with it again next week, so I'll fix it up. (It might compile on stable? I didn't check closely...)

Just want to avoid any unnecessary duplicated effort 😄

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.

3 participants