<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On 22 October 2014 14:53, Pekka Paalanen <span dir="ltr"><<a href="mailto:pekka.paalanen@collabora.co.uk" target="_blank">pekka.paalanen@collabora.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+       pid = fork();<br>
+       if (pid < 0) {<br>
+               fprintf(stderr, "fork failed: %m\n");<br>
+               goto out;<br>
+       }<br>
+<br>
+       if (pid)<br>
+               goto out;<br>
+<br>
+       argvpp = argv.data;<br>
+       if (execve(argvpp[0], argvpp, envp.data) < 0) {<br>
+               fprintf(stderr, "execl '%s' failed: %m\n", argvpp[0]);<br>
+               exit(1);<br>
+       }<br></blockquote><div><br></div><div>Hmm. Can we please use weston_client_start here instead of open-coding it?</div><div><br></div><div>And, while you're at it - as this was written for kiosk mode, it spawns a shell script which just restarts the video player in a loop. Can we please add an autostart param to weston_client_run/start (as a new enum with WESTON_CLIENT_RESTART, not bool) and to the config (as a bool) which lifts most of desktop_shell_client_destroy/check_desktop_shell_crash_too_early/respawn_desktop_shell_process?</div><div><br></div><div>Aside from that, and splitting refactor / autorestart code move / autorun feature into separate patches, this looks good to me.</div><div><br></div><div>Cheers,</div><div>Daniel</div></div></div></div>