367677 - make error logging in updater.cpp more verbose (especially on error)

Status: RESOLVED DUPLICATE of bug 316890

(Toolkit :: Application Update, defect)

Description

[(not reading, please use [e-mail] instead)]

make error logging in updater.cpp more verbose on error

spun off from bug #367669 and bug #367084

Comment 1

[(not reading, please use [e-mail] instead)]

I've got a patch in bug #367084 to make things a little better.

if the file is in use by another process (like chrome/browser.manifest from shaver, in bug #367669)

we'll get:

remove failed: 13,-1 (chrome/browser.manifest)

note, if the file is just read only (see bug #367084), you would not see this error as the patch will attempt to change the perms.

but if that fails, you'd get:

remove failed: 13,-1 (chrome/browser.manifest)

so after my patch, there still won't be a way to determine the difference between those failures.

Comment 2

[(not reading, please use [e-mail] instead)]

sorry for the bug spam, re-assigning bugs back to default owner if I'm not working actively on them.