Hi,
attached patch to solve the link versus HDF5
as requested now by ov-galois.cc for platform like cygwin
that require explict declaration.

I presume there is a better way to define HDF5_LIBs to "-lhdf5"
than

HDF5_LIBS := $(shell grep "\#define OCTAVE_CONF_HDF5_LIBS" $(shell mkoctfile -p OCTINCLUDEDIR)/oct-conf.h | sed "s/^.*LIBS //" )

but I found no other hack.

to be tested on mingw and OsX

Regards
Marco
--- comm_old/src/Makefile       2008-03-27 16:45:25.000000000 +0100
+++ comm/src/Makefile   2012-02-03 15:40:26.784714000 +0100
@@ -1,5 +1,7 @@
 sinclude Makeconf
 
+HDF5_LIBS := $(shell  grep "\#define OCTAVE_CONF_HDF5_LIBS"  $(shell  
mkoctfile -p OCTINCLUDEDIR)/oct-conf.h | sed "s/^.*LIBS //"  )
+
 GALOISTARGET = gf.oct
 GALOISSOURCES = galois.cc galois-def.cc galoisfield.cc gf.cc \
              op-gm-gm.cc op-gm-m.cc op-gm-s.cc op-m-gm.cc op-s-gm.cc \
@@ -38,7 +40,7 @@
 
 $(GALOISTARGET) : $(DEPENDS) $(GALOISOBJECTS) 
        @echo "Linking $@"; \
-       $(MKOCTFILE) $(MOFLAGS) $(GALOISOBJECTS) -o $@
+       $(MKOCTFILE) $(MOFLAGS) $(GALOISOBJECTS) -o $@ $(HDF5_LIBS)
 
 ifneq (,$(DEPENDS))
   sinclude $(DEPENDS)
@@ -46,7 +48,7 @@
 
 %.oct : %.d %.o
        @echo "Linking $@"; \
-       $(MKOCTFILE) $(MOFLAGS) $(@:.oct=.o) -o $@
+       $(MKOCTFILE) $(MOFLAGS) $(@:.oct=.o) -o $@ $(HDF5_LIBS)
 
 %.d: %.cc
        @echo "Depending $<"; \
------------------------------------------------------------------------------
Try before you buy = See our experts in action!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-dev2
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to