modern-cpp-kafka: new recipe

[kenneth-jia]

Specify library name and version: modern-cpp-kafka/v2022.06.15

Hi,
I'm the author of modern-cpp-kafka (https://github.com/morganstanley/modern-cpp-kafka), a Kafka C++ API, wrapper over librdkafka).
Now I'm trying to add this header-only library to the ConanCenter (morganstanley/modern-cpp-kafka#105)
Thanks!

---

• 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.

[CLAassistant]

All committers have signed the CLA.

[toge]

@kenneth-jia
Hello.
Thanks for creating the conan recipe!

Unfortunately, the recipe you wrote must have test_package folder.
There is a recently created template as a sample of the recipe, could you please refer this as a reference to modify it?
https://github.com/conan-io/conan-center-index/tree/master/docs/package

[kenneth-jia]

@kenneth-jia Hello. Thanks for creating the conan recipe!

Unfortunately, the recipe you wrote must have test_package folder. There is a recently created template as a sample of the recipe, could you please refer this as a reference to modify it? https://github.com/conan-io/conan-center-index/tree/master/docs/package

Just added the test_package part, and now is pending on the access approval. Thanks! @toge

[uilianries]

Thank you for your contribution

[uilianries]

only a small detail

[uilianries]

LGTM

Just need to wait for user approval.

[kenneth-jia]

LGTM

Just need to wait for user approval.

Thank you a lot for the help, @uilianries! 😀

[uilianries]

@kenneth-jia No, thank you for contributing! 💚

[conan-center-bot]

All green in build 27 (acac5710be98e23a6fa34d062abc147dfa7370d1):

• modern-cpp-kafka/2022.06.15@:
  All packages built successfully! (All logs)

  🔸 Informative: This recipe is not ready for Conan v2

  _{We have started the migration process to Conan v2 and exporting recipes successfully will be required in the future.
  This is just an informative note to gain awareness about the process, no need to take any action. The plan is to enforce smaller steps that are easier to fix and, eventually, this conan export step will work.
  See the recipe migration guide to know more about the changes required.}

  ERROR: Error loading conanfile at '/home/conan/w/prod_cci_PR-11430/recipes/modern-cpp-kafka/all/conanfile.py': Unable to load conanfile in /home/conan/w/prod_cci_PR-11430/recipes/modern-cpp-kafka/all/conanfile.py
    File "<frozen importlib._bootstrap_external>", line 728, in exec_module
    File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
    File "/home/conan/w/prod_cci_PR-11430/recipes/modern-cpp-kafka/all/conanfile.py", line 1, in <module>
      from conans import ConanFile, tools
  ImportError: cannot import name 'ConanFile' from 'conans' (/opt/pyenv/versions/3.7.13/lib/python3.7/site-packages/conans/__init__.py)
  

[jgsogo]

🎉

Let me explain the logic with v2 migration and those test folders:

• every recipe should have a test_package, it is the default name that Conan will look for.
• recipes in CCI can have several test_xxxx folders, and the CI here will execute all of them. Why? Sometimes it is useful to write several tests to check the names for different generators.
• if for some reason, there is a test we want to keep and it can only run with Conan v1 (for example, we want to keep testing cmake_find_package), then the folder should be named test_v1_xxxx, i.e.: https://github.com/conan-io/conan-center-index/tree/master/recipes/fmt/all

[uilianries]

LGTM