Use BuildTarget instead of #defines about unity_sdk

[b0rje]

Hi @uerceg,
In the MenuItems its perfectly fine to use #if. But for everthing thats is executed during a buildprocess (especially when running unity in batchmode for creating a build as we do on our buildservers) you should use the BuildTarget (which is passed as a parameter in the OnPostProcess methods). This is the source as it worked for us:

AdjustEditor.cs.zip

[aphexyuri]

This is true - we ran into the same issue and I had to change to using 'target'. I should also add that I'm only seeing this with Adjust (we integrate with many other SDK). I to currentplatform checks any numerous places, and values are always iOS, but the #if UNITY_IPHONE never gets executed - very weird behaviour

[wellle]

Thanks for the suggestion, we'll do that in the upcoming update 👍

[uerceg]

@b0rje @aphexyuri Sorry for delay in solving this issue, planning to add it to next release. I see what you mean, but do you have advice how to detect platform for MenuItem entries in AdjustEditor.cs?

[uerceg]

Hi @b0rje

And thank you very much for this example, we'll probably add it to our next release. Will ping in here once it's live.

Cheers!

[uerceg]

Added to current PR, will be live soon.

For reference: e6cbd18

[uerceg]

Included in latest v4.7.0: https://github.com/adjust/unity_sdk/releases/tag/v4.7.0

Thank you one more time for suggestion.

Cheers!