Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WebGL以外ではOpenWindow()が無効のためIL2CPPがエラーになるので、プラットフォーム依存 #define で囲って回避 #4

Merged

Conversation

zurachu
Copy link
Contributor

@zurachu zurachu commented Dec 28, 2018

こちらを使用したプロジェクトをそのままAndroid対応しようと思い、
Scripting Backend: IL2CPPでビルドしたら、呼び出していなくても以下のエラーになったため、
回避策として、OpenWindow周りを#if UNITY_WEBGLディレクティブで囲いました。

${ProjDir}/Library/il2cpp_android_armeabi-v7a/il2cpp_cache/6A2B8A05D149B3D067E66F08EC7C1266.o: In function `UnityRoomTweet_OpenWindow_m1487466012':
${ProjDir}/Temp/StagingArea/Il2Cpp/il2cppOutput/Bulk_Assembly-CSharp_3.cpp:45969: undefined reference to `OpenWindow'
${ProjDir}/Temp/StagingArea/Il2Cpp/il2cppOutput/Bulk_Assembly-CSharp_3.cpp:45969: undefined reference to `OpenWindow'
clang++: error: linker command failed with exit code 1 (use -v to see invocation)

IL2CPPではなくMonoだと大丈夫だったのですが、Android 64bit対応の件もありましたので…。
http://d.hatena.ne.jp/nakamura001/20180717/1531822914

if (Application.platform == RuntimePlatform.WebGLPlayer)#if UNITY_WEBGLかぶせているのが体裁悪い感じもしますので、何らか方針ありましたら良きにはからっていただけますと幸いです。

${ProjDir}/Library/il2cpp_android_armeabi-v7a/il2cpp_cache/6A2B8A05D149B3D067E66F08EC7C1266.o: In function `UnityRoomTweet_OpenWindow_m1487466012':
${ProjDir}/Temp/StagingArea/Il2Cpp/il2cppOutput/Bulk_Assembly-CSharp_3.cpp:45969: undefined reference to `OpenWindow'
${ProjDir}/Temp/StagingArea/Il2Cpp/il2cppOutput/Bulk_Assembly-CSharp_3.cpp:45969: undefined reference to `OpenWindow'
clang++: error: linker command failed with exit code 1 (use -v to see invocation)
@naichilab
Copy link
Owner

ありがとうございます。WebGL以外あまり確認できていないので助かります。
#if 二重はたしかに気になりますが範囲も狭いのでこのままにしておきます〜。

@naichilab naichilab merged commit f8e8934 into naichilab:master Dec 31, 2018
@naichilab naichilab mentioned this pull request Dec 31, 2018
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