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

Needs modernizing and partial rewriting #1

Open
OdinVex opened this issue Dec 16, 2021 · 3 comments
Open

Needs modernizing and partial rewriting #1

OdinVex opened this issue Dec 16, 2021 · 3 comments

Comments

@OdinVex
Copy link

OdinVex commented Dec 16, 2021

This project needs some rewriting to work with the latest versions of Unreal Engine. Most of its compile issues are relating to how everything is #included, or slate deprecations, but there is also the issue of UE_LOG not working correctly for this plugin.

I'd like to make note that some use Linux. Just allow us to specify a path to the binary for git, and we're content. I've gotten it to compile (a bit of tinkering definitely required) for Linux, hopefully it'll work.

@OdinVex
Copy link
Author

OdinVex commented Mar 3, 2022

Adding a follow up: OdinVex@62631df#commitcomment-67861467

I got it working on Linux, assumed still working with Windows, and untested Mac. It works on UE >= 4.25 as well. Note: You assumed local branches are master when they could be anything. I'm thinking about rewriting it to including a refresh button to:

1. List all local branches.
2. List all remote hosts.
3. List all remote branches specific to each remote host.
4. Allow you to push to any.
5. Optionally assume signing (GPG).
6. Allow refreshing 1-3 to reflect any environmental changes.

If I get around to it. ^_^;

@iUltimateLP
Copy link
Owner

Thanks! Unfortunately, I really don't find the time to further support this plugin, but I'm happy for the contribution!

@OdinVex
Copy link
Author

OdinVex commented Mar 3, 2022

Thanks! Unfortunately, I really don't find the time to further support this plugin, but I'm happy for the contribution!

I'll try to find some time to test on Windows, it should already work on Mac (untested, but the shell is quite similar to Linux). I've got it working flawlessly on Linux, though I'm wondering how I might want to implement everything without trying to invent a whole git client. 🤔

As for the master bit in the code, it was never necessary in the way it was being used and people have begun to move away from the whole 'master and slave' terminology anyway, which is nice. ☮️ Other than that, the plugin just needed refreshing past 4.25 and shell fixes, easily compared in my current temporary fork.

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