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

Test Erlang/OTP 26 + Elixir 1.14 on top Windows #191

Merged
merged 5 commits into from Jun 24, 2023

Conversation

paulo-ferraz-oliveira
Copy link
Collaborator

@paulo-ferraz-oliveira paulo-ferraz-oliveira commented Apr 28, 2023

Let's keep this open until building OTP 26 on Windows is possible.

This seems to be in a mergeable state, now. Just need to make sure CI is issue-less.(1)

Hopefully fixes #189.


(1) Yeah, there's some confusion with master v. main but I'll change everything to main.

@paulo-ferraz-oliveira paulo-ferraz-oliveira force-pushed the feature/otp-26-on-windows branch 4 times, most recently from 68c4c94 to f54c9d9 Compare April 28, 2023 17:37
@paulo-ferraz-oliveira paulo-ferraz-oliveira force-pushed the feature/otp-26-on-windows branch 3 times, most recently from 828aeef to e4a93f0 Compare May 8, 2023 23:23
@paulo-ferraz-oliveira paulo-ferraz-oliveira added the help wanted Extra attention is needed label May 8, 2023
@paulo-ferraz-oliveira paulo-ferraz-oliveira force-pushed the feature/otp-26-on-windows branch 22 times, most recently from 683e919 to 694fcf4 Compare May 9, 2023 21:51
@paulo-ferraz-oliveira paulo-ferraz-oliveira force-pushed the feature/otp-26-on-windows branch 9 times, most recently from 4818cec to ea9183d Compare May 23, 2023 09:47
@paulo-ferraz-oliveira paulo-ferraz-oliveira removed the help wanted Extra attention is needed label May 23, 2023
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I separate cmd and args everywhere to make it easier to spot.

I also separate options to its own variable for similar reasons (and because I wanted to experiment with the differences between Linux and Windows)...

I add a few new debugging options for our own future comfort.

@paulo-ferraz-oliveira
Copy link
Collaborator Author

Now that the action is reported to work on top of Elixir 1.14.5 for Windows, I'm moving this away from draft (hopefully the tests will also show it working properly).

@paulo-ferraz-oliveira
Copy link
Collaborator Author

Force pushed to resolve conflict. This'll probably not move the feature further than that, though.

@paulo-ferraz-oliveira paulo-ferraz-oliveira changed the title Have Erlang/OTP 26 + Elixir 1.14 be usable in Windows Test Erlang/OTP 26 + Elixir 1.14 on top Windows Jun 23, 2023
Copy link
Member

@starbelly starbelly left a comment

Choose a reason for hiding this comment

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

Beautiful!

@starbelly starbelly merged commit 145c80c into erlef:main Jun 24, 2023
64 checks passed
@paulo-ferraz-oliveira paulo-ferraz-oliveira deleted the feature/otp-26-on-windows branch June 24, 2023 21:12
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.

Elixir installation on OTP26 hangs on Windows
3 participants