Skip to content

Conversation

@ospencer
Copy link
Collaborator

No description provided.

@ospencer ospencer requested review from danielledeleo and flaki June 23, 2022 19:34
@ospencer ospencer self-assigned this Jun 23, 2022
Copy link

@flaki flaki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah with suborbital/reactr#272 and an updated Node library plus a patched javy using these updated bindings I could get setHeader() working in JS:

https://functions.fly.dev/ (that is https://github.com/suborbital/flyfun)

@flaki
Copy link

flaki commented Jun 28, 2022

#26 0.222 			&& cargo build --release --target=wasm32-wasi \
#26 0.222 			&& cd -
: profiles for the non root package will be ignored, specify profiles at the workspace root:
#26 1.566 package:   /root/crates/core/Cargo.toml
#26 1.566 workspace: /root/Cargo.toml
#26 12.06     Updating crates.io index
#26 78.36 Killed

Could it be that something is timing out here? @arbourd

@arbourd
Copy link

arbourd commented Jun 28, 2022

RIght above @flaki

#26 [linux/arm64 builder 8/8] RUN make cli
#26 78.36 Killed
#26 78.43 make: *** [Makefile:16: core] Error 137
#26 ERROR: process "/bin/sh -c make cli" did not complete successfully: exit code: 2

Maybe an OOM in the container?

@flaki
Copy link

flaki commented Jun 28, 2022

Maybe an OOM in the container?

The free fly builder comes with 256MB and it builds this happily so that would be...weird.
(*unless this is a swap-configuration thing)

@ospencer ospencer force-pushed the oscar/bind-resp-set-header branch from 5b5061c to 561d112 Compare July 28, 2022 16:23
@ospencer ospencer merged commit 48b7d28 into main Jul 28, 2022
@ospencer ospencer deleted the oscar/bind-resp-set-header branch July 28, 2022 20:50
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.

5 participants