search

conan-io / conan-center-index

Recipes for the ConanCenter repository
https://conan.io/center
MIT License
921 stars 1.66k forks source link

raylib: Add some useful headers to res folder #24472

Open AbrilRBS opened 4 days ago

AbrilRBS commented 4 days ago

Summary

Changes to recipe: *raylib/[]**

Motivation

As explained in https://github.com/conan-io/conan-center-index/issues/24368, the library does not package some useful files that users might want to use as part of their build. This is similar to the imgui situation, so the solution is also similar: Add those files to the res folder, where users can then copy them from

Details

The linked issue and subsequent upstream issue have all the relevant info

As a draft until I get a chance to update the hook that will fail

cc @Ezbob

conan-center-bot commented 4 days ago

Conan v1 pipeline :heavy_check_mark:

All green in build 1 (ccad4a68dd41541f2cc8c778bfeacbfa375f19d9):

Conan v2 pipeline :heavy_check_mark:

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

All green in build 1 (ccad4a68dd41541f2cc8c778bfeacbfa375f19d9):

Ezbob commented 1 day ago

Hey @AbrilRBS , big thanks for this. I have a noob question though, how do you use/copy these headers in your conan file from a consumers perspective?

conan-center-bot commented 1 day ago

Conan v1 pipeline :x:

Sorry, the build is only launched for Access Request users. You can request access writing in this issue.

Conan v2 pipeline :x:

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.

Sorry, the build is only launched for Access Request users. You can request access writing in this issue.

AbrilRBS commented 19 hours ago

Hi @Ezbob, not a noob question at all, I should have been more clear as to how to proceed, sorry about that!

We're going to make this available thru a conf as @uilianries suggested - I'll post a how-to after I commit the changes :)