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

Make crash report IDs easy to copy #8540

Merged
merged 3 commits into from
Apr 8, 2021
Merged

Make crash report IDs easy to copy #8540

merged 3 commits into from
Apr 8, 2021

Conversation

fmoc
Copy link
Contributor

@fmoc fmoc commented Apr 7, 2021

Closes #8130.

Updates libcrashreporter-qt submodule to the latest commit to include dschmidt/libcrashreporter-qt#25.

Also makes building all targets a little faster by excluding all targets from libcrashreporter-qt which are not explicitly linked by any other target. (They can still be buillt manually.)

Fabian Müller added 2 commits April 7, 2021 18:10
This enables a "click to copy crash report ID" feature in owncloud_crash_reporter.

See dschmidt/libcrashreporter-qt#25 for more information.
Makes "make all" runs faster by excluding third-party targets, e.g., tests or libraries, which no other target in the main project links to, from building by default.
@fmoc
Copy link
Contributor Author

fmoc commented Apr 8, 2021

I'm not sure why the CI failed, but I'm relatively sure my changes are not the cause. @TheOneRing if you have a minute, could you please take a quick look?

@TheOneRing
Copy link
Member

Yes fixing the ci. Just go ahead and merge 😊

@fmoc fmoc force-pushed the work/update-crashreporter branch from d702301 to c69d744 Compare April 8, 2021 13:37
@fmoc fmoc merged commit bd656b5 into master Apr 8, 2021
@fmoc fmoc deleted the work/update-crashreporter branch April 8, 2021 13:37
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.

Button for copy ID for crash reporter in client
3 participants