Re: pkg/44916: graphics/gnuplot build failure due to missing .include ../../lang/lua/buildlink3.mk

[Matthias Rampke <matthias.rampke%googlemail.com@localhost>]

The following reply was made to PR pkg/44916; it has been noted by GNATS.

From: Matthias Rampke <matthias.rampke%googlemail.com@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/44916: graphics/gnuplot build failure due to missing
 .include ../../lang/lua/buildlink3.mk
Date: Fri, 29 Apr 2011 10:37:07 +0200

 --4dba78b3_41a7c4c9_15af6
 Content-Type: text/plain; charset="utf-8"
 Content-Transfer-Encoding: 8bit
 Content-Disposition: inline
 
 Do you already have lua from pkgsrc?
 Yes, it was installed as a dependency of something else. 
 >  or DragonFly-2.11 have builtin lua?
 No. 
 > 
 >  Could you find why lua is detected on your environment?
 >  (config.log, below "checking for LUA").
 >  With pkg-config and -llua detection, both should be failed.
 pkg-config fails, but -llua is detected (because it's there, obviously). This 
apparently triggers the lua/TikZ terminal being built.
 
 On the other hand, lua.h is not found (I don't really understand why, it's at 
/usr/pkg/include/lua.h and /usr/pkg/include is in the -I paths.)
 
 see the full config.log here:
 http://pastebin.com/UDWHzwzH
 
 config.h has:
 
 /* Define if you want the lua/TikZ terminal. */
 #define HAVE_LUA 1
 
 /* Define to 1 if you have the <lua.h> header file. */
 /* #undef HAVE_LUA_H */
 
 
 so it tries to build the lua/TikZ terminal and fails.
 > 
 >  Frankly, adding
 >  CONFIGURE_ARGS+= --without-lua
 >  will avoid the issue though.
 I can confirm that this fixes the problem. How about a lua option? This way 
the lua/TikZ terminal is either in or out and we don't depend on whatever the 
configure script may or may not find on the system. I have attached a patch.
 
 Regards,
 M.
 
 
 
 --4dba78b3_41a7c4c9_15af6
 Content-Type: application/octet-stream
 Content-Transfer-Encoding: base64
 Content-Disposition: attachment; filename="gnuplot-lua.patch"
 
 ZGlmZiAtLWdpdCBhL2dyYXBoaWNzL2dudXBsb3Qvb3B0aW9ucy5tayBiL2dyYXBoaWNzL2dudXBs
 b3Qvb3B0aW9ucy5tawppbmRleCBmYzYwYTcyLi43Njk3ZGUyIDEwMDY0NAotLS0gYS9ncmFwaGlj
 cy9nbnVwbG90L29wdGlvbnMubWsKKysrIGIvZ3JhcGhpY3MvZ251cGxvdC9vcHRpb25zLm1rCkBA
 IC0xLDcgKzEsNyBAQAogIyAkTmV0QlNEOiBvcHRpb25zLm1rLHYgMS4yIDIwMTEvMDIvMDggMTM6
 MTM6NDEgYWRhbSBFeHAgJAogCiBQS0dfT1BUSU9OU19WQVI9CVBLR19PUFRJT05TLmdudXBsb3QK
 LVBLR19TVVBQT1JURURfT1BUSU9OUz0JZ2QgcGRmIHgxMQorUEtHX1NVUFBPUlRFRF9PUFRJT05T
 PQlnZCBsdWEgcGRmIHgxMQogUEtHX1NVR0dFU1RFRF9PUFRJT05TPQlnZCB4MTEKIAogLmluY2x1
 ZGUgIi4uLy4uL21rL2JzZC5vcHRpb25zLm1rIgpAQCAtMTIsNiArMTIsMTIgQEAgUExJU1RfVkFS
 Uys9CXgxMQogLmluY2x1ZGUgIi4uLy4uL2dyYXBoaWNzL2dkL2J1aWxkbGluazMubWsiCiAuZW5k
 aWYKIAorLmlmICFlbXB0eShQS0dfT1BUSU9OUzpNbHVhKQorLmluY2x1ZGUgIi4uLy4uL2xhbmcv
 bHVhL2J1aWxkbGluazMubWsiCisuZWxzZQorQ09ORklHVVJFX0FSR1MrPQktLXdpdGhvdXQtbHVh
 CisuZW5kaWYKKwogIyBQREYgb3V0cHV0IGlzIGFsc28gcHJvdmlkZWQgYnkgY2Fpcm8KIC5pZiAh
 ZW1wdHkoUEtHX09QVElPTlM6TXBkZikKIENPTkZJR1VSRV9BUkdTKz0JLS13aXRoLXBkZgo=
 
 --4dba78b3_41a7c4c9_15af6--