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

faker-cxx: add recipe #24018

Merged
merged 8 commits into from
Jun 29, 2024
Merged

faker-cxx: add recipe #24018

merged 8 commits into from
Jun 29, 2024

Conversation

toge
Copy link
Contributor

@toge toge commented May 17, 2024

Specify library name and version: faker-cxx/*


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

@toge
Copy link
Contributor Author

toge commented May 20, 2024

@RubenRBS

c++: fatal error: Killed signal terminated program cc1plus

Since faker-cxx has several large source files, CCI seems to kill the compiler process.
Is there any way to resolve this?

@uilianries
Copy link
Member

@toge We could add a larger container to this project. Let me check it.

@meftunca
Copy link

Any progress on the project? @uilianries

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@uilianries
Copy link
Member

Conan v1 pipeline ✔️

All green in build 9 (1114251fdadc1f346ac3e5042ef2a1716bd19ce6):

* **faker-cxx/2.0.0:**
  All packages built successfully! ([All logs](https://c3i.jfrog.io/c3i/misc/summary.html?json=https://c3i.jfrog.io/c3i/misc/logs/pr/24018/9/faker-cxx/2.0.0//summary.json))

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.

Failure in build 9 (1114251fdadc1f346ac3e5042ef2a1716bd19ce6):

* **faker-cxx/2.0.0:**
  CI failed to create some packages ([All logs](https://c3i.jfrog.io/c3i/misc-v2/summary.html?json=https://c3i.jfrog.io/c3i/misc-v2/logs/pr/24018/9-windows-msvc/faker-cxx/2.0.0//summary.json))
  Logs for packageID `01c9f3604dd77253527cc9af19c3955254c76cc6`:
  ```
  [settings]
  arch=x86_64
  build_type=Release
  compiler=msvc
  compiler.cppstd=20
  compiler.runtime=dynamic
  compiler.runtime_type=Release
  compiler.version=193
  os=Windows
  [options]
  */*:shared=True
  ```
  
  
      
        
      
  
        
      
  
      
    
  ```
  [...]
  WARN: deprecated: Usage of deprecated Conan 1.X features that will be removed in Conan 2.X:
  WARN: deprecated:     'cpp_info.names' used in: fmt/10.2.1
  
  ======== Testing the package ========
  Removing previously existing 'test_package' build folder: C:\J\workspace\prod-v2\bsr\cci-4a790a97\recipes\faker-cxx\all\test_package\build\msvc-193-x86_64-20-release
  faker-cxx/2.0.0 (test package): Test package build: build\msvc-193-x86_64-20-release
  faker-cxx/2.0.0 (test package): Test package build folder: C:\J\workspace\prod-v2\bsr\cci-4a790a97\recipes\faker-cxx\all\test_package\build\msvc-193-x86_64-20-release
  faker-cxx/2.0.0 (test package): Writing generators to C:\J\workspace\prod-v2\bsr\cci-4a790a97\recipes\faker-cxx\all\test_package\build\msvc-193-x86_64-20-release\generators
  faker-cxx/2.0.0 (test package): Generator 'CMakeDeps' calling 'generate()'
  faker-cxx/2.0.0 (test package): CMakeDeps necessary find_package() and targets for your CMakeLists.txt
      find_package(faker-cxx)
      target_link_libraries(... faker-cxx::faker-cxx)
  faker-cxx/2.0.0 (test package): Generator 'CMakeToolchain' calling 'generate()'
  faker-cxx/2.0.0 (test package): CMakeToolchain generated: conan_toolchain.cmake
  faker-cxx/2.0.0 (test package): CMakeToolchain generated: C:\J\workspace\prod-v2\bsr\cci-4a790a97\recipes\faker-cxx\all\test_package\build\msvc-193-x86_64-20-release\generators\CMakePresets.json
  faker-cxx/2.0.0 (test package): CMakeToolchain generated: C:\J\workspace\prod-v2\bsr\cci-4a790a97\recipes\faker-cxx\all\test_package\CMakeUserPresets.json
  faker-cxx/2.0.0 (test package): Generator 'VirtualRunEnv' calling 'generate()'
  faker-cxx/2.0.0 (test package): Generating aggregated env files
  faker-cxx/2.0.0 (test package): Generated aggregated env files: ['conanrun.bat', 'conanbuild.bat']
  
  ======== Testing the package: Building ========
  faker-cxx/2.0.0 (test package): Calling build()
  faker-cxx/2.0.0 (test package): Running CMake.configure()
  faker-cxx/2.0.0 (test package): RUN: cmake -G "Visual Studio 17 2022" -DCMAKE_TOOLCHAIN_FILE="generators/conan_toolchain.cmake" -DCMAKE_INSTALL_PREFIX="C:/J/workspace/prod-v2/bsr/cci-4a790a97/recipes/faker-cxx/all/test_package" -DCMAKE_POLICY_DEFAULT_CMP0091="NEW" "C:/J/workspace/prod-v2/bsr/cci-4a790a97/recipes/faker-cxx/all/test_package"
  -- Using Conan toolchain: C:/J/workspace/prod-v2/bsr/cci-4a790a97/recipes/faker-cxx/all/test_package/build/msvc-193-x86_64-20-release/generators/conan_toolchain.cmake
  -- Conan toolchain: CMAKE_GENERATOR_TOOLSET=v143
  -- Conan toolchain: C++ Standard 20 with extensions OFF
  -- The CXX compiler identification is MSVC 19.36.32532.0
  -- Detecting CXX compiler ABI info
  -- Detecting CXX compiler ABI info - done
  -- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.36.32532/bin/Hostx64/x64/cl.exe - skipped
  -- Detecting CXX compile features
  -- Detecting CXX compile features - done
  -- Conan: Target declared 'faker-cxx::faker-cxx'
  CMake Error at build/msvc-193-x86_64-20-release/generators/cmakedeps_macros.cmake:67 (message):
    Library 'faker-cxx' not found in package.  If 'faker-cxx' is a system
    library, declare it with 'cpp_info.system_libs' property
  Call Stack (most recent call first):
    build/msvc-193-x86_64-20-release/generators/faker-cxx-Target-release.cmake:23 (conan_package_library_targets)
    build/msvc-193-x86_64-20-release/generators/faker-cxxTargets.cmake:24 (include)
    build/msvc-193-x86_64-20-release/generators/faker-cxx-config.cmake:16 (include)
    CMakeLists.txt:4 (find_package)
  
  
  -- Configuring incomplete, errors occurred!
  See also "C:/J/workspace/prod-v2/bsr/cci-4a790a97/recipes/faker-cxx/all/test_package/build/msvc-193-x86_64-20-release/CMakeFiles/CMakeOutput.log".
  
  ERROR: faker-cxx/2.0.0 (test package): Error in build() method, line 20
  	cmake.configure()
  	ConanException: Error 1 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.

@toge The faker-cxx is not prepared to build with shared + Windows. It can be dropped. I'll check with the upstream.

toge and others added 2 commits June 28, 2024 20:06
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
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.

LGTM. Thank you for packaging this project!

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 11 (f1cf8ee0dac7a0518d7d33de9b2081f5e0862243):

  • faker-cxx/2.0.0:
    All packages built successfully! (All logs)

Conan v2 pipeline ✔️

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

All green in build 11 (f1cf8ee0dac7a0518d7d33de9b2081f5e0862243):

  • faker-cxx/2.0.0:
    All packages built successfully! (All logs)

@uilianries
Copy link
Member

Any progress on the project? @uilianries

@meftunca The PR is ready now. The problem was in the upstream, as it required tons of RAM to be built using GCC. For instance, in my Linux, it stopped and took frozen after consuming 32 GB of RAM. Fortunately, the author improved the project a lot and now does not require that huge amount of memory.

@uilianries uilianries linked an issue Jun 28, 2024 that may be closed by this pull request
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.

<3 - great follow up @uilianries :)

@conan-center-bot conan-center-bot merged commit 7aee73a into conan-io:master Jun 29, 2024
12 checks passed
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] faker-cxx/1.0.0
5 participants