[sugar] Re: [PATCH] Bundle API now provides write functionality.

Marco Pesenti Gritti mpg at redhat.com
Fri Mar 2 12:45:32 EST 2007


On Fri, 2007-03-02 at 12:26 -0500, Dan Williams wrote:
> Marco,
> 
> Andrew commented yesterday that the use-case is so that Develop can
> write the activity's .info file without having to duplicate code.
> 

Oh my question about the use case was more about the new "comment"
field, which is not on the spec yet.

> The reason I like this patch is that it doesn't duplicate code.
> 
> The reason I don't like the "set*" parts of this patch is that it
> provides API to let any activity modify it's own .info file at any time,
> which should be _clearly_ forbidden.  If you give people API, they will
> use it.
> 
> So I don't know.  On the balance, I feel like .info files should be
> read-only and those activities that need to modify one should do it
> themselves.  We've got a .info spec anyway, so it should be relatively
> stable.

I tend to agree.

Marco



More information about the Sugar mailing list