Added STEAMWORKS_NET define #484
Conversation
|
Ewww you use that Java style K&R formating 🤢 barbarian 🤣 Jokes aside happy days we have a script define now that other assets can use to detect install ... I will let the groups I work with know its present going forward. |
|
I don't actually prefer it anymore; especially because that's not the default for C# in VS IIRC, but -- switching should be a change on it's own and it's rather pointless especially if it creates merge conflicts with forks |
|
As a developer on a project that also builds for many non steam platforms I prefer the opt in approach on the defines rather than it being defined for me and I have to undefine it in a build step when building for xbox ect |
|
Shouldn't ever be undefining this one ... it only lets other code know this code exists in the project so its not taking over for DISABLESTEAMWORKS or anything Also as for platform compilation the assembly defs do that for you as well ... they wouldn't compile for XBox anyway so no need to use script defines to strip them If Unity ever gets off its backside regarding Package Manager then PackageManager.Client.List would let us programatically check what Package Manager packages where installed and check there version ... so a much better system ... but that isn't really here just yet and it doesn't account for manually installed code That said I can give a think on a better solution, this method is the method used by any asset I have seen thus far but if something is better its better ... I'll ahve a look |
cc: @JamesMcGhee