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

Update to EOS SDK 1.15.1 #35

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

bphillips09
Copy link

@bphillips09 bphillips09 commented Jul 6, 2022

  • Update to EOS C# SDK 1.15.1
  • Update Mirror
  • Add iOS/macOS Editor support

Note that iOS Framework and macOS Dev Tool are not included in this as they are >100MB

@RoxDevvv
Copy link

Hi, what is the news in your commit am going to start a project and i would like to know
thank you

@bphillips09
Copy link
Author

@Roxgame95 It is working fine for me with limited testing. I have only tested Device ID and P2P on iOS/Windows/macOS Catalyst. I haven't tested Lobby or other functionality yet or other platforms (Android/Linux/macOS Native) but in theory it should work fine.

@cabbagegod
Copy link

Can't seem to get it working. Just trying to run the developer test just to see how it goes but can't even get that. For whatever reason it wont launch the auth account portal.

@dperez-sn
Copy link

dperez-sn commented Jan 5, 2023

I was able to get it to work locally but changing line 49 in EoSLobbyUI to this
netManager.networkAddress = attributes.Find((x) => x.Data?.Key == hostAddressKey).Data.Value.Value.AsUtf8;

I also changed all instances of 'Value.ToString' for the lobby address to 'Value.Value.AsUtf8' to fix the naming issues.

@RealHandy
Copy link

RealHandy commented Jan 29, 2023

I was able to get it to work locally but changing line 49 in EoSLobbyUI to this netManager.networkAddress = attributes.Find((x) => x.Data?.Key == hostAddressKey).Data.Value.Value.AsUtf8;

I also changed all instances of 'Value.ToString' for the lobby address to 'Value.Value.AsUtf8' to fix the naming issues.

Yes! It's working for me, but Value.Value.AsUtf8 is even needed for, like, getting the lobby name out of the lobby info when you get a list of lobbies. Otherwise, Value.ToString() returns the name of the Epic namespace of the data type, or something similar.

@EliasVal
Copy link

EliasVal commented Aug 9, 2023

After updating the SDK to v1.15.5, My editor crashes every time I exit play mode. Any ideas?

@EliasVal
Copy link

EliasVal commented Aug 9, 2023

I managed to narrow down the error, in EOSSDKComponent.cs:502:

#if UNITY_EDITOR
            if (libraryPointer != IntPtr.Zero) {
                Bindings.Unhook();

                // Free until the module ref count is 0
                while (FreeLibrary(libraryPointer) != 0) { }

                libraryPointer = IntPtr.Zero;
            }
#endif

Commenting out these lines fixes the crash.
My guess is that it's something with the while loop, probably going into an infinite loop, since this is how Unity usually behaves when a loop never ends.

UPDATE: I was right, commenting out just the while (FreeLibrary(libraryPointer) != 0) { } line stops the crashing.
UPDATE 2: I was wrong, It isn't the while loop that crashes the game, it is the FreeLibrary method.

@EliasVal
Copy link

EliasVal commented Aug 9, 2023

As I've stated in the previous comment, the FreeLibrary method is what's actually causing the crash. I don't know how to fix this unfortunately so we'd have to wait for someone with more knowledge on how this works.

I've pinged FakeByte in the Discord server.

@EliasVal
Copy link

also, the libc++_shared.so files should be added back to the Plugins/Android/libs/ arm64-v8a & armeabi-v7a folders, otherwise the EOSSDK won't work on Android

@MadkevOP7
Copy link

MadkevOP7 commented Mar 28, 2024

Hello, EOS transport is no longer maintained by FakeByte; I'm continuing to use it for my project, tho, and it has updated to SDK 1.15.1 as well. If you are interested, I'm hoping to collab to maintain it ;)

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.

7 participants