Skip to content

install.iss: add Microsoft Edit as an editor option #617

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mjcheetham
Copy link
Member

Add the newly re-written and open-source 'Edit' terminal-based text editor, inspired by the old MS-DOS Editor.

Note that at this time the edit.exe program is only available as a simple zip download, or via the WinGet package manager for Windows. Since we can only rely on a stable path for the WinGet package manager we should look there (per-user).

Add the newly re-written and open-source 'Edit' terminal-based text
editor, inspired by the old MS-DOS Editor.

Note that at this time the edit.exe program is only available as a
simple zip download, or via the WinGet package manager for Windows.
Since we can only rely on a stable path for the WinGet package manager
we should look there (per-user).

Signed-off-by: Matthew John Cheetham <[email protected]>
@mjcheetham mjcheetham requested a review from dscho May 20, 2025 11:41
@@ -1884,6 +1886,11 @@ begin
Result:=ExtractFilePath(ExtractCommandPath(Command)) + Suffix;
end;

function GetMicrosoftEditPath:String;
begin
Result:=ExpandConstant('{localappdata}\Microsoft\WinGet\Packages\Microsoft.Edit_Microsoft.Winget.Source_8wekyb3d8bbwe\edit-1.0.0-x86_64-windows\edit.exe');
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to support ARM64, and also not hardcode the package version here.

Copy link
Member

@dscho dscho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good so far!

@rimrul
Copy link
Member

rimrul commented May 20, 2025

Note that at this time the edit.exe program is only available as a simple zip download, or via the WinGet package manager for Windows.

There might soon be a MINGW-package we could theoretically use

@ognevny
Copy link

ognevny commented May 20, 2025

Note that at this time the edit.exe program is only available as a simple zip download, or via the WinGet package manager for Windows.

There might soon be a MINGW-package we could theoretically use

I wanted to check one extra thing and think about package name more. edit seems a lot generic, it can be ms-edit or something

@mati865
Copy link

mati865 commented May 20, 2025

Also, it sort of conflicts with /usr/bin/edit which, albeit absent in MSYS2, is present in most of Linux distributions.

@ognevny
Copy link

ognevny commented May 21, 2025

Also, it sort of conflicts with /usr/bin/edit which, albeit absent in MSYS2, is present in most of Linux distributions.

there shouldn't be conflict in case of MINGW environments. ${MINGW_PREFIX}/bin/edit.exe will be preferred anyway

@mati865
Copy link

mati865 commented May 21, 2025

I meant the name is ambiguous.

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.

5 participants