Stop installing xorg packages for Wayland profiles

[dylanmtaylor]

Wayland profiles no longer pull in xorg-server and xorg-xinit.

Adds an is_wayland property to Profile. Wayland profiles (GNOME, KDE Plasma, Cosmic, hyprland, sway, niri, river, labwc) extend Profile directly instead of XorgProfile. X11 profiles stay on XorgProfile.

[dylanmtaylor]

Ah, I spotted a bug.

	def __init__(
		self,
		name: str = 'Wayland',
		profile_type: ProfileType = ProfileType.Xorg,
	):

Need to fix that.

[dylanmtaylor]

I still need to test this, but it's probably not too far off from correct.

[h8d13]

I believe you will run into one issue is that you will need to hide wayland from menu choice since it can't really be used standalone, should be hidden in /lib/profile/profiles_handler.py if I remember correctly

	def _find_available_profiles(self) -> list[Profile]:
		"""
		Search the profile path for profile definitions
		"""
		profiles_path = Path(__file__).parents[2] / 'default_profiles'
		profiles = []
		for file in profiles_path.glob('**/*.py'):
			# ignore the abstract default_profiles classes
			# and wayland standalone
			if file.name in ('profile.py', 'wayland.py'):
				continue

[dylanmtaylor]

I believe you will run into one issue is that you will need to hide wayland from menu choice since it can't really be used standalone, should be hidden in /lib/profile/profiles_handler.py if I remember correctly

	def _find_available_profiles(self) -> list[Profile]:
		"""
		Search the profile path for profile definitions
		"""
		profiles_path = Path(__file__).parents[2] / 'default_profiles'
		profiles = []
		for file in profiles_path.glob('**/*.py'):
			# ignore the abstract default_profiles classes
			# and wayland standalone
			if file.name in ('profile.py', 'wayland.py'):
				continue

Ah, nice callout. I implemented the code from this comment.

[dylanmtaylor]

We can remove 'xorg-xinit', from awesome since it's in the base class now.

[dylanmtaylor]

Tested working with KDE.

This is good to go in.

[h8d13]

Looks great, and a nice gain in download/install size + unnecessary attack surface

[svartkanin]

@dylanmtaylor instead of using inheritance for this, can we move this to a composition pattern instead as it will make refactoring easier in the future. Also the creating a wayland profile indicates that this is a standalone profile which it isn't.
The profiles should have a property is_wayland or something.

[dylanmtaylor]

@dylanmtaylor instead of using inheritance for this, can we move this to a composition pattern instead as it will make refactoring easier in the future. Also the creating a wayland profile indicates that this is a standalone profile which it isn't.
The profiles should have a property is_wayland or something.

I already have handled it not being reported as a standalone pattern in this change as it currently gets excluded from profile selection but I'll rethink the approach.

[dylanmtaylor]

This block is now necessary for the base Xorg profile (no desktop) to install its own packages:

	@override
	def install(self, install_session: Installer) -> None:
		install_session.add_additional_packages(self.packages)

[dylanmtaylor]

This is tested (again) and still working after the refactor.

[dylanmtaylor]

@svartkanin Is this closer to what you were thinking now?