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

Rename option to 'passive' and add some example in manual. #232

Merged
merged 2 commits into from
Nov 2, 2020

Conversation

dancerj
Copy link
Contributor

@dancerj dancerj commented Oct 30, 2020

Every time I want to use this command I seem to be looking up how to
do it.

Slave seems to have connotations I would rather use passive.

Keep the old option so that we don't break existing clients though.

sshfs.c Outdated
@@ -3635,7 +3636,7 @@ static void usage(const char *progname)
" -o ssh_protocol=N ssh protocol to use (default: 2)\n"
" -o sftp_server=SERV path to sftp server or subsystem (default: sftp)\n"
" -o directport=PORT directly connect to PORT bypassing ssh\n"
" -o slave communicate over stdin and stdout bypassing network\n"
" -o passive communicate over stdin and stdout bypassing network\n"
Copy link
Contributor

Choose a reason for hiding this comment

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

Please fix alignment

Every time I want to use this command I seem to be looking up how to
do it.
@Nikratio
Copy link
Contributor

Nikratio commented Nov 2, 2020

Thanks for the patch! Could you please fix the minor alignment issue above?

fix alignment
@dancerj
Copy link
Contributor Author

dancerj commented Nov 2, 2020

oops seems like I didn't actually make a change. Now I think I actually did, please take a look.

@Nikratio Nikratio merged commit 9e01ffd into libfuse:master Nov 2, 2020
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