Re: [sugar] [PATCH] fix #6753 Activities should be able to specify mime_types = */*
On Tue, Apr 29, 2008 at 11:19:01AM +0200, Marco Pesenti Gritti wrote: On Tue, Apr 29, 2008 at 1:36 AM, Martin Dengler [EMAIL PROTECTED] wrote: Perhaps this patch got more attention than it needed, given the questions you raised. Besides the one to which I responded, I don't know the answers. I suspect that now might not be the optimal time to answer them. I'm quite happy for this tiny patch to languish / be discarded - the advantage of such a small patch is that it costs little to generate / discard, but if it gets in it's cheap. We have to be careful mainly because it's public ABI... Sure - I'm very very for that. Rather than me saying ooh, good idea, let's do it dudez, I figured a patch would have a higher chance of being useful if people more familiar with the problem agreed. I really am happy to have it languish / be discarded; I'm not the expert, and it only took a few minutes. I'll update trac and we can get back to discussing more important issues ;). Marco Martin pgp4SNJGnfTWM.pgp Description: PGP signature ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
Re: [sugar] [PATCH] fix #6753 Activities should be able to specify mime_types = */*
What sort of activity can handle */*? Can you think of any nasty things that activities might be able to do by advertising suppport for '*/*'? Michael ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
Re: [sugar] [PATCH] fix #6753 Activities should be able to specify mime_types = */*
On Mon, Apr 28, 2008 at 01:11:47PM -0400, Michael Stone wrote: What sort of activity can handle */*? Distribute. The journal ;). Possibly others; I don't know. I'm not really the best person to justify #6753. Can you think of any nasty things that activities might be able to do by advertising suppport for '*/*'? Besides social-engineering/tricking kids into sending their data to someone maliciously? I've nowhere near the familiarity as (IMHO) yourself and others have to the problem space, so don't pretend to be able to opine on this, and have no use case for it. Please definitely consider my patch as a straw man to delay/engender discussion rather than an educated attempt to solve a current problem. I figure as others might have a use case, and you/others have the experience with what works and what doesn't, I'd provide a possibly useful shortcut to the minimum # of lines of code that could be changed to satisfy the use case to save others/you that small amount of time. If it's an interesting problem to consider right now, given everything else that we could be talking about on sugar@ ;). Michael pgpBcNMuSSTqb.pgp Description: PGP signature ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
Re: [sugar] [PATCH] fix #6753 Activities should be able to specify mime_types = */*
On Mon, Apr 28, 2008 at 10:32:48PM +0100, Martin Dengler wrote: On Mon, Apr 28, 2008 at 01:11:47PM -0400, Michael Stone wrote: What sort of activity can handle */*? Distribute. The journal ;). Possibly others; I don't know. I'm not really the best person to justify #6753. Distribute is an excellent example; thanks. Can you think of any nasty things that activities might be able to do by advertising suppport for '*/*'? Besides social-engineering/tricking kids into sending their data to someone maliciously? Well, what would happen if people installed lots of activities supporting '*/*'? * What happens if those are funky hidden activities like Read? * Would our Palettes scroll, or scale, or run off screen, or ...? * Would the Journal be okay if all entries can be opened by, say, ten activities? * Does the Journal/DS's default activity for this mime-type selection mechanism function correctly with */*, *, */foo, foo/*, etc. mimetypes? [other comments] Please keep submitting patches. You're doing a good job. I'm just using your work to try to provoke discussion about how well our designs cope with corner cases. Best, Michael ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar
Re: [sugar] [PATCH] fix #6753 Activities should be able to specify mime_types = */*
On Mon, Apr 28, 2008 at 06:31:00PM -0400, Michael Stone wrote: Well, what would happen if people installed lots of activities supporting '*/*'? ... * Would our Palettes scroll, or scale, or run off screen, or ...? I think the UI is probably going to lag behind gracefully handling this case, not in the least because I just blatted my patch out in 15 minutes without mentioning it to any UI people. I don't know of many activities that really need this feature. The first thing that comes to mind is that there is no way for an activity to request it be the top of the list given no other hints, which could be useful. The second thing that comes to mind is that on IRC, when discussed ages ago, IIRC, the list of activities that should be suggested was to be ordered most recently used first. Please keep submitting patches. You're doing a good job. I'm just using your work to try to provoke discussion about how well our designs cope with corner cases. Perhaps this patch got more attention than it needed, given the questions you raised. Besides the one to which I responded, I don't know the answers. I suspect that now might not be the optimal time to answer them. I'm quite happy for this tiny patch to languish / be discarded - the advantage of such a small patch is that it costs little to generate / discard, but if it gets in it's cheap. Michael Martin pgp4TWgSxHsTn.pgp Description: PGP signature ___ Sugar mailing list Sugar@lists.laptop.org http://lists.laptop.org/listinfo/sugar