-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Libsndfile 1.0.29 cmake recipe #2597
Conversation
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
The error(s) you're seeing are probably caused by not using the
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You will also need to add a CMakeLists.txt
wrapper script (see other recipes).
Also, you will need to test this because I filled in package_info
using a wild guess.
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
3 similar comments
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
4 similar comments
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
1cf214c
to
fa424f6
Compare
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
fa424f6
to
ae71cdf
Compare
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Ok I added your proposition, fix some issue and add the requirements. I still have link issue with the external libraries:
|
ae71cdf
to
96fb18b
Compare
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Please consider MartinDelille#1 And try them in combination with the packages from pull requests: |
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
1 similar comment
Sorry, the build is only launched for Early Access Program users. You can request access writing in this issue. |
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
Co-authored-by: Anonymous Maarten <madebr@users.noreply.github.com>
Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
64e6fb9
f6d8786
to
64e6fb9
Compare
@Croydon Indeed. I just rebased the pull request on master and renamed the folder to all (that will make future version like the recent 1.0.30 easy to add). |
All green in build 40 (
|
Why wasn't this merged yet? |
@jgsogo For some reason, CCI Bot didn't merge it automatically. We have all requirements, no beta user, 3 approvals, CI result is green ... Could you please take a look? |
When I pass my mouse over @prince-chrismc and @madebr I read: only Only reviews by reviewer with write access count toward mergeability. |
Maybe the bot waits for @Croydon to approve the PR (after having requested changes in the past) ? |
Hello! After having a look to this one, first of all some comments: About the number of approvals
About the approvals:We are using the Github API to retrieve the approvals: https://api.github.com/repos/conan-io/conan-center-index/pulls/2597/reviews (see next comment), and we take into account the reviews that correspond to the latest commit in the PR (
Having a closer look to the Github API response:
I really don't know why Github is not showing all the reviews properly, any ideas related to the history of commits/reviews in this PR? |
I'm copying here the API output, as it is now: https://api.github.com/repos/conan-io/conan-center-index/pulls/2597/reviews
|
There is a problem with github API, because @SSE4 , @uilianries and your (@jgsogo ) review all happenned after the last push from @MartinDelille |
Yes, my approval is not shown in the API either... 😕 Oh! https://www.githubstatus.com/ API requests are under maintenance, let's wait a little bit to see what happens after the maintenance |
API request is green again on https://www.githubstatus.com/ |
Looks like the API response is not updated with (at least) my review which does belong to the latest commit for sure. Maybe it is just a matter of time and eventually they will achieve congruency among the different databases (the one for the UI and the one for the API requests)... We can come back to this PR as see the response from https://api.github.com/repos/conan-io/conan-center-index/pulls/2597/reviews, meanwhile I'm merging it manually. Sorry for the inconvenience. I'm afraid other PRs might be affected by this same issue :/ |
Specify library name and version: libsndfile/1.0.29
conan-center hook activated.
depends on
Ok this is a draft but unfortunately it fails linking the library: