[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-11-11 Thread Curtis Hovey
** No longer affects: null -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/612344 Title: Blacklist API sucks Status in Unity: Invalid Status in Unity Files Lens: Invalid

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-05-07 Thread Siegfried Gevatter
** Changed in: zeitgeist Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/612344 Title: Blacklist API sucks Status in NULL Projec

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-05-07 Thread Launchpad Bug Tracker
** Branch linked: lp:~zeitgeist-packagers/zeitgeist/debian -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/612344 Title: Blacklist API sucks Status in NULL Project: Invali

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-04-07 Thread मनीष सिन्हा
** Changed in: null Status: Triaged => Invalid -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/612344 Title: Blacklist API sucks Status in NULL Project: Invalid Sta

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-04-07 Thread Seif Lotfy
** Changed in: zeitgeist Status: In Progress => Fix Committed -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/612344 Title: Blacklist API sucks Status in NULL Project

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-04-05 Thread मनीष सिन्हा
** Branch unlinked: lp:~manishsinha/zeitgeist/fix-blacklist-api ** Branch linked: lp:~manishsinha/zeitgeist/new-blacklist-implementation -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-03-10 Thread Mikkel Kamstrup Erlandsen
** Changed in: unity-place-files Status: Triaged => Invalid ** Changed in: unity Status: Triaged => Invalid ** Changed in: unity-place-files (Ubuntu) Status: Triaged => Invalid -- You received this bug notification because you are a member of Zeitgeist Framework Team, which

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-02-21 Thread Didier Roche
** Changed in: unity Status: New => Triaged ** Changed in: unity-place-files (Ubuntu) Status: New => Triaged -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/61234

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-13 Thread Seif Lotfy
** Changed in: zeitgeist Milestone: 0.7.0 => 0.8.0 -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/612344 Title: Blacklist API sucks Status in NULL Project: Triaged St

Re: [Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread Siegfried Gevatter
2011/1/6 Markus Korn : > @Siegfried, how do you want to identify a template in the collection of > all templates, for example, what do you propose as an argument for the > RemoveTemplate() method? Sending the template (like we're doing now) would be an option. If you're not happy with that, we cou

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread Markus Korn
@Siegfried, how do you want to identify a template in the collection of all templates, for example, what do you propose as an argument for the RemoveTemplate() method? Do you want automatically generated identifier (e.g. auto-integer-id, maybe in combination with sender string)? or do you have a s

Re: [Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread Siegfried Gevatter
I still don't see why blacklist templates would need a manually-constructed identifier... -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/612344 Title: Blacklist API sucks S

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread Markus Korn
My first idea related to RegEx based matches was that I thought we need something very powerful there. But maybe I'm wrong, So I'm happy to change my proposal there and only allow prefix-search and negation. And if we need something more powerful later on we can always change it. -- You received

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread Markus Korn
Please let me explain how I see the whole 'identifier' story: I don't think we need a identifier which tells us which client set an certain blacklist, simply because the client which sets a blacklist might not be related to the actual purpose of this blacklist, let me give you an example: A cale

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread Michal Hruby
My first thought was also "Why regexes, glob patterns would do the trick", so I'm with Mikkel on this one. Overall I think it's what we agreed on with some polishing (the enum in the signal, great formalization of everything...), therefore I'm fine with it. -- You received this bug notification b

Re: [Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread मनीष सिन्हा
On Thu, Jan 6, 2011 at 5:29 PM, Seif Lotfy <612...@bugs.launchpad.net>wrote: > While I actually like Markus's proposal alot (kudos on the awesome work) I > am kind of hesitant on the BlacklistTemplateIdentifier. I feel more > comfortable using an identifier like "actor/#uid" as in > "application:/

Re: [Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread Seif Lotfy
One thing I am missing here and maybe its too new for the proposal is temporary blacklists. Such as: Don't log for the next 20 minutes. Don't log Documents from 19.7 - 19.8 etc... I don't see this option at all in our current API ideas. While I actually like Markus's proposal alot (kudos on the aw

Re: [Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread Mikkel Kamstrup Erlandsen
On 6 January 2011 10:43, Markus Korn wrote: > After re-reading this bugreport, the discussion it includes and the merge > proposal, I'm still not confident that the API which is proposed here is good > enough. This is why I started working on a blacklist API spec. > It is still work in progress,

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-06 Thread Markus Korn
After re-reading this bugreport, the discussion it includes and the merge proposal, I'm still not confident that the API which is proposed here is good enough. This is why I started working on a blacklist API spec. It is still work in progress, and I'm not sure if this points in the right direct

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2011-01-02 Thread Seif Lotfy
** Branch linked: lp:~manishsinha/zeitgeist/fix-blacklist-api -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/612344 Title: Blacklist API sucks Status in NULL Project: Tri

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-12-17 Thread Manish Sinha
** Changed in: zeitgeist Status: Triaged => In Progress -- You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. https://bugs.launchpad.net/bugs/612344 Title: Blacklist API sucks Status in NULL Project: Tr

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-12-15 Thread Manish Sinha
So the final consensus is that the parameter s is any randomly chosen identifier for an application. * GetBlacklists() -> ({sE}) * AddBlacklist(sE) --> Changed(sE) * RemoveBlacklist(sE) --> Changed(sE) I changed the name to Blacklist instead of Template for the method names as it sounds better a

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-12-01 Thread Neil J. Patel
** Project changed: unity => null -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in NULL Project: Triaged Status in Unity Files Place: Triage

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-11-29 Thread Seif Lotfy
** Changed in: zeitgeist Milestone: None => 0.7.0 -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in Unity: Triaged Status in Unity Files

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-11-26 Thread Manish Sinha
> Let 'E' define the event dbus signature. > > Methods: > - GetTemplates() -> ({sE}) > - AddTemplate(s, E) > - RemoveTemplate(s) > Signals: > - Changed({sE}) > Apps would use their normal namespaced bus names to construct blacklist > names. > Like 'org.gnome.Epiphany.PrivateBrowsing' for the

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-11-25 Thread Seif Lotfy
** Changed in: zeitgeist Milestone: 0.7.0 => None -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in Unity: Triaged Status in Unity Files

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-11-10 Thread Didier Roche
** Changed in: unity Status: New => Triaged ** Changed in: unity Importance: Undecided => Low ** Also affects: unity-place-files Importance: Undecided Status: New ** Changed in: unity-place-files Importance: Undecided => Low ** Changed in: unity-place-files Status:

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-11-09 Thread Seif Lotfy
At some point people will need to blacklist over Unity. I think that should be taken in consideration in the new design and development cycle. Example: What if I don't want my .py files to show up in Unity ** Also affects: unity Importance: Undecided Status: New -- Blacklist API sucks

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-10-25 Thread Seif Lotfy
with only a week left and no decision on the development method and API I postponed the bug to 0.7 ** Changed in: zeitgeist Milestone: 0.6 => 0.7 -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitgeist Framework T

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-09-29 Thread Seif Lotfy
thekorn> it sucks, maybe. but do we need a new API for this, that's what I'm not sure about thekorn, u can see mhr3 and kamstrup got into the details of it they wrote the whoel api actually seiflotfy: look into #ubuntu-classroom * mvo has quit (Quit: Ex-Chat) thekorn: what we should really add

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-09-28 Thread Manish Sinha
Meh, type in last comment scrap "This can be possible since people don't want fine grained privacy" I meant This can be possible since many people do want fine grained privacy -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a memb

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-09-28 Thread Manish Sinha
If we had huge number of blacklists. This can be possible since people don't want fine grained privacy. We can set the default modes and give them option. E.g. For chrome, we will log all the events in normal browsing and no logging in incognito mode. Additionally people sometimes watch pr0n in n

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-09-22 Thread Seif Lotfy
Go Manish! Go Manish! Its your birthday :P (please ignore this comment) ** Changed in: zeitgeist Assignee: Seif Lotfy (seif) => Manish Sinha (manishsinha) -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitgeist F

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-09-10 Thread Mikkel Kamstrup Erlandsen
@Manish: Right, that will clear all blacklists I am afraid. Which is also partially why we have this bug open to come up with something which is easier for apps to handle. -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of

Re: [Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-09-10 Thread Siegfried Gevatter
2010/9/10 Manish Sinha <612...@bugs.launchpad.net>: > Will this clear all the blacklists? I just want to remove one specific > blacklist Ask for the list of blacklisting rules, remove the one you don't want from it, and set it again (with that rule removed). -- Blacklist API sucks https://bugs.l

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-09-10 Thread Manish Sinha
Mikkel Will this clear all the blacklists? I just want to remove one specific blacklist -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in Ze

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-09-09 Thread Mikkel Kamstrup Erlandsen
(Note for people looking for ways to clear the blacklist with the currently released API: Just SetBlacklist([])) -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-09-07 Thread Seif Lotfy
** Changed in: zeitgeist Milestone: None => 0.6 -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in Zeitgeist Framework: Triaged Bug descr

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-08-30 Thread Seif Lotfy
** Changed in: zeitgeist Importance: Undecided => Medium ** Changed in: zeitgeist Status: New => Triaged ** Changed in: zeitgeist Assignee: Michal Hruby (mhr3) => Seif Lotfy (seif) -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification be

Re: [Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-08-05 Thread Mikkel Kamstrup Erlandsen
On 5 August 2010 23:04, Siegfried Gevatter wrote: > 2010/8/5 Mikkel Kamstrup Erlandsen : >> Apps would use their normal namespaced bus names to construct blacklist >> names. Like 'org.gnome.Epiphany.PrivateBrowsing' for the blacklist >> template epiphany install when you enter private mode. > > Wh

Re: [Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-08-05 Thread Siegfried Gevatter
2010/8/5 Mikkel Kamstrup Erlandsen : > Apps would use their normal namespaced bus names to construct blacklist > names. Like 'org.gnome.Epiphany.PrivateBrowsing' for the blacklist > template epiphany install when you enter private mode. Why would it use blacklisting for that? The extension should

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-08-05 Thread Mikkel Kamstrup Erlandsen
So if we have 10 clients monitoring the blacklist the daemon it will result in 10 GetTemplates() calls on the bus? ;-) How about putting the new list of templates in the Changed() signal? That way there'll be no extra noise on the buzz when the list changes. Anyway, I am still not entirely convinc

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-08-05 Thread Michal Hruby
Ok, as mentioned in the first post I'd suggest this: [org.gnome.zeitgeist.Blacklist] inteface: Methods: - GetTemplates() -> (Array of Events) - AddTemplate(Event) - RemoveTemplate(Event) Signals: - Changed() A few notes: templates must be unique, adding the same template twice will result in

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-08-02 Thread Mikkel Kamstrup Erlandsen
I suggest you post a full API draft before implementing this. We have the current (admittedly not-so-good) API because it was hard coming up with a nice API for this. -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitg

[Zeitgeist] [Bug 612344] Re: Blacklist API sucks

2010-08-02 Thread Seif Lotfy
** Changed in: zeitgeist Assignee: (unassigned) => Michal Hruby (mhr3) -- Blacklist API sucks https://bugs.launchpad.net/bugs/612344 You received this bug notification because you are a member of Zeitgeist Framework Team, which is subscribed to Zeitgeist Framework. Status in Zeitgeist Frame