Like I said, I didn't write this code - all I can say for certain is that it gets the right answer on the LANL Crays. I'll talk to Nathan (the author) about it tomorrow.
On Jan 28, 2013, at 6:23 PM, Paul Hargrove <phhargr...@lbl.gov> wrote: > Ralph writes > ?? It looks correct to me - if with_alps is "yes", then no path was given and > we have to look at a default location. If it isn't yes, then a path was given > and we use it. > Am I missing something? > > Maybe *I* am the one missing something, but the way I read it the following > defaults are applied > > CLE4: > orte_check_alps_libdir="/usr/lib/alps" > orte_check_alps_dir="/opt/cray/alps/default" > CLE5: > orte_check_alps_libdir="/opt/cray/alps/default/lib64" > orte_check_alps_dir="/usr" > > Unless I am mistaken, the defaults for orte_check_alps_dir should be > exchanged to yield: > > CLE4: > orte_check_alps_libdir="/usr/lib/alps" > orte_check_alps_dir="/usr" > CLE5: > orte_check_alps_libdir="/opt/cray/alps/default/lib64" > orte_check_alps_dir="/opt/cray/alps/default" > > -Paul > > > On Mon, Jan 28, 2013 at 6:14 PM, Ralph Castain <r...@open-mpi.org> wrote: > > On Jan 28, 2013, at 6:10 PM, Paul Hargrove <phhargr...@lbl.gov> wrote: > >> The following 2 fragment from config/orte_check_alps.m4 appear to be >> contradictory. >> By that I mean the first appears to mean that "--with-alps" with no argument >> means /opt/cray/alps/default/... for CLE5 and /usr/... for CLE4, while the >> second fragment appears to be doing the opposite: >> >> if test "$using_cle5_install" = "yes"; then >> orte_check_alps_libdir="/opt/cray/alps/default/lib64" >> else >> orte_check_alps_libdir="/usr/lib/alps" >> fi >> >> >> if test "$using_cle5_install" = "yes" ; then >> AS_IF([test "$with_alps" = "yes"], >> [orte_check_alps_dir="/usr"], >> [orte_check_alps_dir="$with_alps"]) >> else >> AS_IF([test "$with_alps" = "yes"], >> [orte_check_alps_dir="/opt/cray/alps/default"], >> [orte_check_alps_dir="$with_alps"]) >> fi >> >> At least based on header and lib locations on NERSC's XC30 (CLE 5.0.15) and >> XE6 (CLE 4.1.40), the first fragment is correctwhile the second fragment is >> "backwards" (the two calls to AS_IF should be exchanged, or the initial >> "test" should be inverted). > > ?? It looks correct to me - if with_alps is "yes", then no path was given and > we have to look at a default location. If it isn't yes, then a path was given > and we use it. > > Am I missing something? > >> >> Note this same logic is present in both trunk and v1.7 (in SVN - I am not >> looking at tarballs this time). >> >> -Paul >> >> >> >> >> >> >> -- >> Paul H. Hargrove phhargr...@lbl.gov >> Future Technologies Group >> Computer and Data Sciences Department Tel: +1-510-495-2352 >> Lawrence Berkeley National Laboratory Fax: +1-510-486-6900 >> _______________________________________________ >> devel mailing list >> de...@open-mpi.org >> http://www.open-mpi.org/mailman/listinfo.cgi/devel > > > _______________________________________________ > devel mailing list > de...@open-mpi.org > http://www.open-mpi.org/mailman/listinfo.cgi/devel > > > > -- > Paul H. Hargrove phhargr...@lbl.gov > Future Technologies Group > Computer and Data Sciences Department Tel: +1-510-495-2352 > Lawrence Berkeley National Laboratory Fax: +1-510-486-6900 > _______________________________________________ > devel mailing list > de...@open-mpi.org > http://www.open-mpi.org/mailman/listinfo.cgi/devel
