-
Notifications
You must be signed in to change notification settings - Fork 6
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
Any plans to port to Python ? #5
Comments
I started writing it in python, but realised I needed some advanced regular expression stuff that I knew how to do in perl. ifcmerge isn't very complex, I'm sure it can be rewritten in python, contributions welcome. Though importantly git requires that a mergetool is an executable, so on Windows it needs to be an EXE file (which I can do with perl, not sure about python). The Perl version doesn't have any dependencies, so it will run on any machine with perl, ie. Linux and OSX, without any problems. |
Thanks for the reply. Perl does indeed have some awesome regex functionality. Just a quick search returns https://www.tomshardware.com/how-to/create-python-executable-applications |
I have had some experience with pyinstaller, the only disadvantage I see is that the exe can get a very large file size, since it just an script with the whole python interpreter and all the deps in one file...
Source: https://pyinstaller.org/en/stable/operating-mode.html |
@brunopostle Did you use the standard module re
Source: https://docs.python.org/3/library/re.html I don't know any perl, however i learnt doing regex with the re module with this beginner tutorial https://automatetheboringstuff.com/2e/chapter7/ That might help |
It looks like there is no significant difference between installing a perl or python version (the current windows EXE bundles a perl interpreter just as a python version would need to bundle a python interpreter). Python is much better understood by people who might want to work on these tools, so switching would be a good thing, but somebody would need to contribute this (and check it is as fast, perl text handling is very efficient). |
Perl is awesome. The issue is the need to support the perl interpreter as a 3rd party dependency.
Thanks for the amazing work on this!
The text was updated successfully, but these errors were encountered: