search

tbuktu / libntru

C Implementation of NTRUEncrypt
Other
202 stars 59 forks source link

Make the "lib" target depend on the "libntru.so" target #12

Closed hasufell closed 10 years ago

hasufell commented 10 years ago

Depending on the old "lib" target caused a rebuild of libntru.so on any such target (e.g. installation), because "lib" is in .PHONY and not an expected file name.

Fixing this helps with distro policies which require to not do compilation processes during installation phase.

hasufell commented 10 years ago

I pushed a few more Makefile related commits. Tell me if you like them.

I can test the OSX Makefile for the static build too and add a similar target like in Makefile.linux, but I currently don't have access to such a machine (I happen to do exactly that: requiring a static libntru lib on an OSX machine).

hasufell commented 10 years ago

Hm, this currently breaks the tests.

edit: fixed.

tbuktu commented 10 years ago

Looks good, thanks. I have access to a Mac, so if you need anything tested, let me know.