SentrySubsystem platform includes don't play nicely with SN DBS

[mbaig-wildlight]

The platform agnostic header that generates the actual header to include by concatenating the platform token with the "SentrySubsystem" suffix is causing issues with dependency discovery on SN DBS and resulting in failed builds when the expected header isn't found.

We're working around it by not using the SENTRY_COMPILED_PLATFORM_HEADER macro and instead branching on the various platforms we support similarly to how PlatformSentryId.h handles platform switching. Figured I'd bring it up here incase you want to rework that in a future version or if other people run into the issue.

[tustanivsky]

Hey @mbaig-wildlight, thanks for bringing this up!

The main reason we use SENTRY_COMPILED_PLATFORM_HEADER (a wrapper around Unreal’s COMPILED_PLATFORM_HEADER) is to simplify including headers from private plugin extensions that provide platform-specific implementations for consoles. So, in case with SentrySubsystem we can’t follow the same approach as with other entities like SentryId simply because the corresponding platform defines aren’t available in engine out of the box.

The workaround you’ve suggested is a valid option that we can recommend to others who encounter this issue.