pmix 5.0.2 #166807

BrewTestBot · 2024-03-21T12:56:22Z

Created by brew bump

Created with brew bump-formula-pr.

chenrui333 · 2024-03-21T16:06:46Z

brew test --retry --verbose boost-mpi
brew test --retry --verbose cp2k
brew test --retry --verbose mfem
brew test --retry --verbose mpi4py
brew test --retry --verbose open-mpi
brew test --retry --verbose opencoarrays
brew test --retry --verbose otf2

fxcoudert · 2024-03-21T17:08:55Z

  ld: warning: -commons use_dylibs is no longer supported, using error treatment instead
  ld: common symbol '_mpi_fortran_argv_null_' from '/private/tmp/cclH6ubZ.o' conflicts with definition from dylib '_mpi_fortran_argv_null_' from '/opt/homebrew/Cellar/open-mpi/5.0.2_1/lib/libmpi_usempi_ignore_tkr.40.dylib'

Apple linker regression in Xcode 15.3
Reported as FB13194355

Also reported to OpenMPI: open-mpi/ompi#12427

github-actions · 2024-03-23T23:29:22Z

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. To keep this pull request open, add a help wanted or in progress label.

Bo98 · 2024-03-27T18:51:56Z

Formula/o/open-mpi.rb

+
+      # Work around asm incompatibility with new linker (FB13194320)
+      # https://github.com/open-mpi/ompi/issues/11935
+      ENV.append "LDFLAGS", "-Wl,-ld_classic" if DevelopmentTools.clang_build_version >= 1500


I think it needs to be FCFLAGS instead (or in addition?)

https://github.com/open-mpi/ompi/blob/b5328be6ff3f1ae92dc94c922766d1f192bbd3af/config/ompi_setup_fc.m4#L241-L243

I reckon open-mpi wrapper stuff doesn't read from the environment though so might need to hack the configure file to insert.

Basically: whatever is required to make it appear in share/openmpi/mpifort-wrapper-data.txt. I edited that file locally and it seems to work.

fxcoudert · 2024-03-27T20:00:08Z

I tried FCFLAGS and wrapper_extra_ldflags but neither is working. But… according to https://docs.open-mpi.org/en/v5.0.x/installing-open-mpi/configure-cli-options/misc.html#install-wrapper-flags-label this should do the trick: --with-wrapper-ldflags=

Bo98

Thanks!

github-actions · 2024-03-29T09:08:52Z

@fxcoudert has requested bottles to be published to this PR.

github-actions · 2024-03-29T09:09:05Z

⚠️ @fxcoudert bottle publish failed.

fxcoudert · 2024-03-29T09:21:33Z

vineyard is conflicting with #166008

github-actions · 2024-03-29T14:33:58Z

@ZhongRuoyu has requested bottles to be published to this PR.

chenrui333 · 2024-03-30T17:57:31Z

Formula/o/open-mpi.rb

@@ -38,6 +39,10 @@ def install
    if OS.mac?
      # Otherwise libmpi_usempi_ignore_tkr gets built as a static library
      ENV["MACOSX_DEPLOYMENT_TARGET"] = MacOS.version
+
+      # Work around asm incompatibility with new linker (FB13194320)
+      # https://github.com/open-mpi/ompi/issues/11935


this issue reference seems wrong?

#167579

relates to #166807 Signed-off-by: Rui Chen <rui@chenrui.dev>

github-actions bot added the bump-formula-pr PR was created using `brew bump-formula-pr` label Mar 21, 2024

fxcoudert approved these changes Mar 21, 2024

View reviewed changes

chenrui333 added the test failure CI fails while running the test-do block label Mar 21, 2024

fxcoudert force-pushed the bump-pmix-5.0.2 branch from 2df2844 to aa98204 Compare March 21, 2024 16:20

github-actions bot added the CI-linux-self-hosted Build on Linux self-hosted runner label Mar 21, 2024

fxcoudert mentioned this pull request Mar 21, 2024

Apple linker does not accept -commons use_dylibs flag anymore open-mpi/ompi#12427

Closed

github-actions bot added the stale No recent activity label Mar 23, 2024

chenrui333 added in progress Stale bot should stay away and removed stale No recent activity labels Mar 24, 2024

fxcoudert mentioned this pull request Mar 27, 2024

gcc: switch back to ld-classic linker #167319

Closed

fxcoudert force-pushed the bump-pmix-5.0.2 branch from aa98204 to a6be316 Compare March 27, 2024 18:18

fxcoudert added the CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. label Mar 27, 2024

Bo98 reviewed Mar 27, 2024

View reviewed changes

BrewTestBot and others added 2 commits March 27, 2024 21:12

pmix 5.0.2

3a4829c

open-mpi: revision bump (pmix)

f1b2b17

fxcoudert force-pushed the bump-pmix-5.0.2 branch from a6be316 to f1b2b17 Compare March 27, 2024 20:12

fxcoudert removed in progress Stale bot should stay away test failure CI fails while running the test-do block labels Mar 27, 2024

fxcoudert added 2 commits March 28, 2024 07:35

opencoarrays: revision bump (open-mpi)

dbf6433

vineyard: revision bump (open-mpi)

4f6e51f

fxcoudert added the ready to merge PR can be merged once CI is green label Mar 28, 2024

chenrui333 approved these changes Mar 28, 2024

View reviewed changes

chenrui333 requested a review from Bo98 March 28, 2024 21:22

Bo98 approved these changes Mar 29, 2024

View reviewed changes

fxcoudert mentioned this pull request Mar 29, 2024

protobuf 26.1 #166008

Merged

ZhongRuoyu added 4 commits March 29, 2024 14:34

open-mpi: update 5.0.2_1 bottle.

58ef0ae

opencoarrays: update 2.10.2_2 bottle.

58f4c7e

pmix: update 5.0.2 bottle.

f7d41b2

vineyard: update 0.21.5_1 bottle.

d038fb7

github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Mar 29, 2024

github-actions bot approved these changes Mar 29, 2024

View reviewed changes

BrewTestBot enabled auto-merge March 29, 2024 14:36

BrewTestBot added this pull request to the merge queue Mar 29, 2024

Merged via the queue into master with commit 5168b9d Mar 29, 2024
14 checks passed

BrewTestBot deleted the bump-pmix-5.0.2 branch March 29, 2024 14:46

chenrui333 reviewed Mar 30, 2024

View reviewed changes

chenrui333 mentioned this pull request Mar 30, 2024

open-mpi: update upstream issue ref #167579

Merged

6 tasks

chenrui333 added a commit that referenced this pull request Mar 30, 2024

open-mpi: update upstream issue ref

556d8c5

relates to #166807 Signed-off-by: Rui Chen <rui@chenrui.dev>

github-actions bot added the outdated PR was locked due to age label Apr 29, 2024

github-actions bot locked as resolved and limited conversation to collaborators Apr 29, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

pmix 5.0.2 #166807

pmix 5.0.2 #166807

BrewTestBot commented Mar 21, 2024

chenrui333 commented Mar 21, 2024

fxcoudert commented Mar 21, 2024 •

edited

Loading

github-actions bot commented Mar 23, 2024

Bo98 Mar 27, 2024 •

edited

Loading

Bo98 Mar 27, 2024 •

edited

Loading

Bo98 Mar 27, 2024

fxcoudert commented Mar 27, 2024

Bo98 left a comment

github-actions bot commented Mar 29, 2024

github-actions bot commented Mar 29, 2024

fxcoudert commented Mar 29, 2024

github-actions bot commented Mar 29, 2024

chenrui333 Mar 30, 2024 •

edited

Loading

pmix 5.0.2 #166807

pmix 5.0.2 #166807

Conversation

BrewTestBot commented Mar 21, 2024

chenrui333 commented Mar 21, 2024

fxcoudert commented Mar 21, 2024 • edited Loading

github-actions bot commented Mar 23, 2024

Bo98 Mar 27, 2024 • edited Loading

Choose a reason for hiding this comment

Bo98 Mar 27, 2024 • edited Loading

Choose a reason for hiding this comment

Bo98 Mar 27, 2024

Choose a reason for hiding this comment

fxcoudert commented Mar 27, 2024

Bo98 left a comment

Choose a reason for hiding this comment

github-actions bot commented Mar 29, 2024

github-actions bot commented Mar 29, 2024

fxcoudert commented Mar 29, 2024

github-actions bot commented Mar 29, 2024

chenrui333 Mar 30, 2024 • edited Loading

Choose a reason for hiding this comment

fxcoudert commented Mar 21, 2024 •

edited

Loading

Bo98 Mar 27, 2024 •

edited

Loading

Bo98 Mar 27, 2024 •

edited

Loading

chenrui333 Mar 30, 2024 •

edited

Loading