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

[20.09] elixir_1_6: remove #103971

Merged
merged 1 commit into from Nov 19, 2020

Conversation

DianaOlympos
Copy link
Contributor

Additionally removed the now obsolete rebar build dependency for elixir.

Motivation for this change

As noted by @cw789 in his original PR #98649, rebar does not compile anymore on otp 23.

We allowed otp 23 in nixos 20.09, which means that all compilation of elixir are broken sometimes, because they depend on rebar, which does not compile on otp 23. On top of that, rebar has problems with openssl in general.

I know this drop elixir 1.6, which technically could break people using it. Elixir should be fully backward compatible, so they should be able to update to a newer one. On top of this, 1.6 was the last one depending on rebar and cannot be compiled on otp23 in the current state anyway.

So or we break anyone using otp 23 with elixir, or we break people using elixir 1.6.

I understand if the decision is to keep the current (broken) situation. @ankhers what do you think ?

Things done

Remove elixir 1.6. remove elixir dependency on rebar

Additionally removed the now obsolete rebar build dependency for elixir.
@veprbl veprbl changed the title elixir_1_6: remove backport to 20.09 [20.09] elixir_1_6: remove Nov 19, 2020
@veprbl
Copy link
Member

veprbl commented Nov 19, 2020

Why not remove dependency on rebar without removing elixir_1_6? How presence of the latter attribute can break anything?

@DianaOlympos
Copy link
Contributor Author

@veprbl because 1.6 depends on rebar to work sadly.

@veprbl
Copy link
Member

veprbl commented Nov 19, 2020

@DianaOlympos Can we then remove it for all versions except for 1.6?

@veprbl veprbl changed the base branch from nixos-20.09 to release-20.09 November 19, 2020 15:27
@DianaOlympos
Copy link
Contributor Author

@veprbl probably but that is going to be painful. Also it mean that elixir 1.6 will not work on otp 23 and fail to compile, while we do ship otp 23 in 20.09. So that would still be a problem

@ofborg ofborg bot requested a review from couchemar November 19, 2020 15:38
@veprbl
Copy link
Member

veprbl commented Nov 19, 2020

@DianaOlympos you could mark it as broken for that particular combination. If you are sure that impact of removing 1.6 is minimal, we can merge this as is of course.

@DianaOlympos
Copy link
Contributor Author

@veprbl that is why i was @ankhers , he is a maintainer of this, so i think it is fair if he decide on this

@ankhers
Copy link
Contributor

ankhers commented Nov 19, 2020

It should be fine to remove it. Elixir 1.11 is the most recent version and there have been no commits to the 1.6 branch for over two years. Ecto currently also expects 1.8+, which I sometimes use as a litmus test for versions because a couple core Elixir folks work on it.

Copy link
Member

@veprbl veprbl left a comment

Choose a reason for hiding this comment

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

nixpkgs-review on NixOS (x86_64-linux):

5 packages built:
elixir elixir_1_7 elixir_1_8 elixir_1_9 rabbitmq-server

@veprbl veprbl merged commit f36cb61 into NixOS:release-20.09 Nov 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants