Skip to content
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

Building Erlang 23 freezes on macOS #434

Closed
eproxus opened this issue Jan 30, 2023 · 4 comments · Fixed by #437
Closed

Building Erlang 23 freezes on macOS #434

eproxus opened this issue Jan 30, 2023 · 4 comments · Fixed by #437

Comments

@eproxus
Copy link

eproxus commented Jan 30, 2023

If I try to build 23.3.4.18 on macOS Ventura 13.2 it freezes when patching configure.in at this point: https://github.com/kerl/kerl/blob/master/kerl#L768

If I check the build log at $KERL_HOME/builds/my_build_23.3.4.18/otp_build_23.3.4.18.log it only shows this text:

patching file 'erts/configure.in'
Reversed (or previously applied) patch detected!  Assume -R? [y] 

As far as I can tell, that patch is already in OTP 23 (see the list of tags here erlang/otp@4b0467c).

In fact, it even seems that the configure scripts are checked in for OTP 23 so maybe KERL_USE_AUTOCONF=1 shouldn't be used at all?

@paulo-ferraz-oliveira
Copy link
Contributor

Hey.

Yeah, I though about that in the past (that the configure scripts are checked in, and the option might be deprecated by this time around). Care to pull request removing the interesting bits 😄? CI should be able to tell us if stuff's as it should.

Cheers.

@eproxus
Copy link
Author

eproxus commented Jan 31, 2023

If I look at the OTP commit it seems it is in 23.1+, but not e.g. 23.0. I guess there's no easy way to do that test for minor versions in kerl?

@paulo-ferraz-oliveira
Copy link
Contributor

There's something similar here. Does that help?

@electricshaman
Copy link

Hilariously, I just noticed I can get past the freeze by hitting enter at the prompt regarding the patch. It's waiting for input, not doing anything. Not ideal for an automated script, but at least the build moves forward.

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 a pull request may close this issue.

3 participants