[virt-tools-list] [virt-viewer] Use GResource for loading ui files
Fabiano Fidêncio
fabiano at fidencio.org
Mon Feb 29 18:01:16 UTC 2016
On Mon, Feb 29, 2016 at 6:27 PM, Eduardo Lima (Etrunko)
<etrunko at redhat.com> wrote:
> On 02/26/2016 07:38 PM, Fabiano Fidêncio wrote:
>> Let's take advantage of GResource for loading ui files in a better and
>> cleaner way than virt_viewer_util_load_ui() was doing.
>> It also brings the benefit, at least for developers, of being able to
>> test ui changes without having to "make install" virt-viewer.
>>
>
> General question, does this mean that those XML files don't need to be
> installed anymore?
Yes, they end up compiled right into the executable..
> If so, you might also remove them from Makefile.
We still have to distribute them.
>
> Otherwise patch looks good. I don't kwnow much about GResources, so it
> might be interesting to listen from more experienced people.
Thanks for the review.
More information about the virt-tools-list
mailing list