← Back to team overview

apport-hackers team mailing list archive

[Merge] lp:~vorlon/apport/crashdigger-proper-lockfile into lp:apport


Steve Langasek has proposed merging lp:~vorlon/apport/crashdigger-proper-lockfile into lp:apport.

Requested reviews:
  Apport upstream developers (apport-hackers)

For more details, see:

Stash our pid in the lockfile so we know when the process is dead.
Your team Apport upstream developers is requested to review the proposed merge of lp:~vorlon/apport/crashdigger-proper-lockfile into lp:apport.
=== modified file 'bin/crash-digger'
--- bin/crash-digger	2012-12-12 10:13:02 +0000
+++ bin/crash-digger	2013-09-09 19:02:25 +0000
@@ -196,6 +196,7 @@
 if opts.lockfile:
         f = os.open(opts.lockfile, os.O_WRONLY | os.O_CREAT | os.O_EXCL, 0o666)
+        os.write(f, "%u\n" % os.getpid())
     except OSError as e:
         if e.errno == errno.EEXIST:

Follow ups