Skip to content

Conversation

ben
Copy link
Member

@ben ben commented Aug 15, 2017

This aims to address the confusion that leads to pull requests like #733, #732, #707, #671, and probably others. It aims to be more explicit with thecore.editor instructions when it comes to 32-bit editors on a 64-bit system.

On a Windows system, if you want to use a different text editor, you must specify the full path to its executable file.
This can be different depending on how your editor is packaged.

In the case of Notepad++, a popular programming editor, the application is always shipped as a 32-bit executable.

Choose a reason for hiding this comment

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

The following is not quite true ...

In the case of Notepad++, a popular programming editor, the application is always shipped as a 32-bit executable.

As I wrote in #733 .....

the state of affairs today, ...., is that a 64-bit Notepad++ is available. .... the 64-bit version doesn't support most plugins. (see https://notepad-plus-plus.org/download/v7.3.2.html)

So in effect, for the immediate future, one should generally continue to only use the 32 bit version.

So I'd suggest something like the following wording:

This can be different depending on whether your editor or Windows version is 32 bit or 64 bit.

In the case of Notepad++, a popular programming editor, you are likely to want to use the 32-bit version (as at the time of writing the 64-bit version doesn't support all the plug-ins ).

Otherwise this rest of the commit looks right.

Copy link
Member Author

Choose a reason for hiding this comment

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

Nice, I'll use that. Thanks!

@ben ben merged commit 00eaa5b into master Aug 16, 2017
@ben ben deleted the ben/win3264 branch August 16, 2017 16:27
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.

2 participants