← Back to team overview

ubuntu-appstore-developers team mailing list archive

Re: Adding "applications" to manifest file

 

On 07/18/2013 07:51 AM, Marc Deslauriers wrote:
> On 13-07-18 08:19 AM, Colin Watson wrote:
>> On Thu, Jul 18, 2013 at 08:05:06AM -0400, Marc Deslauriers wrote:
>>> On 13-07-17 12:55 PM, Colin Watson wrote:
>>>> I like this layout a lot better: it actually makes some kind of sense to
>>>> me, it's easier to read, and it repeats itself less.
>>>
>>> Except now we have to parse multiple json files and correlate the information we
>>> need. For example, we find myapp.json. We now need to parse manifest.json to
>>> figure out that myapp.json maps to the binary called "myapp", that the version
>>> is "0.1", that the desktop file is "myapp.desktop", etc. It seems to me this is
>>> overly complicated.
>>
>> Maybe it just moves the complexity around; I assure you the complexity
>> is at least as bad with the previous option, it's just not in your code.
>> :-)
> 
> Hehe :)
> 
> OK, I've re-read your updated hooks spec, and I now understand why you prefer
> having them in separate files. It makes sense to me too.
> 

Adding this complexity shouldn't be too bad for aa-clicktool actually. It always
needed to transmogrify the manifest json for easyprof. With this scheme, instead
of reading the manifest it is parsing a symlink and the fact that it is in
multiple json files instead of one shouldn't make things harder (easyprof is
happy with single or multiple entries).

-- 
Jamie Strandboge                 http://www.ubuntu.com/

Attachment: signature.asc
Description: OpenPGP digital signature


References