Skip to content

Conversation

minrk
Copy link
Member

@minrk minrk commented Apr 4, 2017

uses regular build system, rather than the relatively unused cmake builds, which require many patches and don't work reliably.

generate cmake files as a compromise for downstream projects that seem to rely on them.

fixes #20

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@SylvainCorlay
Copy link
Member

Note that all the patches were integrated upstream.

@minrk
Copy link
Member Author

minrk commented Jan 12, 2018

The buggy CMake files for zeromq are again responsible for major issues like conda-forge/pyzmq-feedstock#27. I'm going to revive this PR, because libzmq CMake appears like it should ~never be used and regularly produces invalid libraries.

minrk added 5 commits January 11, 2018 17:48
generate cmake files ourselves as a compromise
dylib.5 is not correct
not the git archive, the actual release
@minrk
Copy link
Member Author

minrk commented Jan 12, 2018

Also previous builds of 4.2.3 on linux/mac should be marked as broken after merge (or immediately), since they create incorrect libraries.

cmake is clearly not well supported, since it doesn’t work from releases
@minrk minrk changed the title build without cmake on unix use normal builds on cmake Jan 12, 2018
@minrk minrk changed the title use normal builds on cmake use normal builds on unix Jan 12, 2018
@minrk minrk requested a review from SylvainCorlay January 12, 2018 02:45
@SylvainCorlay SylvainCorlay mentioned this pull request Jan 12, 2018
Copy link
Member

@SylvainCorlay SylvainCorlay left a comment

Choose a reason for hiding this comment

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

We posted a lengthy answer about the move to configure + make here.
#25

cmake runs files multiple times, so has to check if targets have been defined before defining them.
@JohanMabille JohanMabille merged commit 6348033 into conda-forge:master Jan 16, 2018
@minrk minrk deleted the no-cmake-unix branch January 17, 2018 00:30
@minrk
Copy link
Member Author

minrk commented Jan 17, 2018

Thanks! Travis is having major backlog issues (on all platforms now), so hopefully this will get published eventually.

@JohanMabille
Copy link
Contributor

Thanks for all the work you've put in this (both libzmq and xeus stack)! I'll make a conda package for cppzmq 4.2.3 as soon as this is published.

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.

4 participants