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

Update examples for the latest regen #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update examples for the latest regen #2

wants to merge 1 commit into from

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 :)

build_travis.sh Outdated Show resolved Hide resolved
Also remove the futures feature and move to 0.3 from 0.3.preview
@@ -7,8 +7,8 @@ extern crate gio;
extern crate glib;
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