ubuntu-download-manager

stdout and stderr of child processes are not logged

Bug #1205269 reported by Manuel de la Peña
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-download-manager
Fix Released
High
Manuel de la Peña
ubuntu-download-manager 0.3
ubuntu-download-manager (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

The stdout and stderr of the child process are not logged.

Related branches

lp:~mandel/ubuntu-download-manager/assert-return-code
Diego Sarmentero (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 425 lines (+167/-76)
9 files modified
libubuntudownloadmanager/download.cpp (+25/-19)
libubuntudownloadmanager/network_reply.cpp (+4/-36)
libubuntudownloadmanager/network_reply.h (+0/-7)
libubuntudownloadmanager/process.cpp (+20/-10)
libubuntudownloadmanager/process.h (+2/-3)
ubuntu-download-manager-tests/fake_process.cpp (+6/-1)
ubuntu-download-manager-tests/fake_process.h (+2/-0)
ubuntu-download-manager-tests/test_download.cpp (+105/-0)
ubuntu-download-manager-tests/test_download.h (+3/-0)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-download-manager at revision 90, scheduled for release in ubuntu-download-manager, milestone 0.3

Changed in ubuntu-download-manager:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-download-manager - 0.2+13.10.20130727-0ubuntu1

---------------
ubuntu-download-manager (0.2+13.10.20130727-0ubuntu1) saucy; urgency=low

  [ Manuel de la Pena ]
  * Improved the use of the child process by: * Send the error signals
    when the return code != 0 or when the exit status is crash. * Log
    the stdout and stderr of the child proccess. (LP: #1205092,
    #1205269)
  * Append to the log instead of truncate. Later deal with rotations.
  * Define the app as always in DEBUG until we are mature.

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 92
 -- Ubuntu daily release <email address hidden> Sat, 27 Jul 2013 07:01:47 +0000

Changed in ubuntu-download-manager (Ubuntu):
status: New → Fix Released
Manuel de la Peña (mandel)
Changed in ubuntu-download-manager:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.