[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#574288: Please switch from DirectFB to X11



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



Reply to: