-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
new package: niri 0.1.10.1 #52794
base: master
Are you sure you want to change the base?
new package: niri 0.1.10.1 #52794
Conversation
Hey, merged your PR in my personal Void-Packages fork. Build the package and successfully installed it. So far, everything works just fine. But you might add some post-install hooks:
Beside these few additions great work! |
for the portals and desktop, i sadly could not get building with those being moved. i could add a vmkdir to make /usr/share/wayland-sessions and others but i don't know if that's allowed. for the default config, that shouldn't be needed, it automatically fetches config.kdl and adds it to .config. edit: i have added some vmkdir to make the usr directories and i'm gonna try and build it again. |
Looks very good. Exactly the settings I also applied. Works flawlessly so far. Hopefully it will get merged soon, so it can be regularly used through |
you didn't push any of the fixes |
huh, don't know what happened, i'll try pushing them again later |
Just built the Hope this PR gets merged soon since all checks have passed and its reviewed. If I can help test out anything which is holding it back from getting merged, just tell me. Great work! |
Lol, there was just a new release today for 0.1.10. MIght as well update it, since this hasn't gotten merged yet anyways |
There is a bugfix for the current release: https://github.com/YaLTeR/niri/releases/tag/v0.1.10.1 Maybe worth adding to the PR |
Testing the changes
New package
Local build testing
contains templates for the niri window manager. i added extra rustflags for musl as it will not compile without those (cannot check if the actual musl checking part works as of now, but i've tested and it only compiles by adding those flags)
closes #48456