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

Re: [master 1/2] Restarting anaconda.



Hi,

On 05/20/2010 09:37 PM, Chris Lumens wrote:
+int restart_anaconda() {
+    if (access("/tmp/restart_anaconda", R_OK))
+        return 0;
+    puts("Restarting Anaconda.");
+    return 1;
+}
+

Why puts here instead of fprintf?

It's shorter and has less arguments. Why fprintf?

Have you tried killing all processes in anaconda's process group and
seeing what happens that way?  If that worked, it could be a cleaner way
of accomplishing the same thing as above, plus would handle arbitrarily
deep trees of child processes.


I meant the script to be able to handle arbitrarily deep trees. Or can you see a reason why not?

Eh, anaconda's process group is 1 (init, loader). I can try to call setpgid(0,0) at the top and see what happens. That would simplify the restart script a lot!

Ales


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