sparkman1 / jivelite

Automatically exported from code.google.com/p/jivelite
Other
0 stars 0 forks source link

Hard Coding of Location of lua.h #2

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. On Archlinux (or possibly other distros that place lua include files in 
/usr/include rather than /usr/local/include?) compile jivelite
2. Watch compile errors roll by as lua.h (and other lua includes) are not found.
3. profit?

What version of the product are you using? On what operating system?
I'm on Archlinux on the Raspberry Pi (3.6.11-17-ARCH+, 
http://archlinuxarm.org/).

I'd like to create a patch so that one could specify the location of the lua 
include files, if that would be acceptable.

Original issue reported on code.google.com by george.g...@gmail.com on 27 Aug 2013 at 2:53

GoogleCodeExporter commented 8 years ago
Triode, I've put together a patch that updates all of the makefiles (and config 
files) to use the PREFIX variable for the root location of include files.  I've 
created a default of PREFIX=/usr/local in the main Makefile that can be 
overridden when running make (i.e., make PREFIX=/usr when compiling on Arch 
Linux).  I have not tested this in its default setting since I don't have 
access to a distro that places the include files in /usr/local/include, but it 
does work for compiling on Arch where lua.h is located in 
/usr/include/luajit-2.0/  

Patch is attached.  I' hope this is OK.

George

Original comment by george.g...@gmail.com on 27 Aug 2013 at 4:21

GoogleCodeExporter commented 8 years ago
Triode:  The old patch file didn't seem to work, and with the addition of the 
visualizer (I believe) didn't cover all of the appropriate files.  I've created 
a new patch file that seems to work much better.  I believe this converts all 
of the appropriate files to using a predefined "PREFIX" for the location of the 
lua includes.  It defaults to /usr/local/ to maintain compatibility with the 
current set of files, but for those of us that use distros that put includes 
into /usr (Arch & Fedora), you can define PREFIX=/usr before making the binary.

Original comment by george.g...@gmail.com on 5 Nov 2013 at 2:19

Attachments: