Hey, ----- Original Message ----- > From: "Christophe Fergeau" <[email protected]> > To: "Jonathon Jongsma" <[email protected]> > Cc: [email protected] > Sent: Friday, January 17, 2014 7:43:44 AM > Subject: Re: [virt-tools-list] [PATCH virt-viewer] Load ui files first from > installed location > > Hey, > > On Thu, Jan 16, 2014 at 12:36:15PM -0600, Jonathon Jongsma wrote: > > virt_viewer_util_load_ui() looks first in the current directory, and then > > looks > > in the system data dirs for a ui file to load, but if you install > > virt-viewer in > > a different prefix, it will load the system UI file rather than the one > > from the > > install prefix. Try to load the ui file from pkgdatadir first. > > I'm not sure we should try to use g_get_system_data_dirs() at all as I > generally don't expect having the application look in /usr/share when I set > $prefix. > > Christophe >
it looks like the system data dirs stuff was added in commit 1002a220 to fix a problem on windows. I'm not sure of all the details since the changelog just says "Load UI file correctly, to fix Windows support". (side note: it would be nice if the commit logs had more details about what is being fixed, and why). So, I guess we should probably leave that in for the windows port, though it will never be used on linux. Jonathon _______________________________________________ virt-tools-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/virt-tools-list
