Skip to content

+gleam+erlang#138

Merged
jhheider merged 2 commits into
mainfrom
+gleam+erlang
Feb 10, 2023
Merged

+gleam+erlang#138
jhheider merged 2 commits into
mainfrom
+gleam+erlang

Conversation

@jhheider
Copy link
Copy Markdown
Contributor

@jhheider jhheider commented Jan 20, 2023

  • gleam.run compiles to >
  • erlang.org

(epmd segfaults when trying to bind inet in gha ubuntu. ugh.)

resolves #342

@jhheider jhheider force-pushed the +gleam+erlang branch 9 times, most recently from b70df64 to b3bf8d0 Compare January 20, 2023 10:29
@jhheider jhheider marked this pull request as draft January 20, 2023 10:51
@philipgiuliani
Copy link
Copy Markdown

philipgiuliani commented Feb 7, 2023

Whats the status of this PR? I am interested in adding elixir-lang, but need erlang for it.

@jhheider
Copy link
Copy Markdown
Contributor Author

jhheider commented Feb 7, 2023

Stalled for erlang segfaulting. Feel free to look at it, if you want.

@philipgiuliani
Copy link
Copy Markdown

Hi @jhheider,

I've done some experiments. I noticed that the only pipeline that fails, is using a pipeline hosted by GitHub, while the other ones are self-hosted.

I have tried to run the tests locally inside a container using the ubuntu:latest image and the tests passed as expected.

The error message we see in the CI means that the beam cannot connect to epmd. I guess the segfault we see is also from epmd which cannot start. The escript executes fine, but returns econnrefused.

epmd, by default, is binding to the public interface (0.0.0.0), it's possible that this is not allowed inside a Github CI runner? I couldn't find any resource/docs about that.

So one test I would suggest is to add -kernel inet_dist_use_interface {127,0,0,1} inside the test script. If this doesn't work we could try starting epmd as a test with the debug flag (epmd -d).

@jhheider
Copy link
Copy Markdown
Contributor Author

So one test I would suggest is to add -kernel inet_dist_use_interface {127,0,0,1} inside the test script. If this doesn't work we could try starting epmd as a test with the debug flag (epmd -d).

let's try it. it seemed like maybe it was trying to create docker instances, which might be a problem.

@jhheider jhheider force-pushed the +gleam+erlang branch 2 times, most recently from 3b90ecb to c4740ac Compare February 10, 2023 03:39
@mfts mfts mentioned this pull request Feb 10, 2023
@jhheider jhheider marked this pull request as ready for review February 10, 2023 23:56
@jhheider
Copy link
Copy Markdown
Contributor Author

Part of it was segfaults on x86-64 without PIE (as we often see in compiled C). Passing now.

@jhheider jhheider merged commit 89e82f9 into main Feb 10, 2023
@jhheider jhheider deleted the +gleam+erlang branch February 10, 2023 23:57
@mxcl
Copy link
Copy Markdown
Contributor

mxcl commented Feb 11, 2023

beautiful work

@mxcl
Copy link
Copy Markdown
Contributor

mxcl commented Feb 11, 2023

Guess we now need https://hex.pm

@jhheider
Copy link
Copy Markdown
Contributor Author

😵

@rcastrejon rcastrejon mentioned this pull request Apr 26, 2023
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.

+erlang

3 participants