Skip to content

build: Allow custom screenshot impl for downstream SDKs#1697

Merged
jpnurmi merged 2 commits into
masterfrom
build/custom-screenshot
May 12, 2026
Merged

build: Allow custom screenshot impl for downstream SDKs#1697
jpnurmi merged 2 commits into
masterfrom
build/custom-screenshot

Conversation

@jpnurmi
Copy link
Copy Markdown
Collaborator

@jpnurmi jpnurmi commented May 4, 2026

Allow downstream SDKs to select SENTRY_SCREENSHOT=custom so sentry-native skips the built-in screenshot capture source and expects an implementation from the embedding build.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 7083f91. Configure here.

Comment thread CMakeLists.txt
Comment thread CMakeLists.txt
@jpnurmi jpnurmi force-pushed the build/custom-screenshot branch from 7083f91 to 9b6b206 Compare May 4, 2026 19:18
Allow downstream SDKs to select SENTRY_SCREENSHOT=custom so sentry-native
skips the built-in screenshot capture source and expects an implementation
from the embedding build.

Co-Authored-By: OpenAI Codex <noreply@openai.com>
@jpnurmi jpnurmi force-pushed the build/custom-screenshot branch from 9b6b206 to bd5604f Compare May 12, 2026 12:49
@jpnurmi jpnurmi merged commit c81aecb into master May 12, 2026
98 of 99 checks passed
@jpnurmi jpnurmi deleted the build/custom-screenshot branch May 12, 2026 15:47
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