This issues: https://github.com/pmix/pmix/issues/1082

Will be solved with next release of 3.1.1.

Also note that there are an other patch missing in the 3.1.1 that
is also needed ( https://github.com/pmix/pmix/commit/7224a62d029c2e6225cc2ec08184ac9c185c767c )

/M

On 2019-01-22 16:04, Greg Wickham wrote:
I had some time this afternoon so dug into the source code (of slurm and pmix) and found the
issue.

In the file:

slurm-18.08.4/src/plugins/mpi/pmix/pmixp_client.c

line 147 (first instance):

PMIX_VAL_SET(&kvp->value, flag, 0);

“PMIX_VAL_SET” is a macro from /usr/include/pmix_common.h (version 2.2.1)

In version 3.1.1 it is missing.

Digging further it is pmix commit 47b8a8022a9d6cea8819c4365afd800b047c508e
(Sun Aug 12 11:27:28 2018 -0700) that removes the macros.

I don’t understand pmix enough to understand why the change was made.

   -Greg

On 22 Jan 2019, at 5:10 pm, Michael Di Domenico <mdidomeni...@gmail.com <mailto:mdidomeni...@gmail.com>> wrote:

i've seen the same error, i don't think it's you.  but i don't know
what the cause is either, i didn't have time to look into it so i
backed up to pmix 2.2.1 which seems to work fine

On Tue, Jan 22, 2019 at 12:56 AM Greg Wickham <greg.wick...@kaust.edu.sa <mailto:greg.wick...@kaust.edu.sa>> wrote:


Hi All,

I’m trying to build pmix 3.1.1 against slurm 18.08.4, however in the slurm
pmix plugin I get a fatal error:

       pmixp_client.c:147:28: error: ‘flag’ undeclared (first use in this function)
       PMIX_VAL_SET(&kvp->value, flag, 0);

Is there something wrong with my build environment?

The variable ‘flag’ doesn’t appear to be defined anywhere in any file in
the directory slurm-18.08.4/src/plugins/mpi/pmix

thanks,

  -greg

--


--
Dr. Greg Wickham
Advanced Computing Infrastructure Team Lead
Advanced Computing Core Laboratory
King Abdullah University of Science and Technology
Building #1, Office #0124
greg.wick...@kaust.edu.sa <mailto:greg.wick...@kaust.edu.sa> +966 544 700 330
--


--
Magnus Jonsson, Developer, HPC2N, Umeå Universitet

Reply via email to