-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
sway: 1.2 -> 1.4, wlroots: 0.8.1 -> 0.10.0 #76787
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works on my machine 👍
Hello @primeos : I have tested the PR, and it seems to work properly as of I can confirm that |
@primeos : Should we add |
I think we should add those by default for convenience. I originally added |
Now crashes with:
For both 1.3-rc1 and 1.3-rc2. 1.2 works fine. |
@Synthetica9 @elyhaka @Ma27 huge thanks for testing this PR and providing feedback! ❤️ @Synthetica9 strange that both Sway release candidates do now crash :o Was that due to the second wlroots update from 0.9.0 to 0.9.1? On my setup Sway 1.3-rc2 still works fine. |
I think I may have incorrectly tested 1.3-rc1, running 1.2 when I thought I was running 1.3-rc1. |
Filed an upstream issue: swaywm/sway#4898 |
Still no issue on my side : Could your issue be GPU-related @Synthetica9 ? |
I suppose it could be, but I don't suspect it since I have an Intel IGPU laptop. How would I even diagnose that? |
@Synthetica9 good idea, forgot about that, thanks :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
... And now it fixed itself again. Current working hypothesis: PEBCAK (somewhere)
(Did another rebase to re-trigger CI as the evaluation was broken.)
Cool, glad it works now :) Maybe there where also some other relevant dependency updates in the meantime that fixed it (like the Mesa update in my case). |
The default changed in Sway 1.3 from rxvt_unicode to alacritty. For backward compatibility we'll install both terminal emulators by default.
After some testing/reviews this should be finally ready to be merged 🎉. Regarding cage: 0.1.1 doesn't build with wlroots 0.10.0 and IMO backporting the patches is dangerous (I'm not familiar with the codebase and might miss other important changes, but e.g. FreeBSD seems to do this successfully patch-wlroots-0.9). Therefore I decided to use the master version. An alternative would e.g. be to override wlroots back to the old (compatible) version. Any (strong) opinions regarding this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me *\o/*
Everything's running smoothly on 1.4 :) |
Great, thanks everyone for the testing and valuable feedback :) 🎉 Merged now 🎉 |
I have the same problem as @Synthetica9:
How can I upgrade mesa? (I try to keep everything related to |
@NilsIrl normally/officially you can only update it system-wide (i.e. update the channel and You could upgrade Mesa independently via setting |
I tried setting and from what I can tell, it was fixed on its own for @Synthetica9 so it might have nothing to do with mesa. |
I think mine fixed itself after the merge with master, so it might have
been an mismatched mesa version that fixed itself at that point. (I run
nixos-unstable as my system channel)
|
Well, running everything under |
FYI: I've opened #87979 regarding the packaging of the tools in (Thought I'd also mention this here since we discussed it a bit and it was an optional TODO:)
|
Motivation for this change
Ready for final testing :)
On my setup I've successfully tested
sway
,swayidle
,swaylock
, andcage
.nix-review
:TODOs:
master
is fine and I've requested a new release: New release for compatibility with wlroots 0.9.x? cage-kiosk/cage#107-> Will use master for now.
Will be done late in a separate PR:
contrib/
aren't installed by default but we could e.g. create an additional package likesway-contrib
. (or sway-tools, sway-extras, etc.).Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @