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

[sentry-crashpad] Remove #24396

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MartinDelille
Copy link
Contributor

Summary

Remove recipe: sentry-crashpad/*

Motivation

Since #20762, sentry-native doesn't rely on sentry-crashpad. It then make no sense to keep it in this repository

Details


@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

Changes not allowed in build 1:

[sentry-crashpad]

Removing recipes is forbidden. Keep sentry-crashpad as deprecated..


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.

Changes not allowed in build 1:

[sentry-crashpad]

Removing recipes is forbidden. Keep sentry-crashpad as deprecated..

@AbrilRBS
Copy link
Member

CCI's policy is to not outright delete recipes unless there's a major issue, and instead mark them as deprecated=True, but I'll check with the team what to do in this case, thanks for your patience! :)

@AbrilRBS AbrilRBS self-assigned this Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants