Skip to content

simplified and updated README.md#75

Open
UnixJunkie wants to merge 1 commit intomasterfrom
UnixJunkie-patch-1
Open

simplified and updated README.md#75
UnixJunkie wants to merge 1 commit intomasterfrom
UnixJunkie-patch-1

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