Add rpclib as a submodule and compile directly from source

[8W9aG]

This fixes the OSX linker issues when dealing with rpclib.

• Removes the binary distributions of rpclib and replaces them with a source compile
• Adds a submodule to rpclib

As a sidenote, I'm not sure whether you want to use the git subtree or submodule workflow here.

[msftclas]

Hi @8W9aG, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, MSBOT;

[8W9aG]

Also would be grateful if people could test whether this still works on Windows as well (I think it should)

[DJISDKUser]

$ git fetch origin pull/16/head:master
$ git checkout add-rpclib-source-compile
$ git submodule update --init --recursive
$ mkdir cmake/build
$ cd cmail/build/; cmake .. -DCMAKE_INSTALL_PREFIX=/usr/local/Cellar/eigen@3.2/3.2.10/
$ make -j8
...
[ 98%] Linking CXX executable HelloDrone
[ 98%] Built target HelloDrone
1 warning generated.
[100%] Linking CXX executable DroneShell
[100%] Built target DroneShell

(no errors)

[DJISDKUser]

Inside of Unreal/Plugins/AirSim/readme.md it says "Create black C++ Unreal project without starter content", do you mean To select "New Project"-> "C++" -> "Basic Code"?

[DJISDKUser]

Also for me on OSX, the suggested Modular Neighborhood Pack does not support the current Unreal Engine 4.15.0 is it OK to tell it to use the closest version being 4.14 when it asks?

[lovettchris]

Yes you want Unreal 4.14

[DJISDKUser]

Thanks @lovettchris ... Sorry I am not an Unreal user... so next up we have "- Project Settings > Maps - set the map". I am as stated above using Modular Neighborhood Pack, What should I be setting for "Editor Startup Map", and "Game Default Map", "Default GameMode", and "Game Instance Class".

These values are all greek to me, as is what it means to "set" a Map as it were, short of making it default. I just don't know if I want Template_Default, Demo_Map, Overview, or

Also for anyone wondering this is how you can copy the plugins over:
$ cp -rfv Unreal/Plugins ~/Documents/Unreal\ Projects/YourAirSImProjectName/

[DJISDKUser]

For OSX the instructions get a little odd here:
https://github.com/Microsoft/AirSim/blob/master/docs/build.md

"Ready, Set, Go!
You are all ready to go now!
Right click on the Unreal project .uproject file, then Generate Visual Studio Project files.
Double click on .sln file to open the solution."

Is there any alternate wording for us?

FWIW My project file was in: ~/Documents/Unreal\ Projects/AirSImProjectName/AirSImAirSImProjectName.uproject

[sytelus]

Hi @d13-kevinf

The readme at Unreal/Plugins/AirSim/readme.md was old and has been deleted. If you are using Neighbourhood pack, set the Editor Starter Map as well as Game Default Map to Demo_Map in Project Settings > Maps & Modes. Don't change Default Game Mode in that screen. Go to World Settings and then set Game Mode = SimModeGame.

On OSX you will need to generate XCode project files. May be below might answer your question:

https://answers.unrealengine.com/questions/45320/how-to-create-xcodeproj-for-pc-started-project.html

I'm also looking at your PR. Please also free free to submit osx_build.md with steps you followed so far.

[sytelus]

I've pushed this change with slightly different variation.

1. Put rpclib as submodule in external/rpclib folder.
2. For Windows: Modify build.cmd for Windows to init, update as well as build this submodule in its own folder. Modify build.cmd for Windows to copy the build output and include folder to AirLib/deps/rpclib.
3. For Linux: Modify CMakeLists.txt with add_subdirectory(external/rpclib). Modify CMakeLists.txt with POST_BUILD command to copy build output and include folder from external/rpclib to AirLib/deps/rpclib folder.

So, in essence, the folder structure under AirLib/deps remains same so no other changes are required.

I've tested build in Windows however not under Linux yet due to GCC upgrade issue on my current machine. It would be great if you get chance to try it out or if you have any suggestion for any alternative way to achieve above in CMakeLists.txt.