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

I have removed the instances of abrt-hook-cpp. #27

Closed
wants to merge 3 commits into from

Conversation

shivam5522
Copy link

@shivam5522 shivam5522 commented Jan 12, 2021

There are some instances of abrt-addon-cpp which I have left, so should I remove them too? Issue #21

@mgrabovsky mgrabovsky self-requested a review January 14, 2021 12:55
Copy link
Contributor

@mgrabovsky mgrabovsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your pull request. However, it seems you removed a bit too much than what was requested.

Issue #21 pertains to the removal of abrt-hook-ccpp, which was our own version of what is now systemd-coredump. This does not mean that we dropped support for C and C++ completely. I recommend you read the related Fedora wiki page first, to get an idea what changed and which parts of the documentation can be removed.

Specifically, we want to remove mentions of abrt-hook-ccpp and of the abrt-ccpp systemd service. It would also be beneficial to mention that Fedora (and therefore abrt) now use the systemd-coredump service by default.

@mgrabovsky
Copy link
Contributor

Closing because of no activity for more than a month.

@mgrabovsky mgrabovsky closed this Mar 1, 2021
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.

2 participants