On 24/12/2015 06:10, Gilles Gouaillardet wrote:
Marco,

Thanks for the patch, i will apply the changes related to the missing
include files to master and PR to v2.x

on linux, libpmix.so does not depend on libopen-pal.
that being said, libpmix.so has undefined symbols related to hwloc and
libevent, and these symbols are defined in libopen-pal.so
i assume that is ok with linux but cygwin does not accept it.

circular dependency is hard to manage.

an other way to do things (and generally speaking, that could become the
way to go in a near future) is to use external hwloc and libevent (that
assumes of course cygwin provides these libraries and versions meet the
ompi requirements)

hwloc is already available as separate library, I will look to pack also libevent.

could you try to
configure --with-hwloc=external --with-libevent=external ...
and see if it helps ?

some time in the coming week

fwiw, with master you will soon be able to do the same thing with pmix
(e.g. configure --with-pmix=external ...)

I assume no one will really require PMI Exascale on cygwin.
So this should solve the issue anyway

Regards
Marco



Cheers,

Gilles

On 12/24/2015 3:42 AM, Marco Atzeri wrote:
building 2.0.0rc1 on cygwin I was stacked on
opal/mca/pmix/pmix1xx/pmix

Question:
- as pmix depends on hwloc and libevents, is it supposed to
  link against opal shared lib, is it supposed to be included
  in the opal shared lib or is it a circular dependency ?

I solved the build changing from shared to convenience library.

-lib_LTLIBRARIES = libpmix.la
+noinst_LTLIBRARIES = libpmix.la

as circular dependency will be a hell to manage with the
no-undefined flag.

The attached patch solves also

- several lacks of declaration for opal_output functions
  used in the ompi tree.
  They also require libmpi link vs lib@opal_lib_pre...@open-pal.la

- lack of declaration for several <unistd.h> functions
  like access, gethostname, usleep, ..

- add !defined(__CYGWIN__) on libevent2022/libevent/event.h
  as for current 1.8 and 1.10 code.


Please also note that usleep is marked as obsolescent on
Open Group Base Specifications Issue 6
http://pubs.opengroup.org/onlinepubs/009695399/functions/usleep.html
and it is missing on Issue7

With this patch it builds on cygwin 64 bit and passes the
same tests of 1.10.x code

Regards
Marco



_______________________________________________
devel mailing list
de...@open-mpi.org
Subscription:http://www.open-mpi.org/mailman/listinfo.cgi/devel
Link to this 
post:http://www.open-mpi.org/community/lists/devel/2015/12/18453.php



_______________________________________________
devel mailing list
de...@open-mpi.org
Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
Link to this post: 
http://www.open-mpi.org/community/lists/devel/2015/12/18456.php

Reply via email to