On Oct 5, 2011, at 2:30 PM, Barrett, Brian W wrote:

> I don't think we need to go that far; in fact, we really shouldn't use m4
> macros to enforce license policies like that.

I'm not talking about enforcement -- I'm talking about notification.

> But more importantly, we should remove that particular warning from this
> test, since the test is used in places other than SLURM, which don't have
> negative licensing impact.

Fair enough; is there a way to tell the difference between BSD-friendly PMI and 
not-BSD-friendly PMI?


> Brian
> 
> On 10/5/11 11:25 AM, "Jeff Squyres" <jsquy...@cisco.com> wrote:
> 
>> It seems like this should be a field in ompi_info output, too...
>> 
>> Maybe a component can effect a "license" m4 variable, or somesuch...?
>> 
>> 
>> On Oct 5, 2011, at 1:14 PM, hje...@osl.iu.edu wrote:
>> 
>>> Author: hjelmn
>>> Date: 2011-10-05 13:14:24 EDT (Wed, 05 Oct 2011)
>>> New Revision: 25234
>>> URL: https://svn.open-mpi.org/trac/ompi/changeset/25234
>>> 
>>> Log:
>>> first cut at general pmi check
>>> Added:
>>>  trunk/orte/config/orte_check_pmi.m4
>>> 
>>> Added: trunk/orte/config/orte_check_pmi.m4
>>> 
>>> =========================================================================
>>> =====
>>> --- (empty file)
>>> +++ trunk/orte/config/orte_check_pmi.m4    2011-10-05 13:14:24 EDT
>>> (Wed, 05 Oct 2011)
>>> @@ -0,0 +1,59 @@
>>> +# -*- shell-script -*-
>>> +#
>>> +# Copyright (c) 2004-2005 The Trustees of Indiana University and
>>> Indiana
>>> +#                         University Research and Technology
>>> +#                         Corporation.  All rights reserved.
>>> +# Copyright (c) 2004-2005 The University of Tennessee and The
>>> University
>>> +#                         of Tennessee Research Foundation.  All rights
>>> +#                         reserved.
>>> +# Copyright (c) 2004-2005 High Performance Computing Center Stuttgart,
>>> +#                         University of Stuttgart.  All rights
>>> reserved.
>>> +# Copyright (c) 2004-2005 The Regents of the University of California.
>>> +#                         All rights reserved.
>>> +# Copyright (c) 2009      Cisco Systems, Inc.  All rights reserved.
>>> +# Copyright (c) 2011      Los Alamos National Security, LLC. All rights
>>> +#                         reserved.
>>> +# $COPYRIGHT$
>>> +# 
>>> +# Additional copyrights may follow
>>> +# 
>>> +# $HEADER$
>>> +#
>>> +
>>> +# ORTE_CHECK_PMI(prefix, [action-if-found], [action-if-not-found])
>>> +# --------------------------------------------------------
>>> +AC_DEFUN([ORTE_CHECK_PMI],[
>>> +    AC_ARG_WITH([pmi],
>>> +                    [AC_HELP_STRING([--with-pmi],
>>> +                            [Build PMI support (default: no)])],
>>> +            [], with_pmi=no)
>>> +    orte_enable_pmi=0
>>> +
>>> +    AC_MSG_CHECKING([if user requested PMI support])
>>> +    AS_IF([test "$with_pmi" = "no"], [
>>> +        AC_MSG_RESULT([no])
>>> +        orte_want_pmi_support=no],[
>>> +        AC_MSG_RESULT([yes])
>>> +        AS_IF([test ! -z "$with_pmi" -a "$with_pmi" != "yes"],[
>>> +            AS_IF([test -d "$with_pmi/lib64"], LDFLAGS="$LDFLAGS
>>> -L$with_pmi/lib64",
>>> +                LDFLAGS="$LDFLAGS -L$with_pmi/lib")
>>> +            CFLAGS="$CFLAGS -I$with_pmi/include"],[])
>>> +
>>> +        orte_want_pmi_support=yes
>>> +        orte_have_pmi_support=no
>>> +        AC_MSG_CHECKING([if PMI support installed])
>>> +        AC_CHECK_HEADERS([pmi.h
>>> slurm/pmi.h],[orte_have_pmi_support=yes]
>>> [break],[],[AC_INCLUDES_DEFAULT])
>>> +        AC_CHECK_LIB([pmi],[PMI_Init],[],orte_have_pmi_support=no)
>>> +            AS_IF([test "$orte_have_pmi_support" = "yes"], [
>>> +            AC_MSG_RESULT([yes])
>>> +            AC_MSG_WARN([PMI SUPPORT HAS BEEN INCLUDED - RESULTING])
>>> +            AC_MSG_WARN([BINARIES ARE SUBJECT TO ADDITIONAL LICENSING])
>>> +            AC_MSG_WARN([RESTRICTIONS - SEE THE SLURM LICENSE FOR
>>> INFO])
>>> +            orte_enable_pmi=1] [$2],[
>>> +            AC_MSG_RESULT([no])
>>> +            AC_MSG_WARN([PMI support requested (via --with-pmi) but
>>> not found.])
>>> +            AC_MSG_ERROR([Aborting.])] [$3])])
>>> +   AC_DEFINE_UNQUOTED([WANT_PMI_SUPPORT],
>>> +                      [$orte_enable_pmi],
>>> +                      [Whether we want PMI support])
>>> +])
>>> _______________________________________________
>>> svn-full mailing list
>>> svn-f...@open-mpi.org
>>> http://www.open-mpi.org/mailman/listinfo.cgi/svn-full
>> 
>> 
>> -- 
>> Jeff Squyres
>> jsquy...@cisco.com
>> For corporate legal information go to:
>> http://www.cisco.com/web/about/doing_business/legal/cri/
>> 
>> 
>> _______________________________________________
>> devel mailing list
>> de...@open-mpi.org
>> http://www.open-mpi.org/mailman/listinfo.cgi/devel
>> 
>> 
> 
> 
> -- 
>  Brian W. Barrett
>  Dept. 1423: Scalable System Software
>  Sandia National Laboratories
> 
> 
> 
> 
> 
> 
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/devel


-- 
Jeff Squyres
jsquy...@cisco.com
For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/


Reply via email to