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

Compilation error for Linux/UE-4.17/clang-3.8 #223

Closed
mmmaat opened this issue Sep 7, 2017 · 3 comments
Closed

Compilation error for Linux/UE-4.17/clang-3.8 #223

mmmaat opened this issue Sep 7, 2017 · 3 comments

Comments

@mmmaat
Copy link
Contributor

mmmaat commented Sep 7, 2017

Hello,

I'm trying to setup UnrealEnginePython on UE-4.17 within Ubuntu 16.04. I get one warning and one error...

  • Warning

Once I created a new empty project and cloned the Python plugin in .../Plugins/UnrealEnginePython, the UE4Editor complains with the following message: This project requires the 'UnerealEnginePython' plugin, which is not compatible with the current engine version. Would you like to disable it? You will no longer be able to open assets created using it. Just answering No seems ok, but I guess there is a little issue in the plugin integration with UE-4.17.

  • Error [1/4] Compile Module.UnrealEnginePython.2_of_3.cpp

In file included from .../Plugins/UnrealEnginePython/Source/UnrealEnginePython/Private/UEPyEditor.cpp:231:16: error: default initialization of an object of const type 'const FString' without a user-provided default constructor
const FString mobile_device;

The piece of code is here, just adding the initialization in bold fix the error...

#if ENGINE_MINOR_VERSION >= 17
const FString mobile_device = FString("");
GEditor->RequestPlaySession(&v, &r, false, false, mobile_device);
#else
GEditor->RequestPlaySession(&v, &r, false, false);
#endif

Thank you,
Mathieu

@rdeioris
Copy link
Contributor

rdeioris commented Sep 7, 2017

Hi, Mathieu, thanks for reporting, would you like to make a pull request for the FString initialization ?

@mmmaat
Copy link
Contributor Author

mmmaat commented Sep 7, 2017

Ok I'm doing it!

@mmmaat
Copy link
Contributor Author

mmmaat commented Sep 7, 2017

Done ;)

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

No branches or pull requests

2 participants