Comment 8 for bug 546445

Revision history for this message
Michael Vogt (mvo) wrote :

Hey Julien, thanks for your branch! I think it would be good if you could create a merge proposal into lp:ubiquity for it. AFAICS there is none created yet?

I looked over the diff and while I'm not a ubiquity expert I think it looks fine. My only question would be if it needs to run a full lxde session (the other DE environments seem to not run the full session AFAICS) and if the line:

            if os.path.exists('/usr/lib/ubiquity/panel') and "xfwm4" not in wm_cmd:

needs a additional 'not not "openbox-lubuntu" in wm_cmd'.