conan-center-index icon indicating copy to clipboard operation
conan-center-index copied to clipboard

Compile as C

Open datalogics-staylor opened this issue 9 months ago • 12 comments

Specify library name and version: qr-code-generator/1.8.0

  • Compile as C when the package is built with the new "compile_as_c" option.

datalogics-staylor avatar May 06 '24 20:05 datalogics-staylor

Conan v1 pipeline :heavy_check_mark:

All green in build 2 (5a51ce74003a4bd8a9e7542c3623fe8665d7e3d1):

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)


Conan v2 pipeline :heavy_check_mark:

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

All green in build 2 (5a51ce74003a4bd8a9e7542c3623fe8665d7e3d1):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

conan-center-bot avatar May 06 '24 21:05 conan-center-bot

Conan v1 pipeline :heavy_check_mark:

All green in build 3 (786f9267f59f2f132dcb121768903207eba6b146):

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)


Conan v2 pipeline :heavy_check_mark:

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

All green in build 3 (786f9267f59f2f132dcb121768903207eba6b146):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

conan-center-bot avatar May 06 '24 22:05 conan-center-bot

Conan v1 pipeline :heavy_check_mark:

All green in build 4 (786f9267f59f2f132dcb121768903207eba6b146):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)


Conan v2 pipeline :heavy_check_mark:

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

All green in build 4 (786f9267f59f2f132dcb121768903207eba6b146):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

conan-center-bot avatar May 07 '24 21:05 conan-center-bot

Conan v1 pipeline :heavy_check_mark:

All green in build 5 (786f9267f59f2f132dcb121768903207eba6b146):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)


Conan v2 pipeline :heavy_check_mark:

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

All green in build 5 (786f9267f59f2f132dcb121768903207eba6b146):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

conan-center-bot avatar May 08 '24 18:05 conan-center-bot

Hi @datalogics-staylor , thanks for your contribution. Is this a case where it might make sense for the package to just build both libraries, C and C++? if the resulting library files have a different name, they can coexist, and then these can be different components? I suspect that's the case with other libraries in the same situation as well.

jcar87 avatar May 09 '24 11:05 jcar87

Hi @jcar87. That's a very good point. I looked around a bit for recipes that do both C and C++ and didn't find many, so I chose to go this route so that it doesn't build packages that aren't required. It would be a pretty minor change to just build both, so I'd be happy to go that route if that seems to make more sense. Conan and CCI are relatively new to me, so I'm still learning common usages. Thanks for the feedback!

datalogics-staylor avatar May 09 '24 14:05 datalogics-staylor

Conan v1 pipeline :heavy_check_mark:

All green in build 9 (f5ed5d637a0017ff580dd35fb5f324f37a686ee6):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)


Conan v2 pipeline :heavy_check_mark:

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

All green in build 9 (f5ed5d637a0017ff580dd35fb5f324f37a686ee6):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

conan-center-bot avatar May 13 '24 21:05 conan-center-bot

@datalogics-staylor Your change looks correct in terms of CMake, but still need to update the Conanfile recipe in order to list the new library too. Could please update the pacakge_info as well?

uilianries avatar May 14 '24 09:05 uilianries

Conan v1 pipeline :heavy_check_mark:

All green in build 10 (d44ea9f83df680e69c7ef09305deacddbd86a491):

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)


Conan v2 pipeline :heavy_check_mark:

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

All green in build 10 (d44ea9f83df680e69c7ef09305deacddbd86a491):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

conan-center-bot avatar May 14 '24 15:05 conan-center-bot

Conan v1 pipeline :heavy_check_mark:

All green in build 11 (b94fc89da2cf6bfdf0d7881a4f370595db1c145c):

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)


Conan v2 pipeline :heavy_check_mark:

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

All green in build 11 (b94fc89da2cf6bfdf0d7881a4f370595db1c145c):

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

conan-center-bot avatar May 20 '24 23:05 conan-center-bot

Conan v1 pipeline :heavy_check_mark:

All green in build 12 (79ba58e4c16e3ca48794c45aef6cc9a02fa69da8):

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)


Conan v2 pipeline :heavy_check_mark:

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

All green in build 12 (79ba58e4c16e3ca48794c45aef6cc9a02fa69da8):

  • qr-code-generator/1.6.0: All packages built successfully! (All logs)

  • qr-code-generator/1.4.0: All packages built successfully! (All logs)

  • qr-code-generator/1.5.0: All packages built successfully! (All logs)

  • qr-code-generator/1.8.0: All packages built successfully! (All logs)

  • qr-code-generator/1.7.0: All packages built successfully! (All logs)

conan-center-bot avatar May 21 '24 22:05 conan-center-bot

@franramirez688 We're just waiting for one more review - the last approval was 2 weeks ago - can we get some eyes on this one for us?

Thanks!

datalogics-robb avatar Jun 03 '24 18:06 datalogics-robb