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

confu_json version 1.0.1 #23940

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

confu_json version 1.0.1 #23940

wants to merge 22 commits into from

Conversation

werto87
Copy link
Contributor

@werto87 werto87 commented May 9, 2024

confu_json/1.0.1

version bump of confu_json and dependencies. using cmake install to discribe what needs to be coppied.
Switched from
include(${CMAKE_BINARY_DIR}/conanbuildinfo.cmake) conan_basic_setup(TARGETS)
to
find_package(magic_enum REQUIRED)

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

Copy link
Member

@AbrilRBS AbrilRBS left a comment

Choose a reason for hiding this comment

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

Small review, thanks a lot for taking the time to keep the recipe update, we appreciate it :)

recipes/confu_json/all/conanfile.py Outdated Show resolved Hide resolved
recipes/confu_json/all/conanfile.py Outdated Show resolved Hide resolved
@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

Failure in build 11 (1973010508f8f8a35569df0e5ae611abf17107a1):

  • confu_json/1.0.1:
    Didn't run or was cancelled before finishing

  • confu_json/1.1.0:
    CI failed to create some packages (All logs)

    Logs for packageID 5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=gcc
    compiler.libcxx=libstdc++11
    compiler.version=9
    os=Linux
    
    [...]
                     from /home/conan/workspace/prod-v1/bsr/cci-cabae117/recipes/confu_json/all/test_package/test_package.cpp:1:
    /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/to_json.hxx: In function ‘bool confu_json::handleUniquePtr(boost::json::object&, const T&, const string&)’:
    /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/to_json.hxx:215:35: error: need ‘typename’ before ‘boost::mpl::at<BaseToDerivedMapping, typename std::decay<decltype (* t.get())>::type>::type’ because ‘boost::mpl::at<BaseToDerivedMapping, typename std::decay<decltype (* t.get())>::type>’ is a dependent scope
      215 |                   using Derived = boost::mpl::at<BaseToDerivedMapping, uniquePtrType>::type;
          |                                   ^~~~~
          |                                   typename 
    /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/to_json.hxx:216:85: error: ‘Derived’ does not name a type
      216 |                   result[memberName] = to_json<BaseToDerivedMapping> (*dynamic_cast<Derived *> (t.get ()));
          |                                                                                     ^~~~~~~
    /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/to_json.hxx:216:93: error: expected ‘>’ before ‘*’ token
      216 |                   result[memberName] = to_json<BaseToDerivedMapping> (*dynamic_cast<Derived *> (t.get ()));
          |                                                                                             ^
    /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/to_json.hxx:216:93: error: expected ‘(’ before ‘*’ token
      216 |                   result[memberName] = to_json<BaseToDerivedMapping> (*dynamic_cast<Derived *> (t.get ()));
          |                                                                                             ^
          |                                                                                             (
    /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/to_json.hxx:216:94: error: expected primary-expression before ‘>’ token
      216 |                   result[memberName] = to_json<BaseToDerivedMapping> (*dynamic_cast<Derived *> (t.get ()));
          |                                                                                              ^
    In file included from /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/confu_json.hxx:13,
                     from /home/conan/workspace/prod-v1/bsr/cci-cabae117/recipes/confu_json/all/test_package/test_package.cpp:1:
    /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/to_object.hxx: In function ‘void confu_json::handleUniquePtr(T&, U&, const string&)’:
    /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/to_object.hxx:438:31: error: need ‘typename’ before ‘boost::mpl::at<BaseToDerivedMapping, typename std::decay<decltype (* t.get())>::type>::type’ because ‘boost::mpl::at<BaseToDerivedMapping, typename std::decay<decltype (* t.get())>::type>’ is a dependent scope
      438 |               using Derived = boost::mpl::at<BaseToDerivedMapping, uniquePtrType>::type;
          |                               ^~~~~
          |                               typename 
    /home/conan/workspace/prod-v1/bsr/54035/fdeaf/.conan/data/confu_json/1.1.0/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include/confu_json/to_object.hxx:439:36: error: ‘Derived’ was not declared in this scope
      439 |               t = std::make_unique<Derived> (to_object<Derived, BaseToDerivedMapping> (jsonDataForMember.as_object ()));
          |                                    ^~~~~~~
    make[2]: *** [test_package/CMakeFiles/test_package.dir/build.make:82: test_package/CMakeFiles/test_package.dir/test_package.cpp.o] Error 1
    make[1]: *** [CMakeFiles/Makefile2:113: test_package/CMakeFiles/test_package.dir/all] Error 2
    make: *** [Makefile:103: all] Error 2
    WARN: *** Conan 1 is legacy and on a deprecation path ***
    WARN: *** Please upgrade to Conan 2 ***
    confu_json/1.1.0 (test package): WARN: 
         ************************************************
         The 'cmake_find_package_multi' generator is deprecated.
         Please update your code and remove it.
         *************************************************
    
    confu_json/1.1.0 (test package): WARN: 
         ************************************************
         The 'cmake' generator is deprecated.
         Please update your code and remove it.
         *************************************************
    
    confu_json/1.1.0 (test package): WARN: **** The 'from conans import CMake' helper is deprecated. Please update your code and remove it. ****
    ERROR: confu_json/1.1.0 (test package): Error in build() method, line 12
    	cmake.build()
    	ConanException: Error 2 while executing cmake --build '/home/conan/workspace/prod-v1/bsr/cci-cabae117/recipes/confu_json/all/test_v1_package/build/dd4f758f56da1cf19b4ccbf15cc7dfa8149e8b47' '--' '-j3'
    

Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.


Conan v2 pipeline ❌

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

The v2 pipeline failed. Please, review the errors and note this is required for pull requests to be merged. In case this recipe is still not ported to Conan 2.x, please, ping @conan-io/barbarians on the PR and we will help you.

See details:

Failure in build 11 (1973010508f8f8a35569df0e5ae611abf17107a1):

  • confu_json/1.0.1:
    Didn't run or was cancelled before finishing

  • confu_json/1.1.0:
    CI failed to create some packages (All logs)

    Logs for packageID da39a3ee5e6b4b0d3255bfef95601890afd80709:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=gcc
    compiler.cppstd=17
    compiler.libcxx=libstdc++11
    compiler.version=11
    os=Linux
    
    [...]
                     from /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/confu_json.hxx:12,
                     from /home/conan/workspace/prod-v2/bsr/cci-3e797976/recipes/confu_json/all/test_package/test_package.cpp:1:
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/concept.hxx:28:1: error: 'concept' does not name a type
       28 | concept IsSharedPtr = is_shared_ptr<T>::value;
          | ^~~~~~~
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/concept.hxx:28:1: note: 'concept' only available with '-std=c++20' or '-fconcepts'
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/concept.hxx:37:1: error: 'concept' does not name a type
       37 | concept IsUniquePtr = is_unique_ptr<T>::value;
          | ^~~~~~~
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/concept.hxx:37:1: note: 'concept' only available with '-std=c++20' or '-fconcepts'
    In file included from /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/confu_json.hxx:12,
                     from /home/conan/workspace/prod-v2/bsr/cci-3e797976/recipes/confu_json/all/test_package/test_package.cpp:1:
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/to_json.hxx: In function 'bool confu_json::handleUniquePtr(boost::json::object&, const T&, const string&)':
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/to_json.hxx:215:35: error: need 'typename' before 'boost::mpl::at<BaseToDerivedMapping, typename std::decay<decltype (* t.get())>::type>::type' because 'boost::mpl::at<BaseToDerivedMapping, typename std::decay<decltype (* t.get())>::type>' is a dependent scope
      215 |                   using Derived = boost::mpl::at<BaseToDerivedMapping, uniquePtrType>::type;
          |                                   ^~~~~
          |                                   typename 
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/to_json.hxx:216:85: error: 'Derived' does not name a type
      216 |                   result[memberName] = to_json<BaseToDerivedMapping> (*dynamic_cast<Derived *> (t.get ()));
          |                                                                                     ^~~~~~~
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/to_json.hxx:216:93: error: expected '>' before '*' token
      216 |                   result[memberName] = to_json<BaseToDerivedMapping> (*dynamic_cast<Derived *> (t.get ()));
          |                                                                                             ^
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/to_json.hxx:216:93: error: expected '(' before '*' token
      216 |                   result[memberName] = to_json<BaseToDerivedMapping> (*dynamic_cast<Derived *> (t.get ()));
          |                                                                                             ^
          |                                                                                             (
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/to_json.hxx:216:94: error: expected primary-expression before '>' token
      216 |                   result[memberName] = to_json<BaseToDerivedMapping> (*dynamic_cast<Derived *> (t.get ()));
          |                                                                                              ^
    In file included from /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/confu_json.hxx:13,
                     from /home/conan/workspace/prod-v2/bsr/cci-3e797976/recipes/confu_json/all/test_package/test_package.cpp:1:
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/to_object.hxx: In function 'void confu_json::handleUniquePtr(T&, U&, const string&)':
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/to_object.hxx:438:31: error: need 'typename' before 'boost::mpl::at<BaseToDerivedMapping, typename std::decay<decltype (* t.get())>::type>::type' because 'boost::mpl::at<BaseToDerivedMapping, typename std::decay<decltype (* t.get())>::type>' is a dependent scope
      438 |               using Derived = boost::mpl::at<BaseToDerivedMapping, uniquePtrType>::type;
          |                               ^~~~~
          |                               typename 
    /home/conan/workspace/prod-v2/bsr/47817/caaff/p/b/confucb58bbbfbe835/p/include/confu_json/to_object.hxx:439:36: error: 'Derived' was not declared in this scope
      439 |               t = std::make_unique<Derived> (to_object<Derived, BaseToDerivedMapping> (jsonDataForMember.as_object ()));
          |                                    ^~~~~~~
    CMakeFiles/test_package.dir/build.make:62: recipe for target 'CMakeFiles/test_package.dir/test_package.cpp.o' failed
    make[2]: *** [CMakeFiles/test_package.dir/test_package.cpp.o] Error 1
    CMakeFiles/Makefile2:75: recipe for target 'CMakeFiles/test_package.dir/all' failed
    make[1]: *** [CMakeFiles/test_package.dir/all] Error 2
    Makefile:83: recipe for target 'all' failed
    make: *** [all] Error 2
    
    ERROR: confu_json/1.1.0 (test package): Error in build() method, line 21
    	cmake.build()
    	ConanException: Error 2 while executing
    

Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants