Bug 223632

Summary: editors/vim: Lua support is OFF despite LUA option being ON
Product: Ports & Packages Reporter: Dmitri Goutnik <dmgk>
Component: Individual Port(s)Assignee: Adam Weinberger <adamw>
Status: Closed FIXED    
Severity: Affects Only Me Flags: bugzilla: maintainer-feedback? (adamw)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
make showconfig
none
vim --version none

Description Dmitri Goutnik freebsd_committer freebsd_triage 2017-11-12 16:59:45 UTC
make showconfig:

...
====> Optional language bindings
     LUA=on: Lua scripting language support
     PERL=on: Perl scripting language support
     PYTHON2=on: Python 2.x bindings or support
     PYTHON3=on: Python 3.x bindings or support
     RUBY=on: Ruby bindings or support
     SCHEME=off: MzScheme (Racket) bindings
     TCL=off: Tcl scripting language support
...

vim --version:

...
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Nov 12 2017 11:47:50)
Included patches: 1-1289
Compiled by dg@xombo.localdomain
Huge version with GTK2 GUI.  Features included (+) or not (-):
...
+cursorshape     +localmap        +reltime         +wildmenu
+dialog_con_gui  -lua             +rightleft       +windows
+diff            +menu            +ruby            +writebackup
...

Full outputs of "make showconfig" and "vim --version" attached.
Comment 1 Dmitri Goutnik freebsd_committer freebsd_triage 2017-11-12 17:00:18 UTC
Created attachment 187943 [details]
make showconfig
Comment 2 Dmitri Goutnik freebsd_committer freebsd_triage 2017-11-12 17:00:45 UTC
Created attachment 187944 [details]
vim --version
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-11-12 21:43:40 UTC
A commit references this bug:

Author: adamw
Date: Sun Nov 12 21:43:28 UTC 2017
New revision: 454086
URL: https://svnweb.freebsd.org/changeset/ports/454086

Log:
  Fix lua support

  Vim's configure system is completely unable to handle our version-specific
  lua library. This is a really ugly fix. If anybody has a more elegant
  solution I'd be happy to replace this.

  PR:		223632
  Reported by:	Dmitri Goutnik

Changes:
  head/editors/vim/Makefile
Comment 4 Adam Weinberger freebsd_committer freebsd_triage 2017-11-12 21:44:41 UTC
Well, it should be fixed now, but I'm not at all proud of the solution. If you find a better one I'm all ears.