bsimser / Inventory-Pro

https://devdog.io
MIT License
2 stars 0 forks source link

Enabling integration does not re-compile code correctly #260

Open bsimser opened 5 years ago

bsimser commented 5 years ago

Original report by Joris Huijbregts (Bitbucket: jjahuijbregts, ).


When enabling an integration in the integrations helper an scripting define symbol is set. As of Unity 2017.x the code no longer compiles with the new scripting define symbol and users have to manually force a recompile for it to detect the new define symbol.

bsimser commented 5 years ago

Original comment by Herman Tulleken (Bitbucket: herman_tulleken, ).


I am not able to replicate this with Unity 5, 2017 or 2018.

I tested with Rewire. If I enable the integration, it recompiles and subsequently has compiler errors because I don't have the Rewire integration. If I disable it again, it recompiles and the errors are gone. I also put a break point on line 149 (where AssetDatabase.ImportAsset is called), and the breakpoint is hit when a integration is enabled or disabled.