memsharded / conan-zmq

ZeroMQ base C++ library conan package
MIT License
4 stars 5 forks source link

MacOS do not require pthread #5

Open memsharded opened 7 years ago

memsharded commented 7 years ago

Re: the libzmq update - looks great. One thing - MacOS does not require -pthread, so if you could put a condition into the libzmq Conanfile removing -pthread for Mac that would remove a warning on each build:

clang: warning: argument unused during compilation: '-pthread' It is adding Macos to this condition:

    if not self.settings.os == "Windows" and not self.settings.os == "Macos":
        self.cpp_info.cppflags = ["-pthread"]

Thanks!

cc/ @sourcedelica