-
-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
Budgie Desktop #216737
Budgie Desktop #216737
Conversation
* instead of relying on the current "--login" implementation. | ||
*/ | ||
- argv[0] = "/usr/sbin/usermod"; | ||
+ argv[0] = "@shadow@/bin/usermod"; |
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.
When making this patch I looked up every reference to /usr
and found this. This isn't in GNOME Control Center's paths.patch
, I'm not sure if that's on purpose?
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.
This is previously patched in 1de94d5 but it's later dropped in fcf2213, not sure the reason. Is adding users in *-control-center
actually works with e.g. users.mutableUsers
enabled (since I think only cc-add-user-dialog
uses it)?
(Ha! Saw @jtojnar in gnome-control-center!653)
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, maybe it was removed accidentally, though it does not look like the surrounding code changed to cause a merge conflict.
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.
@ofborg test budgie VM starts. This is much more vanilla than my last try on budgie-nix flake, I will try to do some comparison 😆 Do you have a preference (on the defaults) here? (since looks like upstream does not like e.g. Adwaita gtk theme and icon theme) Edit: having a default wallpaper will also be nice :-) |
My recommendation for the defaults are Papirus Icon Theme and either Materia GTK Theme (mind you that Materia is not maintained) or Pocillo, with built-in theme enabled (this is the default). Both Fedora Budgie Spin and the respective Fedora budgie-desktop-defaults package, as well as Solus, use Papirus + Materia. Ubuntu Budgie uses Pocillo for the mix between Arc GTK Theme color palette and Materia Design components, with their Pocillo Icon Theme (mix of Moka and Faba). |
|
I guess it should be fine to do the patching here if they are easy to review (e.g. mate-desktop/caja@f46fe89) |
Budgie Desktop View has Budgie Desktop specific functionality, specifically we have right-click options that open up Budgie Control Center and Budgie Desktop Settings. Per its readme:
I would recommend Nemo over Caja, as Caja has the tendency to trigger its desktop icons implementation and their background / wallpaper rendering uses their own schemas, so there is a chance that can get triggered and doesn't make for an amazing experience. Mind you this was also with their implementation "disabled" and it didn't seem to respect it. But yea, Caja will show up at least if you add that patch. |
Welp, I could've just RTFM I guess 😆. I'm going to leave it as is then.
That's marvelous. |
Very happy to see progress on this! Thank you :D |
Is there a way to rebase this PR on top of a more recent Nixpkgs? I'm on a new system and building the test takes a while because it downloads older packages. |
I commonly rebase to nixos-unstable (not master unless merge conflicts, since if you are not lucky you sometimes need to build webkitgtk) I will actually also recommend testing this against the GNOME 44 PR (though if you rebase against this you cannot push the rebased commits here since things will mess up), the most recent finished eval is 7c49b47, mainly for mutter 44 (I think budgie should already support that) and glib 2.76 (for the slice allocator removal). |
Signed-off-by: Federico Damián Schonborn <fdschonborn@gmail.com>
Signed-off-by: Federico Damián Schonborn <fdschonborn@gmail.com>
I cannot set wallpaper in budgie-control-center ( Otherwise with a brief testing this looks okay to me. |
@ofborg test budgie
Hmmm, probably just add the necessary stuff, I find some of the dep weird to appear here, e.g.
(Btw is your testing all good right now? We really want this in 23.05) |
Search Locations was removed in Budgie Control Center in BuddiesOfBudgie/budgie-control-center@434c551 as Budgie Desktop does not have that sort of functionality. tracker is not required for any Budgie components. In the case of libsoup, as you noted it is not required for Budgie Control Center but rather required for gnome-online-accounts. It may have been pulled in for goa-1.0 / goa-backend-1.0 but it should be a dep of those not BCC. BCC does not link against it. |
Signed-off-by: Federico Damián Schonborn <fdschonborn@gmail.com>
Signed-off-by: Federico Damián Schonborn <fdschonborn@gmail.com>
Signed-off-by: Federico Damián Schonborn <fdschonborn@gmail.com>
Signed-off-by: Federico Damián Schonborn <fdschonborn@gmail.com>
Signed-off-by: Federico Damián Schonborn <fdschonborn@gmail.com>
Okay, I removed those dependencies. |
Signed-off-by: Federico Damián Schonborn <fdschonborn@gmail.com>
Signed-off-by: Federico Damián Schonborn <fdschonborn@gmail.com>
Okay let's go. Improvements can still be done in follow-up PRs 🙈 If you wish, you can add a budgie section, config, screenshot to calamares-nixos-extensions (you might probably grep for e.g. |
Description of changes
Adds the Budgie Desktop to NixOS (this includes all of the remaining packages, and a NixOS module with a test).
This patch fixes #190192.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)