Skip to content

Conversation

@mscolnick
Copy link
Contributor

@mscolnick mscolnick commented Oct 10, 2025

Fixes #6669

This provides a fallback for windows logging which can throw errors like the below

--- Logging error ---
Traceback (most recent call last):
    self.rotate(self.baseFilename, dfn)
  File "C:\Users\nplev\AppData\Roaming\uv\python\cpython-3.11.13-windows-x86_64-none\Lib\logging\handlers.py", line 115, in rotate
    os.rename(source, dest)
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\\Users\\nplev\\.cache\\marimo\\logs\\marimo.log' -> 'C:\\Users\\nplev\\.cache\\marimo\\logs\\marimo.log.2025-10-04'

@vercel
Copy link

vercel bot commented Oct 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
marimo-docs Ready Ready Preview Comment Oct 10, 2025 5:43pm

@mscolnick mscolnick changed the title fix: safer windows logging with rollover fix: safer windows logging on PermissionErrors Oct 10, 2025
@mscolnick mscolnick merged commit d08dbdd into main Oct 10, 2025
46 of 77 checks passed
@mscolnick mscolnick deleted the ms/fix-windows-rollover branch October 10, 2025 21:22
@dmadisetti dmadisetti added the bug Something isn't working label Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

marimo uv sandbox crashes Windows 11 when uv run marimo edit

3 participants