On 09/05/2016 09:15 AM, Christophe Fergeau wrote: > On Tue, Aug 02, 2016 at 06:17:31PM +0200, Christophe Fergeau wrote: >>> Eduardo Lima (Etrunko) (10): >>> ovirt-foreign-menu: Remove timer used to refresh iso list >>> ovirt-foreign-menu: Add accessors for current iso and iso list >>> ovirt-foreign-menu: Remove GtkMenu related functions >>> ovirt-foreign-menu: Notify of new files even if nothing changed >>> UI: Make 'Change CD' menu item a submenu under 'File' toplevel menu >>> Introduce ISO List dialog >>> Run iso-dialog when 'Change CD' menu is activated >>> remote-viewer: Make ovirt-foreign-menu a property >>> iso-dialog: Implement functionality provided by oVirt foreign menu >>> iso-dialog: Use header bar for buttons >> >> I'm not sure how to approach this series as I don't think >> 'ovirt-foreign-menu: Add accessors for current iso and iso list' > > See the attached patch for some variation around this which avoids > relying on notify::file and add an async method to do it. > > Christophe >
Hey Christophe, Thanks for the patch, it seems really straightforward. I will include it in the series, rebase and post again. -- Eduardo de Barros Lima (Etrunko) Software Engineer - RedHat etru...@redhat.com
signature.asc
Description: OpenPGP digital signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@redhat.com https://www.redhat.com/mailman/listinfo/virt-tools-list