On Thu, Dec 03, 2015 at 12:17:41PM -0200, Eduardo Lima (Etrunko) wrote: > On 12/02/2015 09:09 AM, Christophe Fergeau wrote: > > Hey, > > > > On Fri, Nov 27, 2015 at 05:24:00PM -0200, Eduardo Lima (Etrunko) wrote: > >> Most of this patch consists in code being shuffled around to fit the > >> expected flow while using the new APIs. I tried my best to make this > >> patch the less intrusive as possible. Main changes are: > >> > >> - VirtViewerApp is now a subclass of GtkApplication > >> > >> Also, some mainloop calls were replaced, as follows: > >> * gtk_main() -> g_application_run() > >> * gtk_quit() -> g_application_quit() > >> > >> - Unified command line option handling: > >> The logic has moved from the main functions and split in three, the > >> common options, and specific ones for each application. With this, the > >> main functions were highly simplified, and now basically responsible > >> for instantiating the App object and running the main loop. > >> > >> - All Window objects must be associated with the Application, and with > >> this, there is no need to emit our own 'window-added' signal, it will > >> be done by GtkApplication by the time gtk_application_add_window() is > >> called. The 'window-removed' signal has also been removed, as it was > >> not being used anyway. > > > > GApplication was added in glib 2.28, but some of the API you are using ( > > g_application_add_option_group ) was added as recently as glib 2.40. > > configure.ac needs to be updated to reflect this, or some alternatives > > need to be considered if the glib 2.40 is too new (el7.0 had a too old > > glib, el7.1 is fine, fedora 21 and newer are fine too). > > Ok, I will update configure.ac too No, we have to avoid anything from 2.40 - the newest glib we can use based on major distro support is going to be 2.38, as I don't think it is reasonable to drop support for current SLES. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
