Comment # 40 on bug 1030873 from Josef Reidinger

(In reply to Lukas Ocilka from comment #38)
> In my quick test:
> 
> Pattern selected for installation:
> patterns-openSUSE-lxde_laptop-20150918-26.5.x86_64.rpm
> 
> Logs say:
> clients/desktop_finish.rb:128 Package xorg-x11-server selected or installed,
> desktop min_x matches
> clients/desktop_finish.rb:136 Setting logon manager  - package selected
> clients/desktop_finish.rb:142 Setting window manager icewm - package selected
> clients/desktop_finish.rb:166 Default desktop: icewm
> clients/desktop_finish.rb:167 Default logon manager:
> clients/desktop_finish.rb:199 Logon manager shutdown:
> clients/desktop_finish.rb:207 sysconfig/displaymanager/DISPLAYMANAGER=
> 
> And yes, for some reason, icewm-1.3.12-4.4.x86_64.rpm has been actually
> installed
> 
> The answer to WTF is here:
> https://github.com/yast/skelcd-control-openSUSE/blob/openSUSE-42_3/control/
> control.openSUSE.xml#L243-L253
> 
> And here:
> https://github.com/yast/yast-installation/blob/SLE-12-SP3/src/lib/
> installation/clients/desktop_finish.rb#L122-L164
> 
> Installer tries to match the "selected desktop" using "installed packages",
> there is NO lxde definition in <supported_desktops/> so no wonder it's
> not found.
> 
> Guys, as you know what to put there, would you mind creating a PR
> for the control file change?

this behavior is in fact wrong. If custom is selected, then no desktop should
be selected and should be kept empty, so it should be autoselected.