Ya, that's a bozo on my part.  I'll commit without the //.


On Jan 10, 2008, at 11:21 AM, Pavel Shamis (Pasha) wrote:

Here is reply from Oleg :

The Patch didn't work.
Broken code -  my $ret = join(/$delimiter/, @hosts); in
lib/MTT/Values/Functions.pm > sub hostlist_hosts
Must be $ret = join($delimiter, @hosts);



Ethan Mallove wrote:
Works for me.

-Ethan

On Thu, Jan/10/2008 08:48:20AM, Jeff Squyres wrote:

Mellanox told me that the MTT &hostlist() funclet is returning a
comma-delimited list of hosts (and &hostlist_hosts()). That is fine for Open MPI, but it is not for MVAPICH -- MVAPICH requires a space- delimited
list of hosts for their mpirun.

Here's a patch that introduces an optional parameter to &hostlist() and &hostlist_hosts(). The optional parameter is a delimiter for the hostlist.
So if you call:

        &hostlist_hosts()

you'll get the same comma-delimited list that is returned today. But if
you call

        &hostlist_hosts(" ")

you should get a space-delimited list.

Can Mellanox try this patch and see if it works for them? If so, I'll
commit it to the MTT trunk.

--
Jeff Squyres
Cisco Systems






_______________________________________________
mtt-users mailing list
mtt-us...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/mtt-users


_______________________________________________
mtt-users mailing list
mtt-us...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/mtt-users




--
Pavel Shamis (Pasha)
Mellanox Technologies

_______________________________________________
mtt-users mailing list
mtt-us...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/mtt-users


--
Jeff Squyres
Cisco Systems

Reply via email to