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: mp+87629@code.launchpad.net

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 :

Approved.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/log.c'
2--- src/log.c 2010-03-05 12:51:12 +0000
3+++ src/log.c 2012-01-05 14:39:12 +0000
4@@ -51,6 +51,9 @@
5 void
6 log_init (void)
7 {
8+ if (! g_getenv ("LOG"))
9+ return;
10+
11 const char *homedir = g_getenv ("HOME");
12 if (!homedir)
13 homedir = g_get_home_dir ();

Subscribers

People subscribed via source and target branches