Re: [OMPI devel] v1.8 does not compile any more

2014-09-19 Thread Ralph Castain 
Quite so - thanks!

On Sep 19, 2014, at 1:52 AM, Gilles Gouaillardet 
 wrote:

> Folks,
> 
> r32716 broke v1.8 :-(
> 
> the root cause it it included MCA_BASE_VAR_TYPE_VERSION_STRING which has
> not yet landed into v1.8
> 
> the attached trivial patch fixes this issue
> 
> Can the RM/GK please review it and apply it ?
> 
> Cheers,
> 
> Gilles
> ___
> devel mailing list
> de...@open-mpi.org
> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
> Link to this post: 
> http://www.open-mpi.org/community/lists/devel/2014/09/15879.php

[OMPI devel] v1.8 does not compile any more

2014-09-19 Thread Gilles Gouaillardet 
Folks,

r32716 broke v1.8 :-(

the root cause it it included MCA_BASE_VAR_TYPE_VERSION_STRING which has
not yet landed into v1.8

the attached trivial patch fixes this issue

Can the RM/GK please review it and apply it ?

Cheers,

Gilles
Index: opal/mca/base/mca_base_var.c
===
--- opal/mca/base/mca_base_var.c(revision 32763)
+++ opal/mca/base/mca_base_var.c(working copy)
@@ -14,6 +14,8 @@
  * Copyright (c) 2012-2014 Los Alamos National Security, LLC. All rights
  * reserved.
  * Copyright (c) 2014  Intel, Inc. All rights reserved.
+ * Copyright (c) 2014  Research Organization for Information Science
+ * and Technology (RIST). All rights reserved.
  * $COPYRIGHT$
  * 
  * Additional copyrights may follow
@@ -1164,7 +1166,6 @@
 case MCA_BASE_VAR_TYPE_DOUBLE:
 align = OPAL_ALIGNMENT_DOUBLE;
 break;
-case MCA_BASE_VAR_TYPE_VERSION_STRING:
 case MCA_BASE_VAR_TYPE_STRING:
 default:
 align = 0;