On Tue, Jun 21, 2016 at 10:41 PM, Eduardo Lima (Etrunko)
<etru...@redhat.com> wrote:
> On 06/21/2016 05:28 PM, Fabiano Fidêncio wrote:
>> Since commit 1f6f1a48 the resource path for icons has been broken.
>> The reason is that when moving the .ui files to $(srcdir)/resources/ui
>> the define used for the resources was changed to reflect the new
>> directory. However, this change wasn't needed by the icons and ended
>> up with virt-viewer not displaying a few icons.
>>
>> Let's fix it by setting back the define to the previous one and then
>> tweak the virt_viewer_util_load_ui() to add "ui" to the resource path,
>> for loading the ui files.
>> ---
>> src/virt-viewer-util.c | 3 ++-
>> src/virt-viewer-util.h | 2 +-
>> 2 files changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/virt-viewer-util.c b/src/virt-viewer-util.c
>> index 9e52b87..7c95583 100644
>> --- a/src/virt-viewer-util.c
>> +++ b/src/virt-viewer-util.c
>> @@ -50,8 +50,9 @@ virt_viewer_error_quark(void)
>> GtkBuilder *virt_viewer_util_load_ui(const char *name)
>> {
>> GtkBuilder *builder;
>> - gchar *resource = g_strdup_printf("%s/%s",
>> + gchar *resource = g_strdup_printf("%s/%s/%s",
>
> Minor:
>
> "%s/ui/%s" ???
>
> Acked-by: Eduardo Lima (Etrunko) <etru...@redhat.com>
Pushed with your suggestion.
_______________________________________________
virt-tools-list mailing list
virt-tools-list@redhat.com
https://www.redhat.com/mailman/listinfo/virt-tools-list
