Welcome to tomaka/glutin, the official gitter channel for Glutin and Winit! - Github: https://github.com/rust-windowing/glutin, https://github.com/rust-windowing/winit - Crates.io: https://crates.io/crates/glutin, https://crates.io/crates/winit - Docs: https://docs.rs/glutin/0.21.0/glutin, https://docs.rs/winit/0.20.0/winit/
UndeadLeech on Freenode It's usually a good idea to double-check. Harder to slip things by two people too, the second person might test things a little differently covering things more thoroughly.
UndeadLeech on Freenode Of course I'm aware of the issue with lack of testers.
UndeadLeech on Freenode Generally I'd advice to ping Wayland testers and give them at least 24h. And if people regularly don't show up, remove them from the list of testers.
UndeadLeech on Freenode Not out of malice, just because it doesn't benefit anyone to have a list of testers where nobody is interested in testing.
kchibisov on Freenode Yeah, but the thing is that I don't have anything that I can impl in Wayland backend right now.
kchibisov on Freenode So the only things that can show up is purely source level bugs or something like that.
UndeadLeech on Freenode And bugs should be independently verified by two people.
kchibisov on Freenode Kai Mast: the fix for your bug got merged.
kchibisov on Freenode UndeadLeech: I guess,
UndeadLeech on Freenode Anything that changes the behavior of winit, should be tested by at least two people.
UndeadLeech on Freenode And if the capacity doesn't exist, it should be made transparent in the testers list.
UndeadLeech on Freenode I mean if two people tested the original code, it might not have existed in the first place.
UndeadLeech on Freenode Fixing bugs changes the behavior of winit.
kchibisov on Freenode UndeadLeech: the original code was tested by more than just me.
UndeadLeech on Freenode The only changes that do not impact the behavior of winit are style changes.
UndeadLeech on Freenode kchibisov: It's not a guarantee of course.
kchibisov on Freenode Yeah, this is one is a bit wierd, since you should min/max size in window builder.
UndeadLeech on Freenode I mean I'm not much of a winit maintainer, so I don't have much to say. But I would never allow any Alacritty PR that changes anything in the code without being tested by at least two people.
kchibisov on Freenode And this thing is pretty rare to happen, since there's
kchibisov on Freenode since there's no real value in setting those things unless you're writing some 'special' application.
kchibisov on Freenode Like you want to start your window in floating mode.
kchibisov on Freenode UndeadLeech: are you counting the original author or are you counting only testers?
kchibisov on Freenode Ah, how convinient.
UndeadLeech on Freenode The same way we do it in Alacritty.
UndeadLeech on Freenode Yeah and according to the list of testers in winit there should be more than two people capable of testing Wayland patches.
kchibisov on Freenode The thing with Wayland, that proper PRs testing is very hard, since you should check like 4 compositors.
kchibisov on Freenode And verify that something strange you're seeing is compositor bug.
kchibisov on Freenode Most of the time that's true for GNOME.
kchibisov on Freenode But yeah, for something more complex than this one liner, I'd ping folks and wait for day or two.
UndeadLeech on Freenode Perfect is the enemy of good. Merging PRs without any second party verifying them because "testing correctly is difficult" is not exactly constructive.
kchibisov on Freenode I mean if they verify on GNOME it's hard to say anything.
UndeadLeech on Freenode Why?
kchibisov on Freenode Like the amount of GNOME bugs I've seen is rediculuos.
UndeadLeech on Freenode We should be aware of how the application behaves. If something cannot be tested on GNOME then that should be stated ahead of time.
kchibisov on Freenode And it's not that hard to trigger them.
UndeadLeech on Freenode If there is a GNOME bug, we should be aware of it.
kchibisov on Freenode I mean, I report them to GNOME sometimes.
kchibisov on Freenode Like HiDPI on GNOME is completely broken last time I've tried.
UndeadLeech on Freenode So?
kchibisov on Freenode nothing, I guess.
Kai Mast Just tested the bugfix and it resolves the problem for me.
kchibisov on Freenode Kai Mast: yeah, just set min_inner_size if you really want it on a window and not in a builder.
UndeadLeech on Freenode If something applies only to certain compositors, that should be made clear from the start. If it should work on all compositors and it turns out during the testing process that it doesn't, that's exactly what testing is there for.
kchibisov on Freenode I mean the thing with GNOME that it can arbitrary downscale you.