-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Fix Erlang compiler & convert it to typescript #4124
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
@@ -1,7 +1,7 @@ | |||
# Default settings for Erlang | |||
compilers=&erlang | |||
compilerType=erlang | |||
versionFlag= | |||
versionFlag=-noshell -eval io:fwrite("~s~n", [erlang:system_info(otp_release)]), halt(). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh blimey! You had to learn erlang
for this!? :D
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hahaha no! I copied the flags over from the original compiler class
this doesn't work, but i'll see if I can fix it in main |
Fixed in 9f8d2cd Though sidenote; I'm a bit confused now why this noshell eval code is in the optionsForFilter function, it seems not needed. But this was added from the beginning in #3094 .. so I don't know what the deal is |
Thank you so much Partouf. I did indeed not test it as I have no Erlang setup locally. Should have tested in staging, sorry! |
this is live |
Recent refactors to base-lang broke it
Closes #4121