We noticed that there is an assert(fname) in both of the dl components:
  *
https://github.com/open-mpi/ompi/blob/master/opal/mca/dl/dlopen/dl_dlopen_module.c#L53
  *
https://github.com/open-mpi/ompi/blob/master/opal/mca/dl/libltdl/dl_libltdl_module.c#L21

But according to the dl.h, NULL should be an acceptable parameter for fname:
  https://github.com/open-mpi/ompi/blob/master/opal/mca/dl/dl.h#L67

For dlopen() it returns a handle to the program space, which can be handy.


I think it was just an oversight in the original code that has not been
encountered yet. But before I work on a patch, I wanted to check to see if
was a reason to put the assert in the code. git blame says that Jeff
Squyres put this code in there a while back, so I don't know if he
remembers.

Thanks,
Josh

Reply via email to