-
Notifications
You must be signed in to change notification settings - Fork 518
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
r3:do(compile). kills embedded yaws service (posted per request in irc) #1549
Comments
What is your full Erlang release version?
Any idea if using make:all([load]) or manually doing c/1, l/1 on the file you changed also crashes yaws? This would narrow down if its a rebar3 issue or if yaws is not properly coded to support hotload. |
Yeah my bet there is on Yaws not coping with sequential hot-code loads and one of the modules getting killed. You can see process You can see this taking place here in the code: https://github.com/klacke/yaws/blob/b32f7d49eff120f7860453a488acbd358beb5763/src/yaws_server.erl#L421-L430 I don't know that too much can be done if this server dies. |
I'll take a look this evening and let you know what I find. Thank you for your interest and help :) |
@vans163 18.0 2> make:all([load]).
up_to_date
3> c("/home/chris/software/auth.ganimas.com/src/gani_req.erl").
{ok,gani_req}
4> l(gani_req).
{module,gani_req}
5> is there a "rebar3" way of directing the yaws compilation? I'm not sure how rebar3 is managing the yaws compilation, but @vinoski suggests compiling yaws without HiPE erlyaws/yaws#307 |
Oh, I bet it is hipe. |
How can I be certain if HiPE is disabled? I recompiled otp with "--disable-hipe", deleted the _build directory and retried but the same issue is there. |
The shell app reload blacklist feature should cover this now. See https://www.rebar3.org/docs/configuration#section-shell |
using the "rebar3 auto" or calling r3:do(compile). kills the embedded yaws service my app uses.
Environment
Current behaviour
The text was updated successfully, but these errors were encountered: