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

feat: add paho mqtt c lib #914

Merged
merged 10 commits into from
Feb 24, 2020
Merged

Conversation

gocarlos
Copy link
Contributor

@gocarlos gocarlos commented Feb 19, 2020

Specify library name and version: paho-c/1.3.0

based on https://github.com/conan-community/conan-paho-c

closes #916

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

@conan-center-bot
Copy link
Collaborator

Failure in build 1 (c5a32bd1071a60f09dfa360bb97268ed5eb8c2d0):
paho-c/1.3.0:

  • Error processing recipe: Linux x86_64, Release, gcc 5, libstdc++ . Options: paho-c:shared-True
    Unexpected error happened:
ERROR: paho-c/1.3.0: Error in package_id() method, line 79
	if self.options.header_only:
	ConanException: 'options.header_only' doesn't exist
Possible options are ['shared', 'fPIC', 'SSL', 'asynchronous']

@gocarlos gocarlos changed the title feat: add paho c mqtt lib [wip] feat: add paho c mqtt lib Feb 19, 2020
@conan-center-bot
Copy link
Collaborator

Some configurations of 'paho-c/1.3.0' failed in build 3 (62742b9b5d0bd8cb724c0e773ca29040591818ef):

@gocarlos gocarlos changed the title [wip] feat: add paho c mqtt lib [wip] feat: add paho mqtt c lib Feb 19, 2020
@conan-center-bot
Copy link
Collaborator

Some configurations of 'paho-mqtt-c/1.3.0' failed in build 4 (60daeb7a20188056db04f39102dee27598bd7f93):

  • Linux x86_64, Release, gcc 5, libstdc++ . Options: paho-mqtt-c:shared-True
    • Hooks errors detected:
      • [HOOK - conan-center.py] post_source(): ERROR: [CPPSTD MANAGEMENT (KB-H022)] Can't detect C++ source files but recipe does not remove 'self.settings.compiler.cppstd' (https://github.com/conan-io/conan-center-index/wiki/Error-Knowledge-Base#KB-H022)
    • Log download
    • Log error Download
    • Profile
  • Access to all the logs

@conan-center-bot
Copy link
Collaborator

Some configurations of 'paho-mqtt-c/1.3.0' failed in build 5 (6bacbd96c11aa8626dfb1857f1f6323090cb0478):

@conan-center-bot
Copy link
Collaborator

Some configurations of 'paho-mqtt-c/1.3.0' failed in build 6 (b75e678e9c24aa29f8bb66f72e8b024cd1e2cfcc):

@conan-center-bot
Copy link
Collaborator

All green in build 7 (44d2d065b80d8427996b490e341f43e17a1083bc)! 😊

@gocarlos gocarlos changed the title [wip] feat: add paho mqtt c lib feat: add paho mqtt c lib Feb 19, 2020
@conan-center-bot
Copy link
Collaborator

Some configurations of 'paho-mqtt-c/1.3.0' failed in build 8 (8b91a1e1e593bc704e8c32b2416f8b60be0bfd39):

@conan-center-bot
Copy link
Collaborator

All green in build 9 (7eb87f003f06a209fe677894a054bf33658fa127)! 😊

@conan-center-bot
Copy link
Collaborator

All green in build 10 (2922c079b1da7c6cafa4fdfd5f160aed2c4f1c56)! 😊

Copy link
Member

@uilianries uilianries left a comment

Choose a reason for hiding this comment

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

I think Paho MQTT C sounds better nowadays, as they have more projects involved now.

@gocarlos
Copy link
Contributor Author

@danimtb anything else that should be done here to get this merged?

@gocarlos gocarlos mentioned this pull request Feb 21, 2020
4 tasks
@danimtb danimtb self-assigned this Feb 24, 2020
@danimtb danimtb merged commit a51564a into conan-io:master Feb 24, 2020
gocarlos referenced this pull request in bincrafters/bincrafters-conventions Feb 24, 2020
@gocarlos
Copy link
Contributor Author

@uilianries
Copy link
Member

Yeah, I'll do it!

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.

[request] port paho-mqtt-cpp to CCI
6 participants