[Proposal] Replace current C#-based shim with C version

[rasa]

Per:
#3294: Killing the shim process does not kill the child process,
#2921: Wait for console applications only,
#2339: python in cmd Ctrl+C problem,
#2006: Windows cmd gvim.exe shim waits for subprocess,
#1896: Handle Control-C in shim wrapper,
#1606: Support shims to both console and GUI apps,
and extras/#2801: Putty shims open a blank window,
I would like us to consider replacing our current C#-based shim code, (or is it at ScoopInstaller/Shim ?) with this C-based code, at https://github.com/71/scoop-better-shimexe .

Its readme explains the benefits. I can't think of any drawbacks.

[Ash258]

I am not against making this change, after some code review of new shim, but it needs to be optional in first few weeks for proper testing.

[trajano]

Can you provide instructions on how to do that?

[trajano]

Hold off on this one found something that may affect people 71/scoop-better-shimexe#6

[trajano]

Based on the discussions on the bug, seems like it may be something on my PC. No one else had reported any issues. Maybe it is good to go.

[gerardog]

I was unable to reproduce @trajano issue, and I have interests in adopting this proposal since gsudo is affected by #3294 and #1896.

How can I help move this forward?

FWIW, I forked ScoopInstaller/Shim at gerardog/Shim, replaced the C# version with this one, and updated the AppVeyor build scripts. last build

P.S. Unsure if the build script should target x32, so it covers all platforms?

[gerardog]

How can I help move this forward?

I'm sorry I am not aware of how Scoop governance works. Only thing I can think of is to mention @lukesampson, and kindly request his advice.

[lukesampson]

@rasa is the man for this. I assume we need a pull request that lets Scoop use the C shim as an option set with scoop config (according to @Ash258's advice), but ask @rasa.