[Sugar-devel] [SPAM?]: Re: [PATCH sugar-toolkit] Remove incomplete MANIFEST support

[Simon Schampijer]

On 07/05/2011 03:44 PM, Gary Martin wrote:
> On 5 Jul 2011, at 10:22, Simon Schampijer<simon at schampijer.de>  wrote:
>
>> On 07/03/2011 05:54 PM, Sascha Silbe wrote:
>>> From: Bernie Innocenti<bernie at codewiz.org>
>>>
>>> This incomplete feature contributes to confuse new activity authors and
>>> slightly complicates our bundle installation logic.
>>>
>>> The day someone finds something useful to do with the MANIFEST
>>> specification, we can revert this patch in no time.
>>>
>>> Signed-off-by: Bernie Innocenti<bernie at codewiz.org>
>>> Tested-by: Bernie Innocenti<bernie at codewiz.org>
>>> Tested-by: James Cameron<quozl at laptop.org>
>>> Tested-by: Sascha Silbe<silbe at activitycentral.com>
>>> Reviewed-by: Sascha Silbe<silbe at activitycentral.com>
>>> [rebased on git master]
>>> Signed-off-by: Sascha Silbe<silbe at activitycentral.com>
>>> ---
>>> Just installed Backup [1] on an XO-1 running Sugar 0.82.1 using a
>>> bundle without MANIFEST. It installs and starts up fine, no obvious
>>> breakage. Dextrose has been shipping this patch for a long time with no
>>> reported issues either.
>>>
>>> I'm going to push this to master within the next few days.
>>>
>>> [1] http://activities.sugarlabs.org/en-US/sugar/addon/4326
>>
>> Hi Sascha,
>>
>> I have never been a big fan of the MANIFEST myself, so I generally welcome this step.
>>
>> I would like the activity authors to comment on the issue, if they see any downsides, critical paths etc. A quick test worked fine for me.
>
> My primary worry is if folks start releasing activities with no MANIFEST will it fail or degrade performance on existing releases out there in the wild, the widespread OLPC build 802 being one I would want to check first. If I remember a past thread from Bernie correctly, it works, but generates a log warning message for every file in the bundle. I need to go test, that thread was quite some time back :)
>
> --G

Hi Gary,

thanks for your reply.

I did test installing a bundle with no MANIFEST file on 0.84.x (10.1.3) 
and I did get warnings about the missing MANIFEST but that's it. 
Installed fine and runs fine. AFAIK 802 is 0.84 based as well, so I 
presume this should work equally well.

Regards,
    Simon