Merge lp:~seb128/notify-osd/904835 into lp:notify-osd/oneiric

Proposed by Sebastien Bacher on 2012-01-05
Status: Merged
Approved by: Mirco Müller on 2012-02-02
Approved revision: 445
Merge reported by: Mirco Müller
Merged at revision: not available
Proposed branch: lp:~seb128/notify-osd/904835
Merge into: lp:notify-osd/oneiric
Diff against target: 13 lines (+3/-0)
1 file modified
src/log.c (+3/-0)
To merge this branch: bzr merge lp:~seb128/notify-osd/904835
Reviewer Review Type Date Requested Status
Mirco Müller (community) 2012-01-05 Approve on 2012-02-02
Canonical Desktop Experience Team 2012-01-05 Pending
Review via email:

This proposal supersedes a proposal from 2012-01-05.

Description of the change

should fix bug #904835 "notify-osd keeps on writing to .cache/notify-osd.log and waking up my drive"

Turn logging on only if LOG is defined (i.e "LOG=1 notify-osd")

To post a comment you must log in.
Mirco Müller (macslow) wrote :


review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/log.c'
--- src/log.c 2010-03-05 12:51:12 +0000
+++ src/log.c 2012-01-05 14:39:12 +0000
@@ -51,6 +51,9 @@
52log_init (void)52log_init (void)
54 if (! g_getenv ("LOG"))
55 return;
54 const char *homedir = g_getenv ("HOME");57 const char *homedir = g_getenv ("HOME");
55 if (!homedir)58 if (!homedir)
56 homedir = g_get_home_dir ();59 homedir = g_get_home_dir ();


People subscribed via source and target branches