summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorKota Tsuyuzaki <tsuyuzaki.kota@lab.ntt.co.jp>2017-02-20 14:12:43 -0800
committerKota Tsuyuzaki <tsuyuzaki.kota@lab.ntt.co.jp>2017-02-20 14:23:16 -0800
commit19442df2cd9a5bc8ec4deded7046ea7aca1d50a2 (patch)
treeb25eb75db42ec5ce848c9434222604729c0c2e39 /src
parentc9136a62b6e7cc8701cd7206ef0367520a20b8b9 (diff)
downloadliberasurecode-19442df2cd9a5bc8ec4deded7046ea7aca1d50a2.tar.gz
Fix warning on automake tool
This patch fixes automake tools warnings as follows (and a my fault in previous patch [1]): - Change INCLUDES variable to APP_CPP_FLAGS and APP_C_FLAGS according to the warning message - Remove non-POSIX variable to suppress that warnings Note that the latter change may be affected to the testing so please be careful on your review what's going on your testing environment. In the past change [2], they ware designed to use the shared object binaries in the build path. However, Daniel had pointed out the good thing at [1] if we runs the test/liberasruecode_test (it's shell with linker) in the dir, we can touch the expected binaries. My fault in the previous patch is that I didn't replace ./test/.libs/liberasurecode_test to ./test/liberasurecode_test even we use --trace-children=yes option for the valgrind. Now, all tests run via ./test/<test name> syntax and IMO no matters exist even if we remove the non-POSIX variable for test settings. 1: https://review.openstack.org/#/c/434162 2: 93446db9414f311bd9fc7dc047eb4dbbeb3e6feb Change-Id: I0e79bed7755a1f286b746a70fcf56fdc972bfd5d
Diffstat (limited to 'src')
-rw-r--r--src/Makefile.am15
1 files changed, 9 insertions, 6 deletions
diff --git a/src/Makefile.am b/src/Makefile.am
index eb2f89f..b043de8 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -2,12 +2,15 @@ SUBDIRS = builtin/xor_codes builtin/null_code builtin/rs_vand
lib_LTLIBRARIES = liberasurecode.la
-INCLUDES = \
- -I$(top_srcdir)/include/erasurecode \
- -I$(top_srcdir)/include/xor_codes \
- -I$(top_srcdir)/include/rs_vand \
- -I$(top_srcdir)/include/isa_l \
- -I$(top_srcdir)/include/shss
+INCLUDE = \
+ -I$(abs_top_srcdir)/include/erasurecode \
+ -I$(abs_top_srcdir)/include/xor_codes \
+ -I$(abs_top_srcdir)/include/rs_vand \
+ -I$(abs_top_srcdir)/include/isa_l \
+ -I$(abs_top_srcdir)/include/shss
+
+AM_CPPFLAGS = $(CPPFLAGS) $(INCLUDE)
+AM_CFLAGS = $(AM_CPPFLAGS)
# liberasurecode params
liberasurecode_la_SOURCES = \