[virt-tools-list] [PATCH 3/4] install script: add install_script_avatar entity

[Zeeshan Ali (Khattak)]

On Tue, Aug 28, 2012 at 1:54 AM, Fabiano Fidêncio <fabiano at fidencio.org> wrote:
> On Mon, Aug 27, 2012 at 7:43 PM, Zeeshan Ali (Khattak)
> <zeeshanak at gnome.org> wrote:
>> On Tue, Aug 28, 2012 at 12:57 AM, Fabiano Fidêncio <fabiano at fidencio.org> wrote:
>>> On Mon, Aug 27, 2012 at 6:03 PM, Zeeshan Ali (Khattak)
>>> <zeeshanak at gnome.org> wrote:
>>>> On Sun, Aug 26, 2012 at 11:50 AM, Fabiano Fidêncio <fabiano at fidencio.org> wrote:
>>>>> Add a new entity called OsinfoInstallScriptAvatar to help the creation
>>>>> of an Avatar file for an install script. If avatar entity needs to be
>>>>> set, just add, in the .xml's script file:
>>>>> <avatar>
>>>>>     <image-type>...</image-type>
>>>>>     <extension>...</extension>
>>>>>     <alpha>...</alpha>
>>>>>     <width>...</width>
>>>>>     <height>...</height>
>>>>> </avatar>
>>>>
>>>> I don't quit follow one major thing here: What is the direction of the
>>>> OsinfoInstallScriptAvatar? Is app setting it on libosinfo or app
>>>> getting it from libosinfo? How would a typical usage look like?
>>>
>>> App is getting it from libosinfo. With this entity, we can drop
>>> AvatarFormat boxes' class and get this info from install-script.
>>
>> Ah ok so its all about script telling app that if it needs to add a
>> user avatar, which format it needs to be in? If so, how does app know
>> which filename to use with avatar?
>
> How is it done in gnome-boxes?

You are writing the Boxes patch so you tell me. :) If you are asking
about existing code, it simply uses the username + extention as the
filename. We'll need a way to create/get avatar filename for/to the
application. My idea would be to treat this similar to how we deal
with windows XP's .cmd and .reg files. They are not scripts either but
we treat them as such.

BTW, how does this patch work with your avatar releted code in commit 1d9822ae ?


-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124