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

RTL-TCP: Make it visible and rearrange command-line help #35

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

lars18th
Copy link

@lars18th lars18th commented Mar 6, 2018

This is based on the issue #27 .

@JvanKatwijk
Copy link
Owner

Would not it be better - if you insist of having RTL_TCP input documented - to add descriptions for how to specify address and port

@lars18th
Copy link
Author

lars18th commented Mar 6, 2018

Hi @JvanKatwijk ,

Would not it be better - if you insist of having RTL_TCP input documented - to add descriptions for how to specify address and port

I'm not sure of your request. Please see this (although I feel like you already know):

I don't know what more I need to describe. Perhaps an example?

In any case, the PR contains more than only RTL_TCP stuff. An I hope you agree to commit it. 😄

@lars18th
Copy link
Author

lars18th commented Mar 7, 2018

Hi @JvanKatwijk ,

From my point of view this patch is finished. Please, review it. I feel I do all required cleans.
If you think I missed something, let me know.

Regards.

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