Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theia Blueprint fails to delete file to the trash #12780

Closed
Tracked by #12856
kittaakos opened this issue Jul 30, 2023 · 3 comments · Fixed by #12797
Closed
Tracked by #12856

Theia Blueprint fails to delete file to the trash #12780

kittaakos opened this issue Jul 30, 2023 · 3 comments · Fixed by #12797
Labels
filesystem issues related to the filesystem

Comments

@kittaakos
Copy link
Contributor

kittaakos commented Jul 30, 2023

Bug Description:

When I delete a file with Theia Blueprint 1.39.0, I get an exception, and the file deletion to the trash fails. I think this is a build problem. This is likely due to some missing executes in the webpacked backend application.

Originally reported here:

Caused by: EntryNotFound (FileSystemError): Error: spawn E:\incoming\review\2144\1-090098d\resources\app\lib\backend\windows-trash.exe ENOENT
at u (E:\incoming\review\2144\1-090098d\resources\app\lib\backend\main.js:2:1306074)

I could reproduce it on macOS with the Theia Blueprint (1.39.0) app:

Screenshot 2023-07-30 at 17 58 12
023-07-30T15:58:04.844Z root ERROR Error deleting with trash: Error: Request 'delete' failed
    at Proxy.<anonymous> (file:///Applications/TheiaBlueprint%201.39.0.app/Contents/Resources/app/lib/frontend/bundle.js:2:2325688)
    at b.delete (file:///Applications/TheiaBlueprint%201.39.0.app/Contents/Resources/app/lib/frontend/bundle.js:2:3041806)
    at x.doDelete (file:///Applications/TheiaBlueprint%201.39.0.app/Contents/Resources/app/lib/frontend/bundle.js:2:2964609)
    at async x.delete (file:///Applications/TheiaBlueprint%201.39.0.app/Contents/Resources/app/lib/frontend/bundle.js:2:2963744)
    at async p.moveFileToTrash (file:///Applications/TheiaBlueprint%201.39.0.app/Contents/Resources/app/lib/frontend/bundle.js:2:8366190)
    at async Promise.all (index 1)
    at async p.delete (file:///Applications/TheiaBlueprint%201.39.0.app/Contents/Resources/app/lib/frontend/bundle.js:2:8365936)
    at async Promise.all (index 0)
    at async p.execute (file:///Applications/TheiaBlueprint%201.39.0.app/Contents/Resources/app/lib/frontend/bundle.js:2:8364422)
    at async g.executeCommand (file:///Applications/TheiaBlueprint%201.39.0.app/Contents/Resources/app/lib/frontend/bundle.js:2:2247983)
Caused by: EntryNotFound (FileSystemError): Error: spawn /Applications/TheiaBlueprint 1.39.0.app/Contents/Resources/app/lib/backend/macos-trash ENOENT
    at u (/Applications/TheiaBlueprint 1.39.0.app/Contents/Resources/app/lib/backend/557.js:1:61635)
    at A.toFileSystemProviderError (/Applications/TheiaBlueprint 1.39.0.app/Contents/Resources/app/lib/backend/987.js:2:118859)
    at A.delete (/Applications/TheiaBlueprint 1.39.0.app/Contents/Resources/app/lib/backend/987.js:2:113571)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async g.onRequest (/Applications/TheiaBlueprint 1.39.0.app/Contents/Resources/app/lib/backend/19.js:1:53110)
    at async c.handleRequest (/Applications/TheiaBlueprint 1.39.0.app/Contents/Resources/app/lib/backend/19.js:1:46888)

I see the same problem downstream, starting the app from the sources. Quick fix: arduino/arduino-ide@129436b.

Steps to Reproduce:

file_deletion.mp4
  1. Start The Blueprint 1.39.0
  2. Open a workspace with at least one file
  3. Delete the file

Additional Information

  • Operating System: macOS 13.4.1
  • Theia Version: 1.39.0 Theia Blueprint
@kittaakos kittaakos changed the title Theia Blueprint fails to delete file Theia Blueprint fails to delete file to the trash Jul 30, 2023
@msujew msujew added the filesystem issues related to the filesystem label Aug 2, 2023
@JonasHelming
Copy link
Contributor

@msujew Is this something we should patch on the community release?

@msujew
Copy link
Member

msujew commented Aug 22, 2023

@JonasHelming This issue is only relevant for users that are bundling their backend. But yes, I imagine it makes sense to put into the community release, especially since the fix is pretty small.

@JonasHelming
Copy link
Contributor

@tsmaeder Please patch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
filesystem issues related to the filesystem
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants