NixOS / nixpkgs

Nix Packages collection & NixOS
MIT License
17.86k stars 13.93k forks source link

Changing default terminal in KDE does not work with console app file associations in Dolphin #349721

Open donottellmetonottellyou opened 2 hours ago

donottellmetonottellyou commented 2 hours ago

Describe the bug

When you change the terminal in the Default Apps section in Plasma System Settings, it doesn't appear to change the terminal correctly for terminal apps like helix. If konsole is not installed, Dolphin will produce an error.

Unfortunately, I can't reproduce the error to show a screenshot, because the fix was to set mimetype defaults and I don't know how to undo what I did. However, an install of KDE with the default terminal unchanged should be sufficient to reproduce this on other computers.

Steps To Reproduce

  1. Install KDE using services.desktopManager.plasma6.enable
  2. Add helix and kitty to system packages or home manager
  3. Set them as the default text editor and terminal emulator (respectively) using the KDE settings manager
  4. Remove Konsole like so:
    environment.plasma6.excludePackages = with pkgs.kdePackages; [
    konsole
    ];
  5. Try to open a text file in Dolphin

Expected behavior

I believe Dolphin should open the text file without errors in my chosen terminal and terminal editor, and it does with my fix below in additional context.

Screenshots

image

Additional context

I manually fixed this by running sudo xdg-mime default kitty.desktop application/x-desktop inside the folder where kitty.desktop resides, in my case since I am using home-manager ~/.local/share/applications. I am not sure if this is an upstream issue or not.

Notify maintainers

@K900 @SuperSandro2000 @omentic @stuebinm @Stzx @peterhoeg

Metadata

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.56, NixOS, 24.05 (Uakari), 24.05.5744.dc2e0028d274`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.8`
 - channels(root): `"home-manager-24.05.tar.gz, nixos-24.05, nixos-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

K900 commented 2 hours ago

I think this one might be fixed in 6.2.