LiquidBounce icon indicating copy to clipboard operation
LiquidBounce copied to clipboard

[BUG] Destruct does not delete all folders and mod jar

Open libx264 opened this issue 8 months ago • 5 comments

LiquidBounce Branch

Nextgen

LiquidBounce Build/Version

0.29.0

Operating System

Windows 11

Minecraft Version

1.21.4

Describe the bug

It leaves deeplearning, mcef, themes and mod jar

Steps to reproduce

  • Join any server
  • Send .client destruct yes yes
  • Open game folder
  • See LiquidBounce folder and mod in mods

Client Log

[21:15:15] [Render thread/INFO]: [System] [CHAT] LiquidBounce ▸ LiquidBounce is being destructed from your client...
[21:15:15] [Render thread/INFO]: [System] [CHAT] LiquidBounce ▸ LiquidBounce has been destructed from your client. You can clear your chat using F3+D. If wipe was enabled, the chat will be cleared automatically.
[21:15:15] [Render thread/INFO]: [System] [CHAT] LiquidBounce ▸ Wiping client...

Screenshots

Image

libx264 avatar Apr 16 '25 19:04 libx264

#5952 #5877 happens because this

libx264 avatar Apr 17 '25 20:04 libx264

[00:01:34] [wipe-client/INFO] (Minecraft) [STDERR]: java.lang.UnsupportedOperationException
[00:01:34] [wipe-client/INFO] (Minecraft) [STDERR]: 	at java.base/java.util.Collections$UnmodifiableCollection.removeIf(Collections.java:1120)
[00:01:34] [wipe-client/INFO] (Minecraft) [STDERR]: 	at knot//net.ccbluex.liquidbounce.features.misc.HideAppearance.wipeClient$lambda$15(HideAppearance.kt:180)
[00:01:34] [wipe-client/INFO] (Minecraft) [STDERR]: 	at knot//kotlin.concurrent.ThreadsKt$thread$thread$1.run(Thread.kt:30)

MukjepScarlet avatar Apr 18 '25 16:04 MukjepScarlet

In my test jcef_helper.exe is not stopped after wipe, can you try that?

taskkill -f -im jcef_helper.exe after wipe finished

MukjepScarlet avatar Apr 18 '25 16:04 MukjepScarlet

In my test jcef_helper.exe is not stopped after wipe, can you try that?

taskkill -f -im jcef_helper.exe after wipe finished

From my testing killing jcef_helper crashes the Minecraft process. The issue with destruct is that it cannot delete files that are actively used. On Linux and macOS this is not an issue and works flawless, but on Windows it's not possible.

1zun4 avatar Apr 18 '25 22:04 1zun4

I tested it, if I kill the process before destruct, it just restarts, if after, it does not restart and not crashing, but the file deletion does not continue.

libx264 avatar May 15 '25 08:05 libx264

Can't be fixed because ntfs file locking and dlj, jcef dll natives

libx264 avatar Oct 04 '25 20:10 libx264