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

zeromq 4.3.1 #35940

Closed
wants to merge 1 commit into from
Closed

zeromq 4.3.1 #35940

wants to merge 1 commit into from

Conversation

aimileus
Copy link
Contributor

Created with brew bump-formula-pr.

@aimileus aimileus mentioned this pull request Jan 12, 2019
@bluca
Copy link
Contributor

bluca commented Jan 12, 2019

Thanks!

@fxcoudert fxcoudert closed this in 1a53dc7 Jan 12, 2019
@scpeters
Copy link
Member

The libzmq.pc pkg-config file has changed in 4.3.1 (I think from zeromq/libzmq#3336) to have the following line:

Requires.private:  libunwind

but there is no libunwind.pc file anywhere that I can find on macOS (I've checked high sierra and mojave), so downstream software that uses pkg-config to find zeromq fails to find it:

Package libunwind was not found in the pkg-config search path.
Perhaps you should add the directory containing `libunwind.pc'
to the PKG_CONFIG_PATH environment variable
Package 'libunwind', required by 'libzmq', not found

@scpeters
Copy link
Member

Maybe we can add a test invocation of pkg-config to the zeromq formula

@bluca
Copy link
Contributor

bluca commented Jan 14, 2019

Upstream's libunwind ships the pkgconfig file, so either fix the brew formula that builds it to install it, or disable libunwind in this formula
https://github.com/libunwind/libunwind/blob/master/src/unwind/libunwind.pc.in

@scpeters
Copy link
Member

libunwind functionality is provided by macOS; this might be a time to add a libunwind.pc to https://github.com/Homebrew/brew/tree/master/Library/Homebrew/os/mac/pkgconfig

@scpeters
Copy link
Member

I'm proposing to add libunwind.pc files to homebrew/brew in Homebrew/brew#5539

@bluca
Copy link
Contributor

bluca commented Jan 15, 2019

Ok. Note that if it's urgent you can disable libunwind manually with --disable-libunwind

@EricLambrecht
Copy link

@bluca Which command is supposed to be run with --disable-libunwind exactly?

@bluca
Copy link
Contributor

bluca commented Jan 15, 2019

that's for ./configure

@bluca
Copy link
Contributor

bluca commented Jan 15, 2019

Also I see the libunwind flags are set manually via ENV - it would be sufficient to remove those so that autoconf doesn't find it

@bluca
Copy link
Contributor

bluca commented Jan 16, 2019

There's something I don't quite follow: the Requires.private are filled by autoconf ONLY if the dependency is found VIA pkg-config - in other words, if there's no libunwind.pc, there won't be a Requires.private: libunwind.
If I understand correctly there's no libunwind.pc at the moment - so how exactly is it being found at build time?
As the autoconf file currently is, unless libunwind.pc is found, it won't even build with libunwind - there's no fallback at all.

@scpeters
Copy link
Member

Specifying the flags seems to make the configure script think that a .pc file exists.

@scpeters
Copy link
Member

Ok, I just submitted a PR to disable libunwind in the zeromq formula until we figure this out.

@bluca
Copy link
Contributor

bluca commented Jan 17, 2019

That's really strange, it's the pkgconfig m4 macro that's used and that should just fail if the pc is not found, is homebrew shipping a custom one?

@aimileus aimileus deleted the zeromq-4.3.1 branch January 18, 2019 06:11
@lock lock bot added the outdated PR was locked due to age label Feb 17, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Feb 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.

4 participants