Skip to content

protobuf 3.6.1.1#34372

Closed
igas wants to merge 1 commit intoHomebrew:masterfrom
igas:protobuf-3.6.1.1
Closed

protobuf 3.6.1.1#34372
igas wants to merge 1 commit intoHomebrew:masterfrom
igas:protobuf-3.6.1.1

Conversation

@igas
Copy link

@igas igas commented Nov 21, 2018

Created with brew bump-formula-pr.

@SMillerDev SMillerDev added the build failure CI fails while building the software label Nov 22, 2018
@Moisan
Copy link
Member

Moisan commented Nov 25, 2018

Fails with Can't locate loadable object for module Fcntl in @INC.

@lembacon
Copy link
Contributor

@BrewTestBot test this please

@lembacon
Copy link
Contributor

(The Perl-related testing failure on Mojave should have been fixed by now, let's just try again. 😉)

@Moisan
Copy link
Member

Moisan commented Nov 25, 2018

Thank you @igas !

@Moisan Moisan removed the build failure CI fails while building the software label Nov 25, 2018
@Moisan Moisan closed this in 3a6b38b Nov 25, 2018
@jd-a
Copy link
Contributor

jd-a commented Nov 25, 2018

Is it still working with clang 7? I get an error now.

@kkdd
Copy link
Contributor

kkdd commented Nov 26, 2018

protobuf 3.6.1.1 or earlier is incompatible with python 3.7 for now.
Don't install/upgrade it with --with-python.

@benmwebb
Copy link
Contributor

protobuf 3.6.1.1 or earlier is incompatible with python 3.7 for now.

See #29660 which fixed this in Homebrew for protobuf 3.6.1. Looks like the patch was accidentally removed for 3.6.1.1, maybe because it was thought it had been incorporated upstream. #34478 should fix this again in Homebrew for 3.6.1.1 by simply readding the patch.

@igas igas deleted the protobuf-3.6.1.1 branch December 18, 2018 23:13
@lock lock bot added the outdated PR was locked due to age label Jan 17, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jan 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated PR was locked due to age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants