Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r26119

2012-03-08 Thread Josh Hursey
Thanks for the reminder. I forget about updating the news. Fixed in r26120. -- Josh On Thu, Mar 8, 2012 at 4:32 PM, Jeffrey Squyres wrote: > Can you add this to NEWS, please? > > Thanks. > > > > On Mar 8, 2012, at 5:02 PM, jjhur...@osl.iu.edu wrote: > > > Author: jjhursey > > Date: 2012-03-08 1

Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r26119

2012-03-08 Thread Jeffrey Squyres
Can you add this to NEWS, please? Thanks. On Mar 8, 2012, at 5:02 PM, jjhur...@osl.iu.edu wrote: > Author: jjhursey > Date: 2012-03-08 17:02:28 EST (Thu, 08 Mar 2012) > New Revision: 26119 > URL: https://svn.open-mpi.org/trac/ompi/changeset/26119 > > Log: > Create an MCA parameter (ess_base_s

Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r26118

2012-03-08 Thread Josh Hursey
Good point (I did not even look at ompi_comm_compare, I was using this for something else). I'll take a pass at converting the ompi_comm_compare to use the ompi_group_compare functionality - it is good code reuse. Thanks, Josh On Thu, Mar 8, 2012 at 4:08 PM, George Bosilca wrote: > Josh, > > Op

Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r26118

2012-03-08 Thread George Bosilca
Josh, Open MPI already have a similar function in the communicator part, function that is not exposed to the upper layer. I think that using the code in ompi_comm_compare (the second part that compare groups) is sound proof. Moreover, if now we have an ompi_group_compare function you should use