Skip to content

fix: correct ssh port flag#65

Merged
stffrdhrn merged 1 commit intoopenrisc:masterfrom
yalagadapavankumar:typo
Mar 14, 2026
Merged

fix: correct ssh port flag#65
stffrdhrn merged 1 commit intoopenrisc:masterfrom
yalagadapavankumar:typo

Conversation

@yalagadapavankumar
Copy link
Contributor

Changed -P to -p since -p is correct flag for specifying the port.

ssh -P 2222 virt@localhost to ssh -p 2222 virt@localhost

@stffrdhrn
Copy link
Member

Changed -P to -p since -p is correct flag for specifying the port.

ssh -P 2222 virt@localhost to ssh -p 2222 virt@localhost

You git commit message only says "Fix typo". Can you amend it to better follow the standards. Git commit messages should explain what is changed and why. They should have a subject and a body.

For example, copy what you have in the PR.

linux-on-qemu: correct ssh port flag

Changed -P to -p since -p is correct flag for specifying the port.

Changed -P to -p since -p is correct flag for specifying the port.
@yalagadapavankumar
Copy link
Contributor Author

Changed -P to -p since -p is correct flag for specifying the port.
ssh -P 2222 virt@localhost to ssh -p 2222 virt@localhost

You git commit message only says "Fix typo". Can you amend it to better follow the standards. Git commit messages should explain what is changed and why. They should have a subject and a body.

For example, copy what you have in the PR.

linux-on-qemu: correct ssh port flag

Changed -P to -p since -p is correct flag for specifying the port.

Thank you for the feedback.
I have amended the commit message to follow the recommended format with a subject and body explaining the change.

@stffrdhrn stffrdhrn merged commit e95725f into openrisc:master Mar 14, 2026
@stffrdhrn
Copy link
Member

Thank you

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