Re: [MTT devel] MTT on Windows

On Wed, Mar/11/2009 03:47:22PM, Jeff Squyres wrote: > Thanks for your patience! Yes, this looks good to me with one minor nit: > >> +if(($sys_type == "Cygwin" || $sys_type == "Msys") && >> +$config->{compiler_name} == "microsoft") { > > should be > >> +if(($sys_type eq "Cygwin" ||

Re: [MTT devel] MTT on Windows

Thanks for your patience! Yes, this looks good to me with one minor nit: +if(($sys_type == "Cygwin" || $sys_type == "Msys") && +$config->{compiler_name} == "microsoft") { should be +if(($sys_type eq "Cygwin" || $sys_type eq "Msys") && +$config->{compiler_name} eq

Re: [MTT devel] GSOC apps now open

I`ll help, lead us master. On Tue, Mar 10, 2009 at 6:01 PM, Josh Hursey wrote: > Yeah I have some time to dedicate do this. We should talk about a couple of > specific topics to propose from this list we posted on the wiki. > > I can start digging in later this

Re: [MTT devel] MTT on Windows

On Mar 11, 2009, at 7:32 AM, Shiqing Fan wrote: That's an old patch, Here attatched is the new one. Ah, excellent. > Can we think of a better test here? Will we always be running MTT > under Cygwin? On windows, there isn't any choice to run MTT in a linux-shell, another option is

Re: [MTT devel] MTT on Windows

Should this code rather be in a different .pm and the user selects which one to use via the .ini file? Yes, that's also what I thought. Then we need to change a little in upper level, e.g. OMPI.pm, we should test the environment and compiler, in order to choose the right install module

Re: [MTT devel] MTT on Windows

Hi Jeff, That's an old patch, Here attatched is the new one. I was just writing the email to be sent to MTT-devel, now I think it's better to put it in this thread. === --- lib/MTT/Common/GNU_Install.pm(revision 1271) +++

Re: [MTT devel] MTT on Windows

(moving to mtt-devel) Shiqing -- In looking at your patch, I see a few nits. See below. On Feb 26, 2009, at 12:56 PM, Shiqing Fan wrote: Index: lib/Filesys/DiskFree.pm === --- lib/Filesys/DiskFree.pm (revision 1271) +++