crash log path should decode spaces ?

cogentcore / core

A free and open source framework for building powerful, fast, and elegant 2D and 3D apps that run on macOS, Windows, Linux, iOS, Android, and the Web with a single pure Go codebase, allowing you to Code Once, Run Everywhere.

http://cogentcore.org/core

BSD 3-Clause "New" or "Revised" License

1.31k stars 71 forks source link

crash log path should decode spaces ? #966

Closed gedw99 closed 2 weeks ago

gedw99 commented 1 month ago

Describe the bug

----- END OF STACK TRACE -----
2024/05/19 12:15:03 SAVED CRASH LOG TO /Users/apple/Library/Plot View/crash-logs/crash_2024-05-19_12-15-03

The actual path needs tube "escaped" to allow for the way spaces are done for folders in MAC OS, and perhaps other OS too. Correct output is:

/Users/apple/Library/Plot\ View/crash-logs/crash_2024-05-19_12-15-03

How to reproduce

I got this running the Plotview demo.

Example code

No response

Relevant output

No response

Platform

macOS

kkoreilly commented 1 month ago

Yep, I will work on implementing this soon. Thanks for reporting this!

kkoreilly commented 1 month ago

Also, I am in the middle of a major rewrite of our widget configuration structure (see the tree branch), so it may take a little longer than usual for this to be fixed on the main branch.

gedw99 commented 1 month ago

No worries. Thanks for letting me know…

will leave it open as an issue

kkoreilly commented 2 weeks ago

Apologies for the delayed fix; I just finished (almost all of) the major refactoring I was working on. It should be working now; please let me know if you have any other issues.