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

aws-sdk-cpp: fix recipe #19612

Closed
wants to merge 1 commit into from
Closed

aws-sdk-cpp: fix recipe #19612

wants to merge 1 commit into from

Conversation

vient
Copy link

@vient vient commented Sep 6, 2023

Requirements were changed but this lib not, so it did not build with errors like

ERROR: Component 'aws-c-cal::aws-c-cal-lib' not found in 'aws-c-cal' package requirement

Specify library name and version: aws-sdk-cpp/all-versions


  • I've read the contributing guidelines.
  • I've used a recent Conan client version close to the currently deployed.
  • I've tried at least one configuration locally with the conan-center hook activated.
  • I've manually inserted these changes in downloaded package, after that conan install managed to build package successfully

Requirements were changed but this lib not, so it did not build with errors like
```
ERROR: Component 'aws-c-cal::aws-c-cal-lib' not found in 'aws-c-cal' package requirement
```
@CLAassistant
Copy link

CLAassistant commented Sep 6, 2023

CLA assistant check
All committers have signed the CLA.

@ghost
Copy link

ghost commented Sep 6, 2023

I detected other pull requests that are modifying aws-sdk-cpp/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

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


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.

See details:

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

@vient
Copy link
Author

vient commented Sep 6, 2023

Please note that there is no new functionality, this just fixes currently broken package. We've lost our artifactory so now we are stuck with broken version in conan-center.

@AbrilRBS
Copy link
Member

AbrilRBS commented Sep 6, 2023

Hi @vient thanks a lot for your contribution! We have just approved the final changes in #17125 that also include your fixes. I'm closing this one in favour of that one which should get merged today :)

@AbrilRBS AbrilRBS closed this Sep 6, 2023
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.

4 participants