Skip to content
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

Discussion #28

Open
7 of 11 tasks
RAVENz46 opened this issue Mar 16, 2024 · 7 comments
Open
7 of 11 tasks

Discussion #28

RAVENz46 opened this issue Mar 16, 2024 · 7 comments

Comments

@RAVENz46
Copy link
Contributor

RAVENz46 commented Mar 16, 2024

Thanks for your work!
I tried this module and I think it need some modifications but I'm still newbie and other people also may want some modification, so I create this.

Anyway I think it is better to

  • add services.gvfs.enable = true; (and remove security.polkit.enable = true; cause it is included?) 1914ef3
  • add services.xserver.updateDbusEnvironment = true;

and we have to discuss about

  • use pipewire default instead of pulse audio or not 413848e
  • add services.power-profiles-daemon.enable = mkDefault true; (it will available in the future) or not 0aa2a04
  • add programs.dconf.enable = true; or not 803fb65
  • add services.colord.enable = true; or not
  • add networking.networkmanager.enable = mkDefault true; or not 1914ef3
  • add environment.pathsToLink = [ "/share" ]; or not (described here)
  • add programs.system-config-printer.enable = (mkIf config.services.printing.enable (mkDefault true)); or not
  • add services.fwupd.enable = true; or not
  • rename option services.xserver.desktopManager.cosmic to services.desktopManager.cosmic like plasma6 module did or not 9332bcd
@lilyinstarlight
Copy link
Owner

I'm gonna number these to make it easy:

  1. Does cosmic-files do GVFS? I do agree it should probably be enabled, though (I'm gonna keep polkit though to be explicit rather than implicit)
  2. Does that affect anything for non-X servers? I thought about adding it, but I couldn't tell that it did anything when not using Xorg

Discuss:

  1. I would not be opposed, but I would probably follow what Pop does there (and I haven't looked it up)
  2. We probably should enable PPD yeah if it's gonna use it (otherwise was considering sys76-power)
  3. Does COSMIC use dconf anywhere? It may be useful for GNOME-y user apps though
  4. That's probably not a bad idea. I'll research what colord does more
  5. Yes, we should probably do NM by default (I was kinda waiting for it to be implemented in cosmic-settings/applets)
  6. I'll need to research that one more
  7. We probably should if it does something for COSMIC
  8. I planned to add that when cosmic-settings get fwupd support or see if it gets implemented via something else
  9. The plasma6 module rename was incomplete (e.g. references weren't updated and it's not been updated for displayManager options yet either) and tbh we should move way more options about that instead of doing it DE-by-DE

Thank you for bringing these up! :)

@RAVENz46
Copy link
Contributor Author

RAVENz46 commented Mar 17, 2024

Does that affect anything for non-X servers? I thought about adding it, but I couldn't tell that it did anything when not using Xorg

cosmic requires dbus as dependencies so I think it needs for xwayland probably...? Sorry I'm not sure so I have to figure it out.

We probably should enable PPD yeah if it's gonna use it (otherwise was considering sys76-power)

I see. It looks like it is better to wait until they implement this feature.

Does COSMIC use dconf anywhere? It may be useful for GNOME-y user apps though

Probably not. But even plasma module enables it, so it is not too bad I think.

Anyway, I'll wait patiently cause it's too early to decide some stuffs.
Thanks!

@lilyinstarlight
Copy link
Owner

I've gone ahead and added dconf, renamed the cosmic options to be outside of xserver, and double checked that updateDbusEnvironment will do nothing for COSMIC (it only affects environments that use services.xserver.displayManager.session (which generate start scripts and .desktop files for environments that have none) and only for Xorg sessions (not wayland sessions, even with Xwayland))

@KaiserCalm
Copy link

In the latest blogpost, it is mentioned that support for gvfs was added and that now it is used for usb drive and network drive mounts.

@lilyinstarlight
Copy link
Owner

lilyinstarlight commented May 9, 2024

I've enabled GVFS and NetworkManager in the module by default now

@malikwirin
Copy link

I am currently experiencing the following issue:

error: The option `services.power-profiles-daemon.enable' has conflicting definition values:
       - In `/nix/store/xlmgxngv4xiipb5dvmcq7phmyv6jc64x-source/nixos/cosmic/module.nix': false
       - In `/nix/store/7840m01bg4qkhh3kq6mlrsi3s352qkc1-source/nixos/modules/services/x11/desktop-managers/gnome.nix': true
       Use `lib.mkForce value` or `lib.mkDefault value` to change the priority on any of these definitions.

Isn`t it possible to have gnome and cosmic be enabled at the same time?

@malikwirin
Copy link

I am currently experiencing the following issue:

error: The option `services.power-profiles-daemon.enable' has conflicting definition values:
       - In `/nix/store/xlmgxngv4xiipb5dvmcq7phmyv6jc64x-source/nixos/cosmic/module.nix': false
       - In `/nix/store/7840m01bg4qkhh3kq6mlrsi3s352qkc1-source/nixos/modules/services/x11/desktop-managers/gnome.nix': true
       Use `lib.mkForce value` or `lib.mkDefault value` to change the priority on any of these definitions.

Isn`t it possible to have gnome and cosmic be enabled at the same time?

Not having the following line fixed the issue:

hardware.system76.enableAll = true;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants