Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1553#discussion_r186920564 --- Diff: core/sqf/src/seabed/src/Makefile --- @@ -55,6 +55,16 @@ LIBSBUTIL = $(LIBEXPDIR)/libsbutil.so LIBSQSTATESB = $(LIBEXPDIR)/libsqstatesb.so # neet -lrt for clock_gettime LIBSX += -lrt +# +# the order of these objects have to be swapped based on the GCC version +MS_MPITMSG_O = $(OUTDIR)/mpitmsg.o \ + $(OUTDIR)/ms.o +ifeq ($(GCC_MAJOR_VERSION_GT_3),1) +ifeq ($(GCC_MINOR_VERSION_GT_4),1) --- End diff -- You are right Dave, so it should be GCC_MAJOR_VERSION_EQ_4 and GCC_MINOR_VERSION_GT_4 We don't know how GCC 5.1 behave now, so let only check GCC 4.4 vs. GCC 4.8 at present. Let me change the logic more accurate That is, we only change the link order for GCC 4.x , where x>4
---