Open MPI logo

Open MPI Development Mailing List Archives

  |   Home   |   Support   |   FAQ   |  

This web mail archive is frozen.

This page is part of a frozen web archive of this mailing list.

You can still navigate around this archive, but know that no new mails have been added to it since July of 2016.

Click here to be taken to the new web archives of this list; it includes all the mails that are in this frozen archive plus all new mails that have been sent to the list since it was migrated to the new archives.

Subject: Re: [OMPI devel] RFC: enable the use of source in platform files
From: Nathan Hjelm (hjelmn_at_[hidden])
Date: 2012-07-09 17:25:57


LANL's platform file's (for a particular platform) are 90% the same. By allowing a source directive those settings can be kept in a common file then sourced by the platform file. When something changes I wouldn't have to change a dozen files only a couple.

-Nathan

On Mon, Jul 09, 2012 at 02:21:28PM -0700, Ralph Castain wrote:
> I'm confused - how does this help maintain a platform file???
>
>
> On Jul 9, 2012, at 2:09 PM, Nathan Hjelm wrote:
>
> > When: Thurs, Jul 12, 5 PM MDT
> >
> > Why: Useful feature. Enabling source may cut down on the maintenance required to keep platform files up to date.
> >
> > How: Change directories to the platform file's directory before sourcing it (not after).
> >
> > diff --git a/config/ompi_load_platform.m4 b/config/ompi_load_platform.m4
> > index 7231f83..250e7d8 100644
> > --- a/config/ompi_load_platform.m4
> > +++ b/config/ompi_load_platform.m4
> > @@ -50,7 +50,18 @@ AC_DEFUN([OMPI_LOAD_PLATFORM], [
> > # eval into environment
> > OPAL_LOG_MSG([Loading environment file $with_platform, with contents below])
> > OPAL_LOG_FILE([$with_platform])
> > - . "$with_platform"
> > +
> > + # setup by getting full pathname for the platform directories
> > + platform_base="`dirname $with_platform`"
> > + platform_file="`basename $with_platform`"
> > + # get full pathname of where we are so we can return
> > + platform_savedir="`pwd`"
> > + # go to where the platform file is located
> > + cd "$platform_base"
> > + # get the full path to this location
> > + platform_file_dir=`pwd`
> > +
> > + . "$platform_file"
> >
> > # see if they left us a name
> > if test "$OMPI_PLATFORM_LOADED" != "" ; then
> > @@ -63,14 +74,6 @@ AC_DEFUN([OMPI_LOAD_PLATFORM], [
> >
> > # look for default mca param file
> >
> > - # setup by getting full pathname for the platform directories
> > - platform_base="`dirname $with_platform`"
> > - # get full pathname of where we are so we can return
> > - platform_savedir="`pwd`"
> > - # go to where the platform file is located
> > - cd "$platform_base"
> > - # get the full path to this location
> > - platform_file_dir=`pwd`
> > # return to where we started
> > cd "$platform_savedir"
> >
> >
> > -Nathan
> > _______________________________________________
> > devel mailing list
> > devel_at_[hidden]
> > http://www.open-mpi.org/mailman/listinfo.cgi/devel
>
>
> _______________________________________________
> devel mailing list
> devel_at_[hidden]
> http://www.open-mpi.org/mailman/listinfo.cgi/devel