Skip to content

Add Cutefish profile - #558

Merged
Torxed merged 5 commits into
archlinux:masterfrom
JakobDev:cutefish
Jun 9, 2021
Merged

Add Cutefish profile #558
Torxed merged 5 commits into
archlinux:masterfrom
JakobDev:cutefish

Conversation

@JakobDev

@JakobDev JakobDev commented Jun 7, 2021

Copy link
Copy Markdown
Contributor

Installs the Cutefish DE

@JakobDev
JakobDev requested a review from Torxed as a code owner June 7, 2021 12:44
@dylanmtaylor

Copy link
Copy Markdown
Contributor

-1 because it's a niche DE and we already have plenty of DE profiles. Custom Xorg exists for a reason. I also can't find the packages by doing a Google search for Cutefish so that makes it a very strong -1 from me.

@dylanmtaylor

Copy link
Copy Markdown
Contributor

It took me a very long time to even find info on this DE. https://cutefishos.com/
I still feel this shouldn't go in. @Torxed what do you think?

@dylanmtaylor

Copy link
Copy Markdown
Contributor

I suppose it'd be okay if we add this file but do not put it in the profiles/desktop.py choice so it can be used for custom scripted installs as a compromise.

@JakobDev

JakobDev commented Jun 8, 2021

Copy link
Copy Markdown
Contributor Author

It's a very new De but it's already supported by Arch with it's own group. I think archinstall, as the official installer of Arch, should support every DE that is supported by Arch. The profile is just a package list and a command to enable sddm, so it's nothing that requires a lot of work to maintain. Manjaro has also already a Community Edition with Cutefish.

@dylanmtaylor

dylanmtaylor commented Jun 8, 2021

Copy link
Copy Markdown
Contributor

I think archinstall, as the official installer of Arch, should support every DE that is supported by Arch

This isn't really feasible as we have limited resources. Also, things that are more flexible like openbox which requires decent amounts of configuration to get set up in a way that is usable by users are best served by a fully custom installation based on the Xorg profiles. Having lots of options makes regression testing more difficult.

@dylanmtaylor

Copy link
Copy Markdown
Contributor

It's also apparently a fork of Cyber with very few changes, from what I'm seeing. https://getcyberos.org/#/

@ryzendew

ryzendew commented Jun 8, 2021

Copy link
Copy Markdown

Add both cyber and cutefish it be good

@dylanmtaylor

Copy link
Copy Markdown
Contributor

Does this require SDDM or will lightdm suffice? Historically, the way we have written the profiles is that we do a lightdm installation unless the developers of the DE recommend something else. Specifically, sway doesn't officially support a greeter so we don't use one, KDE and LXQt both only support SDM officially, and GNOME wants GDM. I'm going to test an installation of this DE from the generated ISO now to validate it at least works.

@dylanmtaylor

dylanmtaylor commented Jun 9, 2021

Copy link
Copy Markdown
Contributor

I can validate that this does boot
VirtualBox_Arch_08_06_2021_21_41_42
.

Comment thread profiles/cutefish.py
"cutefish",
"noto-fonts",
"konsole",
"sddm"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's switch sddm to lightdm unless there's a good reason not to.

Comment thread profiles/cutefish.py
# Install the Cutefish packages
archinstall.storage["installation_session"].add_additional_packages(__packages__)

archinstall.storage["installation_session"].enable_service("sddm")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Switch here too

@ryzendew

ryzendew commented Jun 9, 2021

Copy link
Copy Markdown

yes it works best with sddm

@Torxed

Torxed commented Jun 9, 2021

Copy link
Copy Markdown
Member

It's a very new De but it's already supported by Arch with it's own group. I think archinstall, as the official installer of Arch, should support every DE that is supported by Arch. The profile is just a package list and a command to enable sddm, so it's nothing that requires a lot of work to maintain. Manjaro has also already a Community Edition with Cutefish.

I agree with @dylanmtaylor on this. Just because Arch Linux supports things, doesn't automatically mean we can.
This DE is in a very early stage and it adds additional testing before a release of archinstall can happen (since we aim to test every combination and feature we have).

DE's is especially tricky because they have a lot of hidden dependencies like:

  • greeters
  • graphics drivers
  • Locale's
  • hardware requirements

All of which has to be tested for possible combinations and on a wire range of hardware.
I don't mind merging in this PR allowing people to script their own installations and using this profile, but for test and maintenance reasons I do not wish to keep this listed by desktop.py as a general profile.
I don't mind circling back once this DE has reached a more mature stage in terms of development and testing or the user base gets so big it outshadow's some of the more common ones.

Remove this line for now and I can merge it:

installation.install_profile('cutefish')

Will still work even after that line is removed.

@dylanmtaylor

Copy link
Copy Markdown
Contributor

yes it works best with sddm

Is this documented somewhere?

@dylanmtaylor

Copy link
Copy Markdown
Contributor

It's a very new De but it's already supported by Arch with it's own group. I think archinstall, as the official installer of Arch, should support every DE that is supported by Arch. The profile is just a package list and a command to enable sddm, so it's nothing that requires a lot of work to maintain. Manjaro has also already a Community Edition with Cutefish.

I agree with @dylanmtaylor on this. Just because Arch Linux supports things, doesn't automatically mean we can.
This DE is in a very early stage and it adds additional testing before a release of archinstall can happen (since we aim to test every combination and feature we have).

DE's is especially tricky because they have a lot of hidden dependencies like:

  • greeters
  • graphics drivers
  • Locale's
  • hardware requirements

All of which has to be tested for possible combinations and on a wire range of hardware.
I don't mind merging in this PR allowing people to script their own installations and using this profile, but for test and maintenance reasons I do not wish to keep this listed by desktop.py as a general profile.
I don't mind circling back once this DE has reached a more mature stage in terms of development and testing or the user base gets so big it outshadow's some of the more common ones.

Remove this line for now and I can merge it:

installation.install_profile('cutefish')

Will still work even after that line is removed.

@Torxed you can merge this as-is and I can make a commit to remove the line.

@JakobDev

JakobDev commented Jun 9, 2021

Copy link
Copy Markdown
Contributor Author

Cutefish has now been removed from .desktop, but I think it would be nice to use the profile in the guided installer. Maybe a experimental section?

@Torxed
Torxed merged commit 3909333 into archlinux:master Jun 9, 2021
@JakobDev
JakobDev deleted the cutefish branch June 9, 2021 19:32
@dylanmtaylor

Copy link
Copy Markdown
Contributor

@JakobDev You can use it like this:

VirtualBox_Arch_10_06_2021_08_18_03

No "experimental" section needed :)

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

Successfully merging this pull request may close these issues.

4 participants