Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

dbr
Copy link
Contributor

@dbr dbr commented Jan 5, 2022

Required for "--features freetype" to work when using via crates.io

Closes #594
Closes #589

Required for "--features freetype" to work when using via crates.io

Closes imgui-rs#594
Closes imgui-rs#589
@zmoote
Copy link

zmoote commented Jan 27, 2022

Hi guys,

I'm trying to investigate the cause behind the failing linter from
22 days ago (linter 1.54)
https://github.com/imgui-rs/imgui-rs/runs/4710306933?check_suite_focus=true

It seems that the linter is throwing errors with the public enum
template types(?) within winit-0.26.0, ie:

EventLoop<T: 'static>
EventLoopWindowTarget<T: 'static>
EventLoopProxy<T: 'static>
EventLoopWindowTarget

Which are located in
/home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/winit-0.26.0/src/platform_impl/linux/mod.rs
and
/home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/winit-0.26.0/src/event_loop.rs

It seems the error is due to unused parameters.

Is this something that could be solved on imgui-rs's end,
or something I should bring up with the winit team?

Thank you!

ZM

@dbr
Copy link
Contributor Author

dbr commented Jan 27, 2022

Thanks for looking into that! I only briefly glanced at the lint failures, didn't realise it was coming from winit, interesting..

I wonder if the "error: Please select a feature to build for unix: x11, wayland" error is the main one to worry about, and the subsequent unsed-parameters errors are side-effects of that as the mentioned parameter appear to be used, e.g:

https://github.com/rust-windowing/winit/blob/v0.26.0/src/event_loop.rs#L32-L33

@dbr
Copy link
Contributor Author

dbr commented Jan 27, 2022

Hmmm, actually, the solution might be even simpler.. The failing clippy command is

cargo clippy --manifest-path imgui-winit-support/Cargo.toml --all-features --all-targets

from here:

- run: cargo clippy --workspace --all-targets
# supported winit versions
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --all-features --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-19 --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-20 --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-22 --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-23/default --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-24/default --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-25/default --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-26/default --all-targets

..which just isn't present in current main branch

# supported winit versions (with otherwise default features)
- run: cargo doc
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-19 --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-20 --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-22 --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-23/default --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-24/default --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-25/default --all-targets
- run: cargo clippy --manifest-path imgui-winit-support/Cargo.toml --no-default-features --features winit-26/default --all-targets

So we should probably just remove (as done here)

@sanbox-irl sanbox-irl merged commit e62ffea into imgui-rs:v0.8.2-patch Feb 20, 2022
@dbr dbr deleted the freetypecrate branch January 11, 2023 23:13
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.

imgui-rs 0.8.2 docs failing to build 0.8.1 docs build failing
3 participants