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

breakpad: delete #31746

Merged
merged 1 commit into from Nov 16, 2017
Merged

breakpad: delete #31746

merged 1 commit into from Nov 16, 2017

Conversation

orivej
Copy link
Contributor

@orivej orivej commented Nov 16, 2017

Motivation for this change

This library does not compile with the current glibc (#31696) and is unused since 74f7db0.

Things done

nox-review: nothing changed

/cc @abbradar

This library does not compile with the current glibc and is unused since
74f7db0.
@orivej orivej requested a review from abbradar November 16, 2017 17:23
@lukateras
Copy link
Member

lukateras commented Nov 16, 2017

Why delete if it could be marked as broken?

It's still actively developed and I'm sure glibc compatibility will be eventually resolved.

@lukateras
Copy link
Member

Actually, it seems to be resolved at master: google/breakpad@bddcc58

@abbradar
Copy link
Member

I'd favor deletion -- we don't have this package used or tested anyhow now and if there are any external users they can step up and request readdition.

@abbradar
Copy link
Member

@orivej Thanks for the patch!

@orivej orivej mentioned this pull request Nov 16, 2017
22 tasks
@orivej orivej deleted the breakpad branch November 16, 2017 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants