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 cpp #936

Merged
merged 5 commits into from
Mar 2, 2020
Merged

Conversation

gocarlos
Copy link
Contributor

@gocarlos gocarlos commented Feb 21, 2020

Specify library name and version: paho-mqtt-cpp/1.0.1

depends on #914
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 (4f5742959f87d0fc3f65ea6e88ffa5d8f23d50c9):
paho-mqtt-cpp/1.0.1:

  • Error processing recipe: Linux x86_64, Release, gcc 5, libstdc++ . Options: paho-mqtt-cpp:shared-True
    You are depending on 'paho-mqtt-c/1.3.0' but it is not in the repository

@uilianries
Copy link
Member

At first view, it looks good.

@gocarlos gocarlos closed this Feb 24, 2020
@gocarlos gocarlos reopened this Feb 24, 2020
@conan-center-bot
Copy link
Collaborator

Some configurations of 'paho-mqtt-cpp/1.0.1' failed in build 2 (4f5742959f87d0fc3f65ea6e88ffa5d8f23d50c9):

@conan-center-bot
Copy link
Collaborator

Some configurations of 'paho-mqtt-cpp/1.0.1' failed in build 3 (72d960aeb16e87556e37dae86bfa964be84bed87):

@gocarlos
Copy link
Contributor Author

@uilianries what to do here? Could NOT find PahoMqttC (missing: PAHO_MQTT_C_LIBRARIES)

@uilianries
Copy link
Member

@gocarlos #956

Co-Authored-By: Uilian Ries <uilianries@gmail.com>
@conan-center-bot
Copy link
Collaborator

Some configurations of 'paho-mqtt-cpp/1.0.1' failed in build 4 (8b71a2e98fde40af62d1d18f2a6856eaab47ad72):

@conan-center-bot
Copy link
Collaborator

All green in build 5 (953338a94bae6927f141accb2e86c41f32b610ca)! 😊

@gocarlos gocarlos marked this pull request as ready for review February 24, 2020 22:23
uilianries
uilianries previously approved these changes Feb 24, 2020
SSE4
SSE4 previously approved these changes Feb 24, 2020
@gocarlos
Copy link
Contributor Author

@gocarlos #956

@uilianries
this MR is green even though #956 is still not merged?

@danimtb
Copy link
Member

danimtb commented Feb 25, 2020

@gocarlos I think this recipe is not using the find_package names, is it?

@gocarlos
Copy link
Contributor Author

@gocarlos I think this recipe is not using the find_package names, is it?

#936 (comment)

@danimtb
Copy link
Member

danimtb commented Feb 25, 2020

Sorry, I did not read the message. I am very confused about this then... 😕

@gocarlos gocarlos closed this Feb 25, 2020
@gocarlos gocarlos reopened this Feb 25, 2020
@conan-center-bot
Copy link
Collaborator

All green in build 6 (953338a94bae6927f141accb2e86c41f32b610ca)! 😊

@gocarlos gocarlos dismissed stale reviews from SSE4 and uilianries via e1ac4d7 February 28, 2020 14:20
@conan-center-bot
Copy link
Collaborator

All green in build 7 (e1ac4d74c49b905a1b19b14d7d2b9d313092fe5f)! 😊

@danimtb danimtb self-assigned this Mar 2, 2020
@danimtb danimtb merged commit 0a02999 into conan-io:master Mar 2, 2020
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
5 participants