Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions archinstall/default_profiles/desktops/plasma.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,23 +29,23 @@ def package_details(self) -> str:
match self:
case PlasmaFlavor.Meta:
ty = tr('Package')
desc = tr('Curated selection of KDE Plasma packages')
desc = tr('A meta package that depends on a curated set of packages. Installing it pulls in the full desktop \nenvironment, including extras. Recommended for most users who want a complete desktop.')
info = available_package(self.value)

if info is not None:
details = tr('Dependencies') + '\n'
details += '\n'.join(f'- {entry}' for entry in info.get_depends_on)
case PlasmaFlavor.Plasma:
ty = tr('Package group')
desc = tr('Extensive KDE Plasma installation')
desc = tr('A package group containing all Plasma packages. Installing the group pulls in all packages together. \nUnlike a meta package, individual packages can be removed afterward without affecting the group.')
group = package_group_info(self.value)

if group is not None:
details = tr('Packages in group') + '\n'
details += '\n'.join(f'- {entry}' for entry in group.packages)
case PlasmaFlavor.Desktop:
ty = tr('Package group')
desc = tr('Minimal KDE Plasma installation')
desc = tr('The core KDE Plasma desktop package. Provides only the essential components needed to run a \nfunctional Plasma session. Suitable for users who want a minimal setup with fewer packages.')
info = available_package(self.value)

if info is not None:
Expand All @@ -57,9 +57,9 @@ def package_details(self) -> str:
def packages(self) -> list[str]:
match self:
case PlasmaFlavor.Meta:
return ['plasma-meta']
return ['plasma-meta', 'konsole', 'dolphin', 'kate', 'ark']
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reason for adding this granular setting was to avoid any additional packagtes. Why do we need these?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using the TTY after installing a desktop environment just to install a terminal doesn’t provide a great first-user experience, in my opinion. I had assumed that was the purpose of the minimal install, rather than the plasma-meta/full plasma package. I didn’t realize there was a deliberate decision not to include applications like Dolphin or Konsole. In that case, disregard this PR.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tbh, there's not any clear consensus what to do here but we've gotten some pretty taxing feedback from both packagers of KDE and users whenever we try to fiddle with the KDE profile.

So until there's consensus from both sides, we should consider the KDE profile and on-off switch.
If you want a working thing, listen to the KDE packagers and install everything, and if you want choices, use the non working options and build a .json conf with additional packages and re-use that.

Apologies that we can't do anything with the KDE profile, but that's where we are at trying our best in the middle.

case PlasmaFlavor.Plasma:
return ['plasma']
return ['plasma', 'konsole', 'dolphin', 'kate', 'ark']
case PlasmaFlavor.Desktop:
return ['plasma-desktop']

Expand Down