Miouyouyou / armbian-desktop

A specific branch for dealing with default desktop environments
GNU General Public License v2.0
4 stars 1 forks source link

Support matrix #13

Open igorpecovnik opened 3 years ago

igorpecovnik commented 3 years ago
Miouyouyou commented 3 years ago

Anyway, yeah, an error should be thrown if unavailable desktop / appgroups are chosen, to avoid any confusion.

igorpecovnik commented 3 years ago

Some desktop doesn't have support for armhf - at least binary packages doesn't exits (dde for example). Perhaps reverse logic? Not sure atm, just we need to solve this somehow.

Miouyouyou commented 3 years ago

Yeah. I guess we'll need some arch specific folder. I need to think about it a little more.

igorpecovnik commented 3 years ago

also in this context we need DE support status like we have for userland. We will officially only support A and B desktop, others will be in "not supported" status. On the desktop names, perhaps also shorting to DE and showing status in ()

xfce | Xfce desktop enfironment -> xfce | Xfce DE (supported) cinanamon | Cinanamon desktop enfironment -> cinanamon | Cinanamon DE (CSC)

Miouyouyou commented 3 years ago

Alright, I've added support for package.remove. I'll try to squeeze off arch limitations today.

I'm thinking about which way would be better. My best guess would be a file named "only_for" inside desktop environments and appgroups.

If, and only if, the file is present, its content is grep'ed for the current targeted architecture. If the architecture is not found (grep returned nothing), the option is not provided in the menus. Then the same logic can be applied to generate errors, if an invalid option is selected.

Miouyouyou commented 3 years ago

Ok, preliminary support has been added (much later though) : https://github.com/Miouyouyou/armbian-desktop/commit/e9c7a34e6502484af374e4f58ff1baa10cfa27ff

I'll see how to generalize this for appgroups, and then anything just in case "cli appgroups" were to be added afterwards.