-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
nixos/xfce4-14: init #66859
nixos/xfce4-14: init #66859
Conversation
0dbd199
to
63a5220
Compare
Dropped the assertion since there's no way to rule that out completely, and synced this with module changes I've made on master. |
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.
hmm, I forgot to submit this.
environment.pathsToLink = [ | ||
"/share/xfce4" | ||
"/lib/xfce4" | ||
"/share/gtksourceview-3.0" |
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.
We should probably move this into a separate module.
Not sure if it would be worth it, I plan if possible to deprecate the previous version next release. |
obsoleted by xfce4-pulseaudio-plugin / dead package
This is pretty much identical to the xfce test we currently have.
9fc58aa
to
fd7d31b
Compare
Added a copy cat test of the one we already have so this can be in the tested jobset. |
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.
Tested on a VM. Looks good to me.
The only last case I think we should be concerned about is the closure size vs the older version of xfce. |
Are we also sure that with nixpkgs-update it won't send pr's updating things to release candidates/unstable releases? (i.e properly version ignored). |
For GNOME, I regularly check my packages and flag anything that is unstable. Usually, I catch them pretty soon after they appear in Fedora Rawhide, preventing nixpkgs-update from picking them up. After two release cycles, almost every package related to GNOME platform has an ignore pattern on Repology, so only few infrequently released packages remain to be flagged in the future. I guess something similar will need to be done for Xfce. |
Motivation for this change
Based off and completes #37594.
Ideally we would have one xfce module and package set, but things aren't as simple as that.
We have 7af5c81 to prevent both of the modules from being used simultaneously, as I'm assuming this could be dangerous.
The following is a quick summary of the changes between #37594
xfce4-pulseaudio-plugin
for pulseaudio support in panel (the other one was extra)services.xserver.displayManager.sessionCommands
overextraSessionCommands
Lastly we dropped xfce4-mixer because it's abandoned.
Further work/Questions
I haven't resynced the thunar plugins patch, not sure if there's compatible plugins at this version?
And should networkmanager be default enabled?
Things done
Checked in a VM that with this configuration, we could get past the login and at least show the desktop. No glaring issues are apparent to me.
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @