Skip to content
This repository was archived by the owner on Jun 8, 2021. It is now read-only.

Finish feature "futures-nightly" removal #186

Merged

Conversation

EPashkin
Copy link
Member

@GuillaumeGomez PR in high-probable case that gtk-rs/glib#373 fails again

cc @sdroege

@GuillaumeGomez
Copy link
Member

Oh crap, we forgot to remove it...

Copy link
Member

@sdroege sdroege left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't this still fail because the binary depends on this feature?

@EPashkin
Copy link
Member Author

Strange, but gtk-rs/glib#373 builds fine without this.
Build failure is just network error.
IMHO better merge this anyway, but seems it also safe to just close this PR.
@sdroege, @GuillaumeGomez what you think?

@sdroege
Copy link
Member

sdroege commented Aug 30, 2018

I wanted to keep as much as possible in place so it can easily be added back later, but I don't really care much. Whatever seems better to you :)

@GuillaumeGomez
Copy link
Member

If it works, it works so sounds good to me.

@EPashkin
Copy link
Member Author

EPashkin commented Sep 1, 2018

Ok, I close this, as it don't break build, gio_futures_await still needed "futures-nightly".

@EPashkin EPashkin closed this Sep 1, 2018
@EPashkin EPashkin reopened this Sep 3, 2018
@EPashkin EPashkin force-pushed the finish_futures-nightly_removal branch from d01f9bd to a6e9383 Compare September 3, 2018 03:19
@EPashkin EPashkin mentioned this pull request Sep 3, 2018
@GuillaumeGomez
Copy link
Member

I merge then.

@GuillaumeGomez GuillaumeGomez merged commit a55a01e into gtk-rs:pending Sep 3, 2018
@EPashkin EPashkin deleted the finish_futures-nightly_removal branch September 3, 2018 08:03
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