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

Conversation

@alatiera
Copy link

Also remove the futures feature and move to 0.3 from 0.3.preview

@sdroege
Copy link
Member

sdroege commented Nov 18, 2019

error: Package `gtk-rs-examples4 v0.0.1 (/home/travis/build/gtk-rs/examples4)` does not have feature `futures`. It has a required dependency with that name, but only optional dependencies can be used as features.

@sdroege
Copy link
Member

sdroege commented Nov 18, 2019

You also have to update the minimum version to 1.39 from 1.36

@sdroege
Copy link
Member

sdroege commented Nov 18, 2019

@GuillaumeGomez good to go :)

Also remove the futures feature and move to 0.3 from 0.3.preview
extern crate gtk;

use gio::prelude::ApplicationExtManual;
use gio::ApplicationExt;
Copy link
Member

Choose a reason for hiding this comment

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

nit: Should we also use gio::prelude::ApplicationExt; here, for consistency?

Copy link
Member

Choose a reason for hiding this comment

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

No, we should use gio::prelude::* here for both :)

@sfanxiang
Copy link
Member

I'm slightly worried by the unused import warnings in CI. That generally means some library is reexporting symbols when it shouldn't. I'll double check on that.

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