SRombauts / UEGitPlugin

Unreal Engine 5 Git LFS 2 Source Control Plugin (beta)
http://srombauts.github.io/UEGitPlugin
MIT License
810 stars 165 forks source link

Unable to use on mac #23

Closed LoeizD closed 9 years ago

LoeizD commented 9 years ago

I followed the setup guide, I use the github visual client. When I launch unreal I get the following messages:

capture d ecran 2015-02-28 a 13 03 36

SRombauts commented 9 years ago

Yes, as stated in the Know Issues list of the README, the plugin is "Windows only (64 bits) -> Mac compiles but needs testing/debugging (Linux source control is not supported by Editor)"

This is because I have no access to a Mac, sorry!

If you are willing to help, we could add more debug logs to your copy of the plugin to try fix this

LoeizD commented 9 years ago

I don't know much about git, but if you tell me what to do I sure can help.

tgunr commented 9 years ago

Who created the UE4Editor-GitSourceControl.dylib for Mac that is distributed with the 4.7 engine. I am a Mac developer and we are planning on using Unreal and having a GIT plugin is a must.

SRombauts commented 9 years ago

Epic did, and perhaps the Git plugin was tested successfully on certain configuration, but does not handle well certain paths (like spaces on the path to the project) or other things they did not try

SRombauts commented 9 years ago

Today I had a chance to test the Git plugin with a Mac, and unfortunately it does not work at all, whether there is a space or not in the path to the Project.

The problem come from the double-quotes added around path to handle space (like "/Users/Xxx/Unreal Projects/TestGit") : those does not work with the current UE4 FPlatformProcess::ExecProcess() method on Mac.

I've try with simple-quote instead, but this does not work either.

I've try removing them altogether, and this works if you don't have any space in any of your paths.

I'll report this to Epic, as I have no way to work seriously on resolving this issue.

SRombauts commented 9 years ago

Fixed on the Unreal Engine master branch with my pull request: https://github.com/EpicGames/UnrealEngine/commit/2196a60dd6e5e59576a0e6ade6c3a4b621b4fe86