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.

Sign up for GitHub

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

Jump to bottom

ARROW-6518: [Packaging][Python] Flight failing in OSX Python wheel builds #5350

Closed
wants to merge 3 commits into from
Closed

ARROW-6518: [Packaging][Python] Flight failing in OSX Python wheel builds #5350

wants to merge 3 commits into from

Conversation

kszucs
Copy link
Member

@kszucs kszucs commented Sep 11, 2019

No description provided.

@kszucs
Copy link
Member Author

kszucs commented Sep 11, 2019

@ursabot crossbow submit wheel-osx-cp36m

@ursabot
Copy link

ursabot commented Sep 11, 2019

AMD64 Conda Crossbow Submit (#60894) builder has been succeeded.

Revision: 515cb1a

Submitted crossbow builds: ursa-labs/crossbow @ ursabot-161

Task Status
wheel-osx-cp36m TravisCI

pitrou
pitrou reviewed Sep 11, 2019
View reviewed changes
dev/tasks/python-wheels/travis.osx.yml Outdated
@@ -47,7 +47,7 @@ before_install:
- brew uninstall boost cgal postgis sfcgal
- brew update
- brew upgrade cmake
- brew install bison flex grpc openssl llvm@7 zlib
- brew install bison flex grpc openssl llvm@7 zlib gperftools
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FTR, why is gperftools needed?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't know, grpc misses it.

@pitrou
Copy link
Member

pitrou commented Sep 11, 2019

The Travis-CI error ("Symbol not found: _EVP_MD_CTX_free") is probably because the runtime version of OpenSSL is not the same as the version used for building. Apparently EVP_MD_CTX_free exists on OpenSSL 1.1 but not on OpenSSL 1.0.

@kszucs
Copy link
Member Author

kszucs commented Sep 11, 2019

Yeah, grpc depends on openssl@1.1 brew package, but openssl is also installed. CMake picks up the latter one.

@kszucs
Copy link
Member Author

kszucs commented Sep 11, 2019

@ursabot crossbow submit wheel-osx-cp36m

@ursabot
Copy link

ursabot commented Sep 11, 2019

AMD64 Conda Crossbow Submit (#60914) builder has been succeeded.

Revision: 4c380bc

Submitted crossbow builds: ursa-labs/crossbow @ ursabot-164

Task Status
wheel-osx-cp36m TravisCI

@kszucs
Copy link
Member Author

kszucs commented Sep 11, 2019

@ursabot crossbow submit wheel-osx-cp36m

@ursabot
Copy link

ursabot commented Sep 11, 2019

AMD64 Conda Crossbow Submit (#60956) builder has been succeeded.

Revision: 16471d1

Submitted crossbow builds: ursa-labs/crossbow @ ursabot-165

Task Status
wheel-osx-cp36m TravisCI

@kszucs
Copy link
Member Author

kszucs commented Sep 11, 2019

@ursabot crossbow submit wheel-osx-cp27m wheel-osx-cp35m wheel-osx-cp36m wheel-osx-cp37m

@ursabot
Copy link

ursabot commented Sep 11, 2019

AMD64 Conda Crossbow Submit (#60996) builder has been succeeded.

Revision: 16471d1

Submitted crossbow builds: ursa-labs/crossbow @ ursabot-168

Task Status
wheel-osx-cp27m TravisCI
wheel-osx-cp35m TravisCI
wheel-osx-cp36m TravisCI
wheel-osx-cp37m TravisCI

kszucs
kszucs commented Sep 12, 2019
View reviewed changes
Copy link
Member Author

@kszucs kszucs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kszucs kszucs closed this in 2393472 Sep 12, 2019
pprudhvi pushed a commit to pprudhvi/arrow that referenced this pull request Sep 16, 2019
@kszucs @pprudhvi
ARROW-6518: [Packaging][Python] Flight failing in OSX Python wheel bu…
5b1d488 
…ilds

Closes apache#5350 from kszucs/ARROW-6518 and squashes the following commits:

16471d1 <Krisztián Szűcs> force use openssl@1.1
4c380bc <Krisztián Szűcs> use openssl@1.1
515cb1a <Krisztián Szűcs> install gperftools

Authored-by: Krisztián Szűcs <szucs.krisztian@gmail.com>
Signed-off-by: Krisztián Szűcs <szucs.krisztian@gmail.com>
bkmgit added a commit to bkmgit/arrow that referenced this pull request Oct 23, 2022
@bkmgit
Use openssl3
7a2a249 
Avoid error due to openssl1 missing `EVP_MD_CTX_free` mentioned in
apache#5350
However default version is Openssl1.0 on CentOS7
https://stackoverflow.com/questions/63508872/upgrading-centos-7-to-openssl-1-1-1-by-yum-install-openssl11
CMake may have trouble finding this
https://discourse.cmake.org/t/centos7-and-openssl-1-1/6229/2
Installation roughly follows:
- https://blacksaildivision.com/how-to-install-openssl-on-centos
- https://dev.to/nikolastojilj12/update-openssl-to-3-0-on-centos7-150o
@asfimport asfimport mentioned this pull request Sep 12, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pitrou pitrou pitrou left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kszucs @ursabot @pitrou