Skip to content

Klipper: Don't write tmp files to ramdisk#188

Merged
suchmememanyskill merged 2 commits into
mainfrom
james/klipper-tmp-dir
May 31, 2026
Merged

Klipper: Don't write tmp files to ramdisk#188
suchmememanyskill merged 2 commits into
mainfrom
james/klipper-tmp-dir

Conversation

@jamesturton
Copy link
Copy Markdown
Collaborator

Hopefully fixes #87. Not tested yet.

@github-actions
Copy link
Copy Markdown

✅ Build Artifacts

Branch: james/klipper-tmp-dir
Build: 1372ac5 (merge into main)

Artifact Size
CC1 Firmware 91.21 MB

View workflow run

@suchmememanyskill
Copy link
Copy Markdown
Contributor

suchmememanyskill commented May 20, 2026

@jamesturton i question if this is the correct fix.

Like, sure, this works technically but

  • Can we not just symlink /tmp to /user-resource/.tmp?
  • Can we PR to kalico that they start using the tempfile module from python (so we can set a temp directory like via moonraker)?

For now i think i'd prefer the latter

@jamesturton
Copy link
Copy Markdown
Collaborator Author

jamesturton commented May 26, 2026

* Can we not just symlink /tmp to /user-resource/.tmp?

Maybe, but this won't work well for any socket files. If it used /var/log that would be a better thing to symlink.

* Can we PR to kalico that they start using the tempfile module from python (so we can set a temp directory like via moonraker)?

I like this too :) Maybe I will do that

@jamesturton jamesturton deployed to approval-given May 29, 2026 22:48 — with GitHub Actions Active
@jamesturton
Copy link
Copy Markdown
Collaborator Author

@suchmememanyskill Does this look any better?

@suchmememanyskill
Copy link
Copy Markdown
Contributor

From a quick glance, looks fine

@github-actions
Copy link
Copy Markdown

✅ Build Artifacts

Branch: james/klipper-tmp-dir
Build: 759b48f (merge into main)

Artifact Size
CC1 Firmware 91.29 MB

View workflow run

@suchmememanyskill suchmememanyskill merged commit e848e46 into main May 31, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bed mcu seems to crash when testing resonnances

2 participants