Subject: Re: [MTT users] hostlist enhancement
From: Jeff Squyres (jsquyres_at_[hidden])
Date: 2008-01-10 11:26:23
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-users_at_[hidden]
>>> http://www.open-mpi.org/mailman/listinfo.cgi/mtt-users
>>>
>>
>> _______________________________________________
>> mtt-users mailing list
>> mtt-users_at_[hidden]
>> http://www.open-mpi.org/mailman/listinfo.cgi/mtt-users
>>
>>
>
>
> --
> Pavel Shamis (Pasha)
> Mellanox Technologies
>
> _______________________________________________
> mtt-users mailing list
> mtt-users_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/mtt-users
-- Jeff Squyres Cisco Systems