On 03/24/2015 04:07 PM, Vinod Koul wrote:
On Tue, Mar 24, 2015 at 10:32:27AM +0000, Qais Yousef wrote:
This series converts the build system to use autotools and splits the files into lib and utils.
Also patch 5 fixes error handling code.
I can't test Android builds but both cross and native compiling worked for me.
I left AUTHORS, ChangeLog and NEWS files empty. I used the license header to fill COPYING.
Changes in v2:
- Use -M option when generating the patches
- Add Makefile.am to include directory that exports tinycompress.h
- New patch (7) that fixes include dependency in tinycompress.h
This looks okay to me. I have two issue though:
- So after applying tha patches, I did ./autogen.sh, that seems to have
updated INSTALL, do you know why?
I'm no autotools expert. Either I'm passing wrong args to autoreconf or I need to replace it with something better. I'll have a look.
- For the fun part, I did make and then make install. Running cplay w/o
args should have printed the help menu, but that didnt ahppen
$ cplay cplay: error while loading shared libraries: libtinycompress.so.0: cannot open shared object file: No such file or directory
Not sure why...
I will try Android build possibly tomorrow
Weird. I can't reproduce this. Can you see libtinycompress.so.0 in /usr/local/lib?Do you need to update your LD_LIBRARY_PATH to include /usr/local/lib?
Thanks for having a look and testing.