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'.

-- 
Support for lxdm in ubiquity (autologin, only-ubiquity support)
https://bugs.launchpad.net/bugs/546445
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to