Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the PMIx and PRRTE pointers #8145

Merged
merged 1 commit into from
Nov 4, 2020
Merged

Update the PMIx and PRRTE pointers #8145

merged 1 commit into from
Nov 4, 2020

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Oct 27, 2020

Signed-off-by: Ralph Castain rhc@pmix.org

@jsquyres
Copy link
Member

jsquyres commented Nov 2, 2020

Oops -- didn't mean to have the PMIx PR close this PR.

@jsquyres jsquyres reopened this Nov 2, 2020
@rhc54
Copy link
Contributor Author

rhc54 commented Nov 3, 2020

@jsquyres Not sure why this would be the only place to hit such an error:

  CC       libpmixglobal.lo
In file included from ../../../../../../3rd-party/openpmix/src/include/libpmixglobal.c:20:0:
/home/ubuntu/workspace/open-mpi.build.distcheck/src/openmpi-gitclone/_build/sub/3rd-party/openpmix/src/include/pmix_config.h:29:41: fatal error: src/include/pmix_config_top.h: No such file or directory
compilation terminated.

@rhc54
Copy link
Contributor Author

rhc54 commented Nov 3, 2020

Hmmm....got same error when AWS is trying to build the nightly PMIx tarball:

Starting build for master
Found new revision 2a0f0b7
Exceuting run-with-autotools.sh failed:
make  all-am
make[4]: Entering directory '/mnt/data/nightly-tarball/scratch/pmix/master-202011030225/pmix.git/pmix-master-202011030225-2a0f0b7/_build/sub/src/include'
 CC       libpmixglobal.lo
In file included from ../../../../src/include/libpmixglobal.c:20:0:
/mnt/data/nightly-tarball/scratch/pmix/master-202011030225/pmix.git/pmix-master-202011030225-2a0f0b7/_build/sub/src/include/pmix_config.h:29:41: fatal error: src/include/pmix_config_top.h: No such file or directory
compilation terminated.
Makefile:784: recipe for target 'libpmixglobal.lo' failed

@rhc54
Copy link
Contributor Author

rhc54 commented Nov 3, 2020

Seeing the following when doing "make_dist_tarball" for PMIx itself on the Mac:

make[3]: warning: -jN forced in submake: disabling jobserver mode.
  CC       pevent.o
/bin/sh: ../../../../../config/getdate.sh: No such file or directory
  CCLD     pevent

I also see that pmix_config_top.h is not in the build include tree, so that might be the issue. Again, though, it doesn't fail on my Mac. I suspect it is failing only on the one AWS platform because that is the only place that does "make distcheck".

@rhc54
Copy link
Contributor Author

rhc54 commented Nov 3, 2020

@jsquyres I'm getting reports of broken builds of PMIx due to variations in Python behavior across specific versions. I think we are going to have to remove dictionary.h from automatic generation and go back to generating it prior to release where we have more control of the environment.

In our efforts to fix the integration problem, we appear to have broken something in the PMIx build system. I suspect the best path forward is for me to first take us back on the dictionary generation, and then try to figure out what is going on with the srcdir issue for getdate as that is a separate problem. I'll tackle the dictionary part in the morning.

@rhc54
Copy link
Contributor Author

rhc54 commented Nov 4, 2020

@jsquyres Well, we are down to this error:

Traceback (most recent call last):
  File "../../../../../../3rd-party/openpmix/contrib/construct_dictionary.py", line 247, in main
    constants = open(outpath, "w+")
PermissionError: [Errno 13] Permission denied: '/home/ubuntu/workspace/open-mpi.build.distcheck/src/openmpi-gitclone/3rd-party/openpmix/include/dictionary.tmp'

Not sure how to get around that one! It is during "make distcheck".

Signed-off-by: Ralph Castain <rhc@pmix.org>
@rhc54 rhc54 merged commit 62a76c9 into open-mpi:master Nov 4, 2020
@rhc54 rhc54 deleted the topic/up branch November 4, 2020 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants