Bug 236477 - lang/rubinius: switch to LLVM_DEFAULT
Summary: lang/rubinius: switch to LLVM_DEFAULT
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Ashish SHUKLA
URL:
Keywords:
Depends on:
Blocks: 231286
  Show dependency treegraph
 
Reported: 2019-03-11 21:55 UTC by Brooks Davis
Modified: 2019-03-20 21:41 UTC (History)
0 users

See Also:
ashish: maintainer-feedback+


Attachments
Switch to LLVM_DEFAULT (1.31 KB, patch)
2019-03-11 21:55 UTC, Brooks Davis
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brooks Davis freebsd_committer freebsd_triage 2019-03-11 21:55:51 UTC
Created attachment 202802 [details]
Switch to LLVM_DEFAULT

The attached patch switches rubinius to use LLVM_DEFAULT removing the defunct OPTIONS support for picking a version.

Note: this port is currently broken and this patch does not change that.  On 11.2-amd64 this fails to package.  On 12.0-amd64 it fails to configure due to the updated base OpenSSL.
Comment 1 Ashish SHUKLA freebsd_committer freebsd_triage 2019-03-18 18:05:36 UTC
(In reply to Brooks Davis from comment #0)

Sorry for the delay. I approve this change. You can go ahead and commit. I'll see about getting the port, or deprecate it, in next few days.

Thanks!
Comment 2 commit-hook freebsd_committer freebsd_triage 2019-03-20 21:40:14 UTC
A commit references this bug:

Author: brooks
Date: Wed Mar 20 21:40:03 UTC 2019
New revision: 496388
URL: https://svnweb.freebsd.org/changeset/ports/496388

Log:
  Switch to LLVM_DEFAULT from an hardcoded llvm40.

  This removes the LLVM40 option.

  PR:		236477, 231286
  Approved by:	ashish (maintainer)

Changes:
  head/lang/rubinius/Makefile
Comment 3 Brooks Davis freebsd_committer freebsd_triage 2019-03-20 21:41:41 UTC
Thanks for approving.