Bug 209887

Summary: textproc/libucl: Fails to ./configure when multiple lua versions are installed
Product: Ports & Packages Reporter: Jimmy Olgeni <olgeni>
Component: Individual Port(s)Assignee: Kubilay Kocak <koobs>
Status: Closed FIXED    
Severity: Affects Only Me CC: olgeni
Priority: --- Flags: koobs: maintainer-feedback+
koobs: merge-quarterly-
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Build log none

Description Jimmy Olgeni freebsd_committer freebsd_triage 2016-05-31 13:04:45 UTC
Created attachment 170863 [details]
Build log

textproc/libucl fails to build when the LUA option is enabled and more than one Lua version is installed (for example lua51-5.1.5_9, lua52-5.2.4, lua53-5.3.2_1).

Poudriere builds only get lua52 and work fine.
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2016-06-30 08:45:53 UTC
Thank you for the report Jimmy. Any thoughts on the possible root cause(s) and fix?
Comment 2 commit-hook freebsd_committer freebsd_triage 2016-06-30 12:24:46 UTC
A commit references this bug:

Author: koobs
Date: Thu Jun 30 12:23:47 UTC 2016
New revision: 417823
URL: https://svnweb.freebsd.org/changeset/ports/417823

Log:
  textproc/libucl: Fix builds with multiple lua versions installed

  The LUA_{LIB,INC}DIR variables point to the default Lua version
  specified in ports (in this case 5.2) or by the user.

  When multiple versions of Lua are installed (in this case 5.2 and 5.3),
  libucl's ./configure auto-detects Lua 5.3 (or perhaps the latest version
  available), which is a mismatch against those in the provided
  include/library paths:

  checking for a Lua interpreter with version >= 5.1... lua53 <-- detected
  checking for lua53... /usr/local/bin/lua53
  checking for lua53 version... 5.3
  ...
  checking for Lua header version... 5.2 <-- Passed by LUA_INCDIR
  checking if Lua header version matches 5.3... no

  This change explicitly specifies the version of the Lua interpreter we
  want to use, which appears to sufficienty influence the behaviour of the
  ./configure macros.

  PR:		209887
  Reported by:	olgeni
  MFH:		2016Q2

Changes:
  head/textproc/libucl/Makefile
Comment 3 Jimmy Olgeni freebsd_committer freebsd_triage 2016-07-19 14:11:15 UTC
Got into 2016Q3 on 2016-07-01, so I'm closing this.