Skip to content

Conversation

yriveiro
Copy link
Contributor

@yriveiro yriveiro commented Feb 16, 2024

Documentation here mentions to install libev from homebrew.

 $ brew install libev

The problem I found is that the pre-build package from pip doesn't have the extensions compiled PYTHON-1376. Building from source on macOS doesn't find libev include folder for Homebrew as well.

This PR fixes the missing include folder to be aligned with the current documentation.

@absurdfarce
Copy link
Collaborator

Thanks for the PR @yriveiro !

Have you signed the Contributor License Agreement for contributions to DataStax open source projects? If not you can find it at https://cla.datastax.com/. Thanks!

@absurdfarce
Copy link
Collaborator

FTR PYTHON-1376 has been closed as a duplicate of PYTHON-1378. That effort focuses on getting the wheel build fixed, however, while @yriveiro 's fix in this PR facilitates building the extension correctly on OS X + Homebrew. Since it's addressing a slightly different problem than what's discussed in PYTHON-1378 I'd like to keep this PR open and handle it just like any other PR; it seems like a worthwhile improvement to consider.

@yriveiro
Copy link
Contributor Author

Thanks for the PR @yriveiro !

Have you signed the Contributor License Agreement for contributions to DataStax open source projects? If not you can find it at https://cla.datastax.com/. Thanks!

I think I had done time ago but I reset it and signed again.

@absurdfarce
Copy link
Collaborator

Thanks @yriveiro !

I'm finding myself working in a very similar area of the code base for a different ticket so I'm going to get your change merged now @yriveiro so as not to create any merge conflicts later on. Thanks again for the PR!

@absurdfarce absurdfarce merged commit 2c61ab2 into datastax:master Feb 27, 2024
fruch added a commit to fruch/python-driver that referenced this pull request Jun 6, 2024
…sync_with_upstream_3.29.1

version 3.29.1

* tag '3.29.1' of https://github.com/datastax/python-driver:
  Release 3.29.1: changelog & version (datastax#1199)
  PYTHON-1378 Expand search directories for includes (datastax#1198)
  clean up last "import six" (datastax#1197)
  fix build from source on macos using homebrew (datastax#1196)
  Added 3.29.0 to docs.yaml
fruch added a commit to fruch/python-driver that referenced this pull request Jun 6, 2024
…sync_with_upstream_3.29.1

version 3.29.1

* tag '3.29.1' of https://github.com/datastax/python-driver:
  Release 3.29.1: changelog & version (datastax#1199)
  PYTHON-1378 Expand search directories for includes (datastax#1198)
  clean up last "import six" (datastax#1197)
  fix build from source on macos using homebrew (datastax#1196)
  Added 3.29.0 to docs.yaml
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.

2 participants