-
Notifications
You must be signed in to change notification settings - Fork 5
crow #7
Comments
You mean, have you started trying to make a conan package for it, or that you are experimenting with crow, and that you might submit a PR to the crow repo, so the Boost dependency is upgraded to a more modern one? Also, would you need sqlpp11 for your use case? Even if it is optional for crow, if you plan to use it, it might worth having its own issue. |
Haven't started on its conan package yet, just started code-reviewing on the boost dependency side. Found an old issue that tried to remove the boost dependency entirely, so I'm keeping that in the back of my head as I review. Hmm, good point about sqlpp11, might as well add another issue for it |
@hoxnox Can you please add a LICENSE file with a license for your recipe (not the packaged library)? |
Can be closed in favour of conan-io/conan-center-index#3064 |
@Croydon Aww but was keen on my 4 year old issue being resolved 😝 |
@SamuelMarks It is work in progress now, have a look here: conan-io/conan-center-index#3123 |
Yeah I know, just not sure why this #3064 wasn't closed in favour of this old issue of mine (considering crow itself hasn't been updated in as many years) Doesn't matter anymore, seeing as the other issue has now been closed by the bot, thanks to the merge |
The wishlist here is deprecated. New package request should be made directly in the issue tracker of the Conan Center Index and if we get a request both here and in the Index, we are always closing in favour of the Index issue |
Requirements (from CMakeLists.txt):
PS: I've started experimenting with it, and am considering preparing a PR that'll bump the boost version
The text was updated successfully, but these errors were encountered: