← 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:
https://code.launchpad.net/~vorlon/apport/crashdigger-proper-lockfile/+merge/184660

Stash our pid in the lockfile so we know when the process is dead.
-- 
https://code.launchpad.net/~vorlon/apport/crashdigger-proper-lockfile/+merge/184660
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:
     try:
         f = os.open(opts.lockfile, os.O_WRONLY | os.O_CREAT | os.O_EXCL, 0o666)
+        os.write(f, "%u\n" % os.getpid())
         os.close(f)
     except OSError as e:
         if e.errno == errno.EEXIST:


Follow ups