Hello Brian, or anyone else affected,

Accepted apport into quantal-proposed. The package will build now and be
available at http://launchpad.net/ubuntu/+source/apport/2.6.1-0ubuntu8
in a few hours, and then in the -proposed repository.

Please help us by testing this new package.  See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to
enable and use -proposed.  Your feedback will aid us getting this update
out to other Ubuntu users.

If this package fixes the bug for you, please change the bug tag from
verification-needed to verification-done.  If it does not, change the
tag to verification-failed.  In either case, details of your testing
will help us make a better decision.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  Thank you in
advance!

** Description changed:

+ [Test Case]
+ 
+ 1) (xeyes &); sleep 1; killall -SEGV xeyes
+ 2) check /var/crash
+ 3) (xeyes &); sleep 1; killall -SEGV xeyes
+ 3) check /var/crash notice that .crash is newer than .upload and .uploaded
+ 
+ Original Description:
+ 
  I was testing errors in the cloud using juju and created a crash report
  for d-feet.  The first time I did this the a .crash file was created and
  then a corresponding .upload and .uploaded file were created.  The
  second time I did this a new .crash file was created but a new .upload
  file was not and subsequently the 2nd occurrence of this crash was not
  sent to the error tracker.
  
  My /var/crash directory:
  
  -rw-rw-r-- 1 bdmurray whoopsie       0 Nov 28 10:25 
_usr_bin_d-feet.1000.upload
  -rw------- 1 whoopsie whoopsie       0 Nov 28 13:14 
_usr_bin_d-feet.1000.uploaded
  -rw-r----- 1 bdmurray whoopsie 3401262 Nov 28 13:43 _usr_bin_d-feet.1000.crash
  
  In apport/fileutils.py we can see that there are no checks done to see
  if there is already a .upload or .uploaded file:
  
  def mark_report_upload(report):
-     report = '%s.upload' % report.rsplit('.', 1)[0]
-     with open(report, 'a'):
-         pass
+     report = '%s.upload' % report.rsplit('.', 1)[0]
+     with open(report, 'a'):
+         pass
  
  So if a .upload for the crash already exists a new one is not created
  nor is the existing one modified.

** Changed in: apport (Ubuntu Quantal)
       Status: In Progress => Fix Committed

** Tags added: verification-needed

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1084296

Title:
  possible for a 2nd occurrence of a crash not to be sent to errors

To manage notifications about this bug go to:
https://bugs.launchpad.net/apport/+bug/1084296/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to