1801767 - Remove fs::remove_dir_all usage in remove_disk_cache()

Status: RESOLVED FIXED

(Core :: Graphics: WebRender, task)

Description

[Sotaro Ikeda [:sotaro]]

From Bug 1799442, fs::remove_dir_all is not stable on Windows. It needs to be replaced like Bug 1799442.

Comment 1

[Sotaro Ikeda [:sotaro]]

Attachment: Bug 1801767 - Remove fs::remove_dir_all usage in remove_disk_cache()

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e2f2270d30e2
Remove fs::remove_dir_all usage in remove_disk_cache() r=gfx-reviewers,bradwerth

Comment 3

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/e2f2270d30e2

Comment 4

[Jan de Mooij [:jandem]]

When I do a (clobber?) build of the JS shell, I now get Cargo.lock changes because something is sorting the dependencies list for me. Can we fix this? :)

  "objc",
  "rayon",
+ "remove_dir_all",
  "swgl",
  "thin-vec",
  "tracy-rs",
  "uuid",
  "webrender",
  "winapi",
  "wr_malloc_size_of",
- "remove_dir_all",
 ]

Comment 5

[Donal Meehan [:dmeehan]]

https://hg.mozilla.org/releases/mozilla-beta/rev/1ac99dfc6900

Comment 6

[Sotaro Ikeda [:sotaro]]

(In reply to Jan de Mooij [:jandem] from comment #4)

When I do a (clobber?) build of the JS shell, I now get Cargo.lock changes because something is sorting the dependencies list for me. Can we fix this? :)

  "objc",
  "rayon",
+ "remove_dir_all",
  "swgl",
  "thin-vec",
  "tracy-rs",
  "uuid",
  "webrender",
  "winapi",
  "wr_malloc_size_of",
- "remove_dir_all",
 ]

"cargo update --workspace" reproduced it for me. I am going to update it.