On 2018-03-22 01:34 +0000, Wookey wrote:
So there is no reason not to use standard flag variable names thoughout and skip the MASTER->local copying. I've done this, using these defaults: CFLAGS := $(CPPFLAGS) -c -Wall -DLINUX -Wno-switch -fpic CXXFLAGS := $(CPPFLAGS) -c -Wall -DLINUX -Wno-switch -fpic -std=c++11 LDFLAGS := -Wl,-z,defs ARFLAGS := rcs
The attached patch makes these changes, allowing extra options to be passed in from outside. Index: libopencsd-0.8.1/decoder/build/linux/ref_trace_decode_lib/makefile
This patch had a stanza missing in this file, which left the old unusued flags in the file.
@@ -34,10 +34,6 @@ CXX := $(MASTER_CXX) LINKER := $(MASTER_LINKER) LIB := $(MASTER_LIB)
-CXX_FLAGS := $(MASTER_CXX_FLAGS) -fpic -Wno-switch -LIB_FLAGS := $(MASTER_LIB_FLAGS) -LINKER_FLAGS := $(MASTER_LINKER_FLAGS) -shared - LIB_NAME= lib$(LIB_BASE_NAME)
BUILD_DIR=./$(PLAT_DIR)
Full patch attached.
Wookey