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

Remove some Win9x legacy from netrw built-in plugin #14732

Closed
wants to merge 6 commits into from

Conversation

nir9
Copy link
Contributor

@nir9 nir9 commented May 8, 2024

Problem: The built-in file browsing plugin "netrw" contains a bunch of code that handles Windows 95/Win16 which are not supported by modern versions of Vim

Solution: Remove some legacy code/documentation from "netrw"

@nir9
Copy link
Contributor Author

nir9 commented May 8, 2024

One CI job fails since I did not rebuild the doc tags, I don't remember if that is necessary for PRs, if so let me know and I will fix

@chrisbra
Copy link
Member

chrisbra commented May 8, 2024

no problem, I can fix it while merging it.

runtime/autoload/netrw.vim Outdated Show resolved Hide resolved
runtime/autoload/netrw.vim Outdated Show resolved Hide resolved
@k-takata
Copy link
Member

k-takata commented May 8, 2024

Could you also update this part?

if has("win32") || has("win95") || has("win64") || has("win16")

@nir9
Copy link
Contributor Author

nir9 commented May 8, 2024

@k-takata Fixed CR 👍

runtime/autoload/netrw.vim Outdated Show resolved Hide resolved
runtime/autoload/netrw.vim Outdated Show resolved Hide resolved
@nir9
Copy link
Contributor Author

nir9 commented May 8, 2024

@dkearns fixed thanks, missed that due to the single quotes :)

@chrisbra
Copy link
Member

chrisbra commented May 8, 2024

Thanks. I'll merge it. However it's strange that is hardcodes 'c:\private.ppk' which doesn't seem to be documented and if you use pscp with a private key often, I would imagine you would typically use different keys for different servers.

@chrisbra chrisbra closed this in 1e34b95 May 8, 2024
clason added a commit to clason/neovim that referenced this pull request May 8, 2024
runtime(netrw): Remove and cleanup Win9x legacy from netrw

closes: vim/vim#14732

vim/vim@1e34b95

Co-authored-by: Nir Lichtman <nir@lichtman.org>
clason added a commit to clason/neovim that referenced this pull request May 8, 2024
runtime(netrw): Remove and cleanup Win9x legacy from netrw

closes: vim/vim#14732

vim/vim@1e34b95

Co-authored-by: Nir Lichtman <nir@lichtman.org>
clason added a commit to neovim/neovim that referenced this pull request May 8, 2024
runtime(netrw): Remove and cleanup Win9x legacy from netrw

closes: vim/vim#14732

vim/vim@1e34b95

Co-authored-by: Nir Lichtman <nir@lichtman.org>
@nir9
Copy link
Contributor Author

nir9 commented May 9, 2024

@chrisbra That's a good point, fixed that on another branch #14739 :)

@nir9 nir9 deleted the remove-win95-legacy-netrw branch May 9, 2024 16:30
huangyingw pushed a commit to huangyingw/neovim that referenced this pull request May 31, 2024
runtime(netrw): Remove and cleanup Win9x legacy from netrw

closes: vim/vim#14732

vim/vim@1e34b95

Co-authored-by: Nir Lichtman <nir@lichtman.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants