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

Boost/1.72.0/mpi #3959

Merged
merged 3 commits into from
Jul 13, 2021
Merged

Conversation

oleurodecision
Copy link
Contributor

@oleurodecision oleurodecision commented Dec 19, 2020

Specify library name and version: lib/1.0

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

(note that this PR replaces the #2842 for branch name change)

@conan-center-bot

This comment has been minimized.

@prince-chrismc
Copy link
Contributor

prince-chrismc commented Dec 22, 2020

Are you have a problem in particular that requires this?

From https://www.boost.org/doc/libs/1_72_0/doc/html/mpi/getting_started.html#mpi.getting_started.config.bootstrap

Alternatively, you can explicitly provide the list of Boost libraries you want to build. Please refer to the --help option of the bootstrap script.

Which I am pretty sure the recipe does here

libraries = ' '.join(libraries)
command = "{bcp} {namespace} {alias} " \
"{boostdir} {libraries} {outdir}".format(bcp=self._bcp_exe,
namespace=namespace,
alias=alias,
libraries=libraries,

Edit also be related to #3900 (comment) #3872

@conan-center-bot

This comment has been minimized.

@stale
Copy link

stale bot commented Jun 25, 2021

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. Thank you for your contributions.

@stale stale bot added the stale label Jun 25, 2021
@leha-bot
Copy link
Contributor

leha-bot commented Jul 1, 2021

Hey, guys, any updates? With this fix Boost.MPI at least properly uses the system MPI library,

@stale stale bot removed the stale label Jul 1, 2021
@prince-chrismc
Copy link
Contributor

Can you confirm the recipe revision you are using?

OP only modified one version... I assume this needs to be applied to all of them

@oleurodecision
Copy link
Contributor Author

Happy to this one alive again.
Well, could only test the patch on 1.72 at the time of writing. I have time to spend on the subject these comnig days, so if required I can update the PR on current master, and check for the other handled versions (1.69 to 1.76).

Please let me know if anything is required or desired from me.

The boost:without_mpi default value is switched to True (disabling Boost MPI support)
The mpi.jam file is patched to enable the error
@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

All green in build 11 (4e8a0baeee812f57787483ef17f22e89978777bd):

  • boost/1.69.0@:
    All packages built successfully! (All logs)

  • boost/1.70.0@:
    All packages built successfully! (All logs)

  • boost/1.71.0@:
    All packages built successfully! (All logs)

  • boost/1.72.0@:
    All packages built successfully! (All logs)

  • boost/1.73.0@:
    All packages built successfully! (All logs)

  • boost/1.75.0@:
    All packages built successfully! (All logs)

  • boost/1.74.0@:
    All packages built successfully! (All logs)

  • boost/1.76.0@:
    All packages built successfully! (All logs)

@conan-center-bot conan-center-bot merged commit b04118f into conan-io:master Jul 13, 2021
AndreyMlashkin pushed a commit to AndreyMlashkin/conan-center-index that referenced this pull request Jul 19, 2021
* Enables Boost MPI when requested (boost:without_mpi=False)

The boost:without_mpi default value is switched to True (disabling Boost MPI support)

* Adds Boost:MPI configuration check when enabled

The mpi.jam file is patched to enable the error

* Applies patch to all version
@oleurodecision oleurodecision deleted the boost/1.72.0/mpi branch April 15, 2022 13:27
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.

6 participants