Smalloc#253
Conversation
zooko
left a comment
zooko
left a comment
There was a problem hiding this comment.
I read over this patch and it looks correct to me.
zooko
left a comment
zooko
left a comment
There was a problem hiding this comment.
I read over this patch and it looks correct to me. :-) (That was easy.)
|
|
||
| if test "$setup_s" = "1"; then | ||
| checkout s $version_s https://github.com/zooko/smalloc | ||
| cargo build --release --package smalloc-ffi |
There was a problem hiding this comment.
You might want to add cargo as a dependency in the install packages phase.
There was a problem hiding this comment.
Yeah, I realised that, but I was nudging github from my phone. I'll fix today.
|
Hey there! Is there anything I can do to help get smalloc merged into main here? Thanks. |
|
@copilot resolve the merge conflicts in this pull request |
Sorry, I can't remember what the blocker was. I'll update the branch and see if there is anything we need from you. |
|
No prob. Looking forward to the results! :-) |
3 participants
Addresses #248. This is based on @zooko's personal fork.