Repository navigation
headroom: init at 0.41.0 - #569784
headroom: init at 0.41.0#569784attilaolah wants to merge 1 commit into
Conversation
0d9c2cd to
46dd15e
Compare
46dd15e to
80ccc90
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
This should be ready for review now. I've been using this package for a little over a month now, on both NixOS and Darwin, both as a proxy and as an MCP server. So far no issues. |
|
@SuperSandro2000 you said in #529964 (comment) you might pick up Headroom, maybe you have some input here. |
80ccc90 to
76603f6
Compare
|
@GraysonTinker I added you since you've already looked at #529964, but feel free to suggest someone else. |
|
Will do when I get the chance, ouch that's a long derivation |
|
Thanks, and no rush at all. It is long indeed, much if it is about forcing the tests into the sandbox though… |
SuperSandro2000
left a comment
There was a problem hiding this comment.
I never got to the tests and then got distracted, but I prepared all the dependencies
This comment was marked as outdated.
This comment was marked as outdated.
e890ec2 to
81ac7cd
Compare
This comment was marked as outdated.
This comment was marked as outdated.
|
Looks like there are dynamic imports or similar in the tests, so a plain |
81ac7cd to
09265aa
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
The move from Let me fix & verify that. This fails on Darwin, but works on Linux: nix run github:NixOS/nixpkgs/fc8c84a279555a6ce7f4f916bd4a17ffed224c9a#headroom -- proxy |
|
The issue appears to be that Headroom on Darwin re-exec's Python to get a more efficient malloc, merged in headroomlabs-ai/headroom#2879. On Linux this path is never taken so the binary executes normally. On Darwin, the re-exec fails due to I'm testing a fix now. |
fc8c84a to
b3dc24c
Compare
|
Confirmed b3dc24c, |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
Version 0.41.0 is now out, I am testing it locally, will update this PR once confirmed working on Linux & Darwin. Or, if merged in the meantime, then will post it as a normal update. |
b3dc24c to
aaaf275
Compare
|
PR rebased & updated to 0.41.0.
This name collision apparently doesn't happen in Python 3.12, which is what upstream tests against. There are some efforts upstream to test against Python 3.14, but I'm not sure about the progress. The targeted Pytest fixture patch allows tests to pass. The runtime issue is still present, but is import-order dependent, and as long as the normal |
aaaf275 to
7ef9e5f
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
5657b95 to
bbf61c7
Compare
|
|
https://github.com/headroomlabs-ai/headroom/releases/tag/v0.41.0 Assisted-by: OpenCode (GPT 5.6 Terra) Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
bbf61c7 to
17532b7
Compare
|
Should be fixed now. Headroom pins |
|
|
Add Headroom. Can be used both as a proxy via
headroom proxyand as an MCP viaheadroom mcp serve.The original attempt at packaging Headroom was in #529964, thanks for the inspiration.
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.