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

simplified and updated README.md #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

UnixJunkie
Copy link
Collaborator

We don't support Python2 anymore.
Also, the brew recipe is supposed to be fully automatic now, so removed outdated instructions.

We don't support Python2 anymore.
Also, the brew recipe is supposed to be fully automatic now, so removed outdated instructions.
@UnixJunkie
Copy link
Collaborator Author

related to #72

@cthoyt
Copy link
Contributor

cthoyt commented Sep 30, 2019

If there's no more python2 support, then is the --with-python3 flag redundant? Maybe this isn't the right PR to make that change

@UnixJunkie
Copy link
Collaborator Author

UnixJunkie commented Oct 1, 2019

'--with-python3' means that you want the Python wrappers to be built.
You could build rdkit without its Python wrappers.

@UnixJunkie
Copy link
Collaborator Author

I should simplify this even further.
I should exemplify the two cases also:

  • you want pip3 to manage numpy
  • you want brew to manage numpy

@UnixJunkie UnixJunkie self-assigned this Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants