Page MenuHomeFreeBSD

lang/elixir: Update to 1.8.1
Needs ReviewPublic

Authored by feld on Feb 4 2019, 11:18 PM.
Tags
None
Referenced Files
F103384706: D19080.id53592.diff
Sun, Nov 24, 8:06 AM
Unknown Object (File)
Fri, Nov 22, 10:45 PM
Unknown Object (File)
Wed, Nov 20, 5:35 AM
Unknown Object (File)
Wed, Oct 30, 7:47 AM
Unknown Object (File)
Wed, Oct 30, 7:47 AM
Unknown Object (File)
Wed, Oct 30, 7:47 AM
Unknown Object (File)
Wed, Oct 30, 7:47 AM
Unknown Object (File)
Wed, Oct 30, 7:47 AM
This revision needs review, but all reviewers have resigned.

Details

Reviewers
dch
Summary

Updates elixir to 1.8.1

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 22422
Build 21580: arc lint + arc unit

Event Timeline

Revert erlang dependency to otp 19

I want to update this to Elixir 1.8.1 but it now requires OTP 20

===>  Building for elixir-1.8.1
gmake[1]: Entering directory '/wrkdirs/usr/ports/lang/elixir/work/elixir-1.8.1'
At least Erlang/OTP 20.0 is required to build Elixir

Update erlang as required

feld retitled this revision from lang/elixir: Update to 1.8.0 to lang/elixir: Update to 1.8.1.Feb 10 2019, 11:41 PM
feld edited the summary of this revision. (Show Details)

Remove hacks required because I overlooked the Makefile patch

submitted exp-run https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=235733 for removing the older OTP versions. Once that's back I'll do one to bump default OTP to 21.2.5.

@feld what about using OTP21_RUN_DEPENDS=${LOCALBASE}/lib/erlang21/bin/erl:lang/erlang-runtime21 and leaving the OTP bump alone in this patch? It would also require a small shim for each of:

/usr/local/lib/elixir/bin/elixir
/usr/local/lib/elixir/bin/elixirc
/usr/local/lib/elixir/bin/iex
/usr/local/lib/elixir/bin/mix
lang/elixir/Makefile
6

this & distinfo need to be based off 1.7.4 for the diff to apply to ports tree:

commit d8c9fa0228450a21b136754afcd69880c8d84555
Author: olgeni <olgeni@FreeBSD.org>
Date: Fri Oct 26 13:22:17 2018 +0000

Update lang/elixir to version 1.7.4.

Notes:

svn path=/head/; revision=483030
dch requested changes to this revision.Feb 14 2019, 9:19 AM

@feld here's https://git.sr.ht/~dch/ports/commit/0af5618836ea22f68f8c4a0c183dee08c042375b alternative implementation using OTP21 via shim, much tidier IMO. I am not clear how to update LOCALBASE within the patch file though, what do you suggest for doing that rather than having it hardwired?

This revision now requires changes to proceed.Feb 14 2019, 9:19 AM

dropped as we have D19911 & D19912 which updates the erlang dependency as well

This revision now requires review to proceed.May 27 2019, 12:41 PM