https://bugzilla.novell.com/show_bug.cgi?id=637215 https://bugzilla.novell.com/show_bug.cgi?id=637215#c18 --- Comment #18 from Ricardo Cruz <rpmcruz@alunos.dcc.fc.up.pt> 2010-09-08 14:42:48 UTC --- Created an attachment (id=388269) --> (http://bugzilla.novell.com/attachment.cgi?id=388269) Show error window patch (In reply to comment #15)
If you write a patch, I'll review it. We're more or less upstream for gnomesu, now anyway (even though we don't use any real vcs for it, just our package).
Here is a try. (I haven't tried it yet, since my laptop runs on Ubuntu.) a) it calls show_error_window() in case su and pam fails. b) show_error_window() forks and shows the error window asynchronously (just like service->spawn_async2() would). (presumably g_thread_init() and gtk_init() were already issued by the program -- since neither su.c nor gnomesu-auth-dialog.c do any initialization at this point) c) gnomesu_spawn_command_sync() was changed to cope with the fact that, now, errors can be asynchronous stuff too. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.