git-istage: Launching console text editor hangs console.
repro:
- Configure core.editor with an editor that runs in text mode in the same console. I used Far Manager and FTE
- Stage changes, press C, git-istage launches
git commit -v, which launchessh ...{edtior}which appears hung. It’s unable to write to the same console?
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Comments: 15 (15 by maintainers)
Commits related to this issue
- Ensure terminal-based commit editors are visible This fixed #33. — committed to terrajobst/git-istage by terrajobst 2 years ago
In case anyone cares what the fix was:
I will admit that this is somewhat selfish on my end. I work on the .NET team, so I’m basically living on cutting edge and I want to dogfood our features. In the context of this tool this would usually involve library- or language features. Most recent example: C# 11’s raw string literals.
My background is library development; for libraries it usually makes sense to target an older version of .NET in order to have more reach. As a library dev, it’s common to multi-target, i.e. build for multiple version of .NET in order to support the latest features while not sacrificing reach. This comes, of course, with some amount of complexity for testing and building.
For app development this kind of thinking feels overly complicated. It makes more sense to me to say that this tool requires the latest version of .NET which anyone can install. Well, until recently I naively assumed that when we release it’s also available for most Linux distros, which I just learned isn’t the case. I should add that I’m mostly on Windows and Mac and for those platforms we (Microsoft) release the builds for.
I guess I’ll need to think about this and reconsider.
Would you mind filing a separate issue, asking to make this tool only use .NET LTS releases?
I will try to explain, but first, let me ask you this: what is so crucial that this tool gains from using latest-and-greatest vs LTS that trumps the convenience of availability and ease of install?
Reasons:
@yurikus, a new version (v0.3.93) was being published to NuGet. It’ll probably take 30-60 min before it will show up though.