Bug 209887 - textproc/libucl: Fails to ./configure when multiple lua versions are installed
Summary: textproc/libucl: Fails to ./configure when multiple lua versions are installed
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: Kubilay Kocak
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-31 13:04 UTC by Jimmy Olgeni
Modified: 2016-07-20 12:11 UTC (History)
1 user (show)

See Also:
koobs: maintainer-feedback+
koobs: merge-quarterly-

Attachments
Build log (7.38 KB, text/x-log)
2016-05-31 13:04 UTC, Jimmy Olgeni 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.