On Wednesday 17 March 2010, Julien Cristau wrote: > On Wed, Mar 17, 2010 at 12:50:04 +0100, Cyril Brulebois wrote: > > --- /dev/null > > +++ b/src/lib/debian-installer.d/S62Xorg > > @@ -0,0 +1,2 @@ > > +Xorg -retro -noreset -nolisten tcp & > > +export DISPLAY=:0 > > This is racy, we probably should do something better here.
Hmmm. And it's also important to distinguish between debian-installer.d and debian-installer-startup.d correctly. debian-installer-startup.d: executed once when D-I boots debian-installer.d: executed after d-i-startup.d and respawned if the debconf frontend should crash So IIUC having it here means we run the risk of starting multiple servers. It seems to me that this should maybe go in debian-installer-startup.d, but that may require more changes in order to determine the frontend to be used early enough. Another solution could be to check if the server is already running and only start it if not. That could provide a restart of the X server if *that* crashes. Cheers, FJP -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org