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

Pin the version of boost, boost-python, boost-python3 in homebrew formula #65

Closed
hsiaoyi0504 opened this issue Jun 5, 2018 · 9 comments

Comments

@hsiaoyi0504
Copy link
Contributor

hsiaoyi0504 commented Jun 5, 2018

I think this can avoid several issues we met before in the future like #62. By default, homebrew formula will install latest boost. By pinning the version, we can make sure rdkit will be installed by specific boost/boost-python/boost-python3 version. I am currently working on submitting the patches that provide formulas like boost@1.66.

@hsiaoyi0504
Copy link
Contributor Author

As far as I know, 2017_09 and 2018_03 releases are not compatible to boost 1.67 (which is the boost version will be installed by homebrew).

@mcs07
Copy link
Contributor

mcs07 commented Jun 5, 2018

Alternatively, it should be possible to temporarily apply this patch to RDKit for Boost 1.67 compatibility:
rdkit/rdkit#1864

Homebrew patch docs: https://github.com/Homebrew/brew/blob/master/docs/Formula-Cookbook.md#patches

It looks like that PR is scheduled for inclusion in the 2018.09.1 release.

@hsiaoyi0504
Copy link
Contributor Author

You are right, but it seems to me that pinning the version will be more stable compared to fix the dependencies frequently.

@hsiaoyi0504
Copy link
Contributor Author

Just a short update.
It seems that the suggested solution is to maintain our own tap.

Homebrew/homebrew-core#28677 (comment)

@mcs07
Copy link
Contributor

mcs07 commented Jun 6, 2018

This repository is the rdkit tap, so you could in theory just put the boost@1.66 formula in the Formula directory alongside rdkit.rb.

@hsiaoyi0504
Copy link
Contributor Author

Finally got my mac laptop back. I put my current effort here: https://github.com/hsiaoyi0504/homebrew-rdkit/tree/pin_boost_version_2.

However, I meet several problems:

  • It seems to me that DBoost_NO_SYSTEM_PATHS is not respected, cmake's find_boost keep finding the system boost.
  • Not sure how to set the boost python paths
  • The building time is too long on Travis.

@hsiaoyi0504
Copy link
Contributor Author

Proposed procedures to be run on local (python3 and using pip install numpy):

brew install ./Formula/boost@1.66.rb --build-from-source
brew install ./Formula/boost-python3@1.66.rb --build-from-source
brew install ./Formula/rdkit.rb --build-from-source --with-python3 --without-numpy

@UnixJunkie
Copy link
Collaborator

This is a very good idea in order to make the brew formula for rdkit much more reproducible.

@UnixJunkie
Copy link
Collaborator

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants