Code review comment for lp:~indicator-applet-developers/evolution-indicator/blacklist-support

Revision history for this message
Neil J. Patel (njpatel) wrote :

> On Wed, 2009-09-16 at 14:40 +0000, Neil J. Patel wrote:
> > Creates/removes the blacklist file when appropriate.
>
> You probably need to create USER_BLACKLIST_DIR as it won't always exist.
> I don't think that build_filename will do that.

Nice catch, I added support for that in rev 53. It seems to work fine when I remove any of the directories in the ~/.config/indicators/messages/applications-blacklist chain.

« Back to merge proposal