Re: [PATCH] fix exec/Makefile.am

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
thanks for your patches. Because it looks like most of the problems are hidden in the fact that we are not using libtool, we decided to migrate master / needle to libtool (Fabio is working on that), what should solve all problems you were hitting. It would be really nice if you could test them.

cs_queue.h shouldn't really be in TOTEM_SRC and we will remove that.

There is no need to patch generated files and change awk to gawk. configure takes AWK environment variable.Fabio is working on patch to make same possible with sed (so SED environment variable can be set to gsed).

One comment is included about this patch.

Now, just for curiosity, did you tried to run compiled version? Did it really work?

Thanks,
  Honza

Grüninger, Andreas (LGL Extern) napsal(a):
I compiled corosync 2.0.1 and master in Solaris 11 and OpenIndiana 151a.

The compilation environment and some additional patches are described in http://grueni.github.com/libqb/.
The additional patches are removing of compiler flags and changes from awk to gawk.

Andreas

diff --git a/exec/Makefile.am b/exec/Makefile.am
index 2f0adef..56d7acc 100644
--- a/exec/Makefile.am
+++ b/exec/Makefile.am
@@ -35,9 +35,16 @@ AM_CFLAGS            = -fPIC -DLOGCONFIG_USE_ICMAP=1

  INCLUDES               = -I$(top_builddir)/include -I$(top_srcdir)/include $(nss_CFLAGS) $(rdmacm_CFLAGS) $(ibverbs_CFLAGS)

-TOTEM_SRC              = totemip.c totemnet.c totemudp.c \
-                         totemudpu.c totemrrp.c totemsrp.c totemmrp.c \
-                         totempg.c cs_queue.h totemcrypto.c
+if BUILD_SOLARIS
+TOTEM_SRC               = totemip.c totemnet.c totemudp.c \
+                          totemudpu.c totemrrp.c totemsrp.c totemmrp.c \
+                          totempg.c            totemcrypto.c
+else
+TOTEM_SRC               = totemip.c totemnet.c totemudp.c \
+                          totemudpu.c totemrrp.c totemsrp.c totemmrp.c \
+                          totempg.c cs_queue.h totemcrypto.c
+endif
+
  if BUILD_RDMA
  TOTEM_SRC              += totemiba.c
  endif
@@ -47,11 +54,20 @@ sbin_PROGRAMS               = corosync

  libtotem_pg_a_SOURCES  = $(TOTEM_SRC)

-corosync_SOURCES       = vsf_ykd.c coroparse.c vsf_quorum.c sync.c \
-                         logsys.c cfg.c cmap.c cpg.c mon.c pload.c \
-                         votequorum.c wd.c util.c schedwrk.c main.c \
-                         apidef.c quorum.c icmap.c timer.c \
-                         ipc_glue.c service.c logconfig.c totemconfig.c
+if BUILD_SOLARIS
+corosync_SOURCES        = vsf_ykd.c coroparse.c vsf_quorum.c sync.c \
+                          logsys.c cfg.c cmap.c cpg.c mon.c pload.c \
+                          votequorum.c      util.c schedwrk.c main.c \
+                          apidef.c quorum.c icmap.c timer.c \
+                          ipc_glue.c service.c logconfig.c totemconfig.c
+else
+corosync_SOURCES        = vsf_ykd.c coroparse.c vsf_quorum.c sync.c \
+                          logsys.c cfg.c cmap.c cpg.c mon.c pload.c \
+                          votequorum.c wd.c util.c schedwrk.c main.c \
+                          apidef.c quorum.c icmap.c timer.c \
+                          ipc_glue.c service.c logconfig.c totemconfig.c
+endif
+

I don't understand this change. Why you decided to simply remove wd.c rather then make it compile on Solaris?

  corosync_LDADD         = -ltotem_pg -lcorosync_common $(LIBQB_LIBS) $(statgrab_LIBS)
  corosync_DEPENDENCIES  = libtotem_pg.so.$(SONAME) ../common_lib/libcorosync_common.so
  corosync_LDFLAGS       = $(OS_DYFLAGS) -L./ -L../common_lib
@@ -78,7 +94,7 @@ else

  if BUILD_SOLARIS
  libtotem_pg.so.$(SONAME): $(TOTEM_OBJS)
-       $(LD) $(LDFLAGS) -G $(TOTEM_OBJS) -o $@ -lpthread
+       $(LD) $(LDFLAGS) -G $(TOTEM_OBJS) -o $@ $(nss_LIBS) $(rdmacm_LIBS) $(ibverbs_LIBS) -lpthread
         ln -sf libtotem_pg.so.$(SONAME) libtotem_pg.so
         ln -sf libtotem_pg.so.$(SONAME) libtotem_pg.so.$(SOMAJOR)

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss



[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux