Skip to content

Enable feature to fix unstabilized #[bench]#26

Closed
vorot93 wants to merge 1 commit into
http-rs:masterfrom
vorot93:patch-1
Closed

Enable feature to fix unstabilized #[bench]#26
vorot93 wants to merge 1 commit into
http-rs:masterfrom
vorot93:patch-1

Conversation

@vorot93

@vorot93 vorot93 commented Aug 5, 2019

Copy link
Copy Markdown

@Nemo157

Nemo157 commented Aug 5, 2019

Copy link
Copy Markdown
Contributor

This will cause this crate to become nightly only, #25 is a better fix that will still allow building on stable 1.38+.

@vorot93

vorot93 commented Aug 5, 2019

Copy link
Copy Markdown
Author

Inspired by the fix in cargo, missed the one you linked. Feel free to close if that one gets merged.

@vorot93

vorot93 commented Aug 5, 2019

Copy link
Copy Markdown
Author

ping @alexcrichton

@miller-time

Copy link
Copy Markdown
Contributor

yeah if rust-lang/cargo#7198 was the inspiration, that's just tests...

@alexcrichton

Copy link
Copy Markdown
Contributor

Sorry, but I am no longer maintaining this crate.

@vorot93

vorot93 commented Aug 8, 2019

Copy link
Copy Markdown
Author

But who is? Perhaps it should be transferred to rustasync since it is used by tide?

@yoshuawuyts

yoshuawuyts commented Aug 8, 2019

Copy link
Copy Markdown
Member

Perhaps it should be transferred to rustasync since it is used by tide?

We'd be happy to help share the maintenance burden if there's help needed!

miller-time added a commit to miller-time/router that referenced this pull request Aug 9, 2019
I guess route-recognizer is no longer maintained? My PR:
http-rs/route-recognizer#25
Comment from Alex Crichton:
http-rs/route-recognizer#26 (comment)
miller-time added a commit to miller-time/router that referenced this pull request Aug 9, 2019
I guess route-recognizer is no longer maintained? My PR:
http-rs/route-recognizer#25
Comment from Alex Crichton:
http-rs/route-recognizer#26 (comment)
miller-time added a commit to miller-time/router that referenced this pull request Aug 9, 2019
closes iron#148

- use patch for route-recognizer
- add `dyn` to trait objects

I guess route-recognizer is no longer maintained? My PR:
http-rs/route-recognizer#25
Comment from Alex Crichton:
http-rs/route-recognizer#26 (comment)
@Nemo157

Nemo157 commented Aug 12, 2019

Copy link
Copy Markdown
Contributor

@vorot93 thanks for this, I've merged #25 instead and published 0.1.13 now which should work on current nightlies.

@Nemo157 Nemo157 closed this Aug 12, 2019
miller-time added a commit to miller-time/router that referenced this pull request Aug 13, 2019
closes iron#148

- use patch for route-recognizer
- add `dyn` to trait objects

I guess route-recognizer is no longer maintained? My PR:
http-rs/route-recognizer#25
Comment from Alex Crichton:
http-rs/route-recognizer#26 (comment)
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.

5 participants