Commit 127915c4 authored by Greg Williams's avatar Greg Williams Committed by Scott Vokes
Browse files

Fixed Makefile dependency on JSON-C lib causing unnecessary rebuild. 

Disabled secure erase of drive in P2P system test since it fails with permission issues due to HW retaining erase PIN through erase
parent b0b528dc

Makefile

+15 −14
Original line number Diff line number Diff line
@@ -46,6 +46,7 @@ VERSION_FILE = ./config/VERSION 
VERSION = ${shell head -n1 $(VERSION_FILE)}

THREADPOOL_PATH = ${LIB_DIR}/threadpool

BUS_PATH = ${LIB_DIR}/bus

JSONC_LIB = ${OUT_DIR}/libjson-c.a



KINETIC_LIB_NAME = $(PROJECT).$(VERSION)

KINETIC_LIB = $(BIN_DIR)/lib$(KINETIC_LIB_NAME).a
@@ -103,12 +104,12 @@ LIB_OBJS = \ 
KINETIC_LIB_OTHER_DEPS = Makefile Rakefile $(VERSION_FILE)





default: makedirs json $(KINETIC_LIB)

default: makedirs $(KINETIC_LIB)



makedirs:

	@echo; mkdir -p ./bin/examples &> /dev/null; mkdir -p ./bin/unit &> /dev/null; mkdir -p ./bin/systest &> /dev/null; mkdir -p ./out &> /dev/null



all: default json test system_tests test_internals run examples

all: default test system_tests test_internals run examples



clean: makedirs

	rm -rf ./bin/*.a ./bin/*.so ./bin/kinetic-c-util $(DISCOVERY_UTIL_EXEC)
@@ -186,11 +187,11 @@ apply_license: 
# json-c

#-------------------------------------------------------------------------------



json: ${OUT_DIR}/libjson-c.a

json: ${JSONC_LIB}



$(OUT_DIR)/kinetic_acl.o: json

$(OUT_DIR)/kinetic_acl.o: ${JSONC_LIB}



json_install: json

json_install: ${JSONC_LIB}

	cd ${JSONC} && \

	make install
@@ -208,8 +209,8 @@ ${JSONC}/.libs/libjson-c.a: ${JSONC}/Makefile 
	cd ${JSONC} && \

	make libjson-c.la



${OUT_DIR}/libjson-c.a: ${JSONC}/.libs/libjson-c.a

	cp ${JSONC}/.libs/libjson-c.a ${OUT_DIR}/libjson-c.a

${JSONC_LIB}: ${JSONC}/.libs/libjson-c.a

	cp ${JSONC}/.libs/libjson-c.a ${JSONC_LIB}





#-------------------------------------------------------------------------------
@@ -254,7 +255,7 @@ ${OUT_DIR}/libthreadpool.a: ${LIB_DIR}/threadpool/*.[ch] 
KINETIC_SO_DEV = $(BIN_DIR)/lib$(KINETIC_LIB_NAME).so

KINETIC_SO_RELEASE = $(PREFIX)/lib$(KINETIC_LIB_NAME).so



$(KINETIC_LIB): $(LIB_OBJS) $(KINETIC_LIB_OTHER_DEPS) json

$(KINETIC_LIB): $(LIB_OBJS) $(KINETIC_LIB_OTHER_DEPS) $(JSONC_LIB)

	@echo

	@echo --------------------------------------------------------------------------------

	@echo Building static library: $(KINETIC_LIB)
@@ -262,7 +263,7 @@ $(KINETIC_LIB): $(LIB_OBJS) $(KINETIC_LIB_OTHER_DEPS) json 
	ar -rcs $@ $(LIB_OBJS)

	ar -t $@



$(KINETIC_SO_DEV): $(LIB_OBJS) $(KINETIC_LIB_OTHER_DEPS)

$(KINETIC_SO_DEV): $(LIB_OBJS) $(KINETIC_LIB_OTHER_DEPS) $(JSONC_LIB)

	@echo

	@echo --------------------------------------------------------------------------------

	@echo Building dynamic library: $(KINETIC_SO_DEV)
@@ -277,7 +278,7 @@ $(KINETIC_SO_DEV): $(LIB_OBJS) $(KINETIC_LIB_OTHER_DEPS) 
INSTALL ?= install

RM ?= rm



install: json json_install $(KINETIC_LIB) $(KINETIC_SO_DEV)

install: ${JSONC_LIB} json_install $(KINETIC_LIB) $(KINETIC_SO_DEV)

	@echo

	@echo --------------------------------------------------------------------------------

	@echo Installing $(PROJECT) v$(VERSION) into $(PREFIX)
@@ -361,10 +362,10 @@ systest_names = $(patsubst $(SYSTEST_OUT)/run_%,%,$(systest_executables)) 
list_system_tests:

	echo $(systest_names)



$(SYSTEST_OUT)/%_runner.c: $(SYSTEST_SRC)/%.c $(KINETIC_LIB)

$(SYSTEST_OUT)/%_runner.c: $(SYSTEST_SRC)/%.c

	./test/support/generate_test_runner.sh $< > $@



$(SYSTEST_OUT)/run_%: $(SYSTEST_SRC)/%.c $(SYSTEST_OUT)/%_runner.c

$(SYSTEST_OUT)/run_%: $(SYSTEST_SRC)/%.c $(SYSTEST_OUT)/%_runner.c $(KINETIC_LIB)

	@echo

	@echo ================================================================================

	@echo System test: '$<'
@@ -376,7 +377,7 @@ $(SYSTEST_OUT)/%.testpass : $(SYSTEST_OUT)/run_% 


$(systest_names) : % : $(SYSTEST_OUT)/%.testpass



system_tests: $(systest_passfiles) $(KINETIC_LIB)

system_tests: $(systest_passfiles)





#===============================================================================
@@ -417,7 +418,7 @@ DISCOVERY_UTIL_LDFLAGS += -lm -lssl $(KINETIC_LIB) -L${OUT_DIR} -lcrypto -lpthr 
$(OUT_DIR)/discovery.o: $(DISCOVERY_UTIL_DIR)/discovery.c

	$(CC) -c -o $@ $< $(CFLAGS) -I$(PUB_INC) -I$(DISCOVERY_UTIL_DIR) $(LIB_INCS)



$(DISCOVERY_UTIL_EXEC): $(DISCOVERY_UTIL_OBJ) $(KINETIC_LIB) json

$(DISCOVERY_UTIL_EXEC): $(DISCOVERY_UTIL_OBJ) $(KINETIC_LIB) $(JSONC_LIB)

	@echo

	@echo --------------------------------------------------------------------------------

	@echo Building service discovery utility: $(DISCOVERY_UTIL_EXEC)

test/unit/test_bus_listener_cmd.c

+0 −2
Original line number Diff line number Diff line
@@ -351,8 +351,6 @@ void test_listener_ListenerCmd_CheckIncomingMessages_should_handle_incoming_EXPE 
        .u.expect.box = box,

    };



    void *opaque_result = (void *)((uintptr_t)-25);



    rx_info_t hold_info = {

        .state = RIS_HOLD,

        .timeout_sec = 9,