Skip to content

[openmpi/vcpkg_build_make] Disable parallel build #12975

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Aug 20, 2020
Merged

[openmpi/vcpkg_build_make] Disable parallel build #12975

merged 3 commits into from
Aug 20, 2020

Conversation

JackBoosY
Copy link
Contributor

When building openmpi:x64-osx, the compiler report error:

Can't open profile/psizeof_f08.f90 for writing at ../../../.././../src/openmpi-4-d5305a6659.clean/ompi/mpi/fortran/base/gen-mpi-sizeof.pl line 189.
make[2]: *** [profile/psizeof_f08.f90] Error 2
make[2]: *** Waiting for unfinished jobs....
mkdir: profile: File exists
make[1]: *** [all-recursive] Error 1
make: *** [all-recursive] Error 1

It should be caused by parallel build, so add a parameter to disable parallel build.

Related: #12738.

@JackBoosY JackBoosY added category:port-bug The issue is with a library, which is something the port should already support info:internal category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly labels Aug 18, 2020
@JackBoosY
Copy link
Contributor Author

cc @Neumann-A

@Neumann-A
Copy link
Contributor

Why am I getting CCed? I dont deal with those kind of MPI stuff in general.
Since this is a known issue in different ports the fix seems to be ok.

@JackBoosY
Copy link
Contributor Author

@Neumann-A Because you are one of the vcpkg_build_make authors.

@JackBoosY JackBoosY requested a review from PhoebeHui August 19, 2020 02:21
@JackBoosY JackBoosY changed the title [openmpi] Disable parallel build [openmpi/vcpkg_build_make] Disable parallel build Aug 19, 2020
@PhoebeHui PhoebeHui added the info:reviewed Pull Request changes follow basic guidelines label Aug 19, 2020
@JackBoosY JackBoosY removed the info:reviewed Pull Request changes follow basic guidelines label Aug 19, 2020
@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label Aug 20, 2020
@BillyONeal BillyONeal merged commit ec4e0c3 into microsoft:master Aug 20, 2020
@BillyONeal
Copy link
Member

:( It makes me sad that this is needed. Thanks!

@JackBoosY JackBoosY deleted the dev/jack/fix_openmpi_build branch August 20, 2020 05:47
remz1337 pushed a commit to remz1337/vcpkg that referenced this pull request Aug 23, 2020
@JackBoosY @remz1337
[openmpi/vcpkg_build_make] Disable parallel build (microsoft#12975)
c3f078e
@moritz-h moritz-h mentioned this pull request Jan 16, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PhoebeHui PhoebeHui PhoebeHui approved these changes

Assignees

@PhoebeHui PhoebeHui

Labels
category:port-bug The issue is with a library, which is something the port should already support category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@JackBoosY @Neumann-A @BillyONeal @PhoebeHui