Closed Bug 12694 Opened 21 years ago Closed 19 years ago

File.remove() doesn't schedule in-use files for later deletion

Categories

(Core Graveyard :: Installer: XPInstall Engine, defect, P3)

PowerPC
All
defect

Tracking

(Not tracked)

VERIFIED WORKSFORME
mozilla0.9

People

(Reporter: ssu0262, Assigned: dbragg)

Details

(Whiteboard: [xpibug])

Under win95/98, if the file is in use by a program other than seamonkey,
seamonkey's smartupdate won't be able to delete it, possibly even after a
reboot too.

This also affects Install.FileDelete().
Target Milestone: M10
This works well enough for the M10 release
Target Milestone: M11 → M20
This bug would require more special handling on Win9x to put the files to be
deleted in the WININIT.INI file.  This is the ONLY way to delete a file that is
grabbed by the OS.
Bulk move of XPInstall (component to be deleted) bugs to Installer: XPInstall
Engine
QA believes this bug can be closed out. deleteRegisteredFile() doesn't work, but 
this is mentioned in the registry tracking bug (bug 28896). Test case 
a_delregfile covers this.
Is this bug now about File.remove()? Updating the summary to reflect what I 
think the bug is, please correct if wrong.
Summary: Install.DeleteFile() does not work in all cases → File.remove() doesn't schedule in-use files for later deletion
Yes, it's now about File.remove(). But it's only happening on Mac now. No 
problem deleting open files on the other platforms.
OS: Windows 98 → Mac System 9.0
Hardware: PC → Macintosh
Changing obsolete milestones to nearest moral equivalent
Target Milestone: M20 → Future
Changing milestone for reconsideration... would hate to really really need this 
sometime in the future and not have it there
Keywords: nsbeta1
OS: All
Priority: P3 → --
Target Milestone: Future → ---
Reassigning to Don
Assignee: dveditz → dbragg
Status: ASSIGNED → NEW
Whiteboard: [xpibug]
Moz 0.9 tasks
Target Milestone: --- → mozilla0.9
p3 tasks
Priority: -- → P3
I was looking at the code and it looks like it's calling
DeleteFileNowOrSchedule() -- that is, this might be fixed already. Please
verify, this could be an easy one :-)
Keywords: nsbeta1nsbeta1+
After some confusing investigation, File.remove is calling
DeleteFileNowOrSchedule.  Marking this bug as WORKSFORME since I didn't actually
fix it. 
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → WORKSFORME
Build: 2001-02-21-09-Mtrunk(WIN), 2001-02-21-04-trunk(MAC)

Works for me, too.  Marking Verified!
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.