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

feat(shim): Add another alternative shim written in rust #4229

Merged
merged 2 commits into from
Nov 10, 2021

Conversation

zoritle
Copy link
Contributor

@zoritle zoritle commented Jan 18, 2021

https://github.com/zoritle/rshim
C# version didn't handle Ctrl event properly.
C version randomly crashed.
C++ version =creating infinite processes recursively until OOM.

This version is written in rust that is safe and performant, hopefully eliminate all problems above.

see #3634

VirusTotal: Report

@beatcracker
Copy link

This is the only shim that (kinda) works with TortoiseGit for me: it's detecting Git fine and I'm even able to do stuff with it. Unfortunately, for every TortoiseGit operation I get a lot of flashing console windows, since it needs to run a bunch of Git commands.

@niheaven
Copy link
Member

niheaven commented Nov 5, 2021

Could you upload a VirusTotal report for this exe?

@niheaven niheaven changed the base branch from master to develop November 9, 2021 16:05
@niheaven niheaven merged commit 5ad8c76 into ScoopInstaller:develop Nov 10, 2021
@felixier
Copy link

felixier commented Dec 1, 2021

Could you upload a VirusTotal report for this exe?

There seems to be an issue with the exe file: https://www.virustotal.com/gui/file/03037ad0dffc96a468197aa4a7d3f040eef271fbd2c79337716436d9b649eec3/detection

niheaven pushed a commit that referenced this pull request Dec 1, 2021
rasa pushed a commit that referenced this pull request Dec 1, 2021
…)" (#4553)

This reverts commit 5ad8c76.

Co-authored-by: Zoritle <66551926+zoritle@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

None yet

4 participants