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

fix: Adding in SCP options configuration #340

Merged
merged 5 commits into from
May 6, 2024

Conversation

ericguin
Copy link
Contributor

@ericguin ericguin commented Mar 29, 2024

This changeset adds in additional SCP options to the config. This allows the user to specify a list of flags to send to the SCP command that will be expanded into each shell command.

The primary driver for this is from newer boxes SSHing into pre 9 openSSH boxes. New openSSH uses sftp server under the hood, rather than the older SCP protocol. If you go into a system that does not have these changes, SCP fails to work. The '-O' command line flag was introduced to resolve this.

Using this change, the user can now pass in extra_scp_options = {"-O"} to resolve the issue.

This changeset adds in additional SCP options to the config. This allows
the user to specify a list of flags to send to the SCP command that will
be expanded into each shell command.

The primary driver for this is from newe boxes SSHing into pre 9 openSSH
boxes. New openSSH uses sftp server under the hood, rather than the
older SCP protocol. If you go into a system that does not have these
changes, SCP fails to work. The '-O' command line flag was introduced to
resolve this.

Using this change, the user can now pass in `extra_scp_options = {"-O"}`
to resolve the issue.
@github-actions github-actions bot requested a review from stevearc March 29, 2024 14:53
@ericguin
Copy link
Contributor Author

Note: Couldn't find an open issue relating to this, but ran into the problem and decided to polish and publish the fix.

Copy link
Owner

@stevearc stevearc left a comment

Choose a reason for hiding this comment

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

I think this is a good idea, but table.unpack does not exist in LuaJIT. It instead has the global unpack function from lua 5.1. Did you even test this to see if it fixes your issue?

@ericguin
Copy link
Contributor Author

I've been running it for a few weeks without issue, yeah. That's super odd -- I'll go ahead and change it and push.

@github-actions github-actions bot requested a review from stevearc April 22, 2024 18:27
@ericguin
Copy link
Contributor Author

image
Did a quick test on my luajit -- I appear to have a table.unpack

@stevearc
Copy link
Owner

stevearc commented May 6, 2024

Possible that your luajit is built with DLUAJIT_ENABLE_LUA52COMPAT as specified in http://luajit.org/extensions.html, but I don't think this is guaranteed for all Neovim distributions. I'm kind of surprised that it's not uniform, honestly, but I guess the build system allows for it.

Thanks for the PR!

@stevearc stevearc merged commit 3abb607 into stevearc:master May 6, 2024
8 checks passed
@ericguin ericguin deleted the openssh-9-fixes branch May 6, 2024 17:43
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