Skip to content

netstat: multi-line output#3

Closed
florianl wants to merge 3 commits intoecki:masterfrom
florianl:multi-line
Closed

netstat: multi-line output#3
florianl wants to merge 3 commits intoecki:masterfrom
florianl:multi-line

Conversation

@florianl
Copy link
Copy Markdown
Contributor

Signed-off-by: Lehner Florian dev@der-flo.net

florianl added 2 commits June 28, 2013 22:12
Signed-off-by: Lehner Florian <dev@der-flo.net>
Signed-off-by: Lehner Florian <dev@der-flo.net>
@danrl
Copy link
Copy Markdown

danrl commented Jul 1, 2013

Looks fine!

@ecki
Copy link
Copy Markdown
Owner

ecki commented Jul 4, 2013

I think I would use -W as default if stdout is not a tty, this will please users using grep on the output. But in that case we would need a multi-line flag as well... hmmm

@danrl
Copy link
Copy Markdown

danrl commented Jul 5, 2013

I see objections to a changed default behaviour, some coming from the BSD direction (which use a different netstat) but also some from the GPL world. Most of them because of scripts parsing the output. Either way, we have to display correct IPv6 addresses in some way. Truncated output is very irritating at the moment. A multiline flag sounds reasonable to me, what do others think about this? Especially ecki?

Signed-off-by: Lehner Florian <dev@der-flo.net>
@florianl
Copy link
Copy Markdown
Contributor Author

florianl commented Jul 5, 2013

A multi-line flag is a great idea. I added it in the last commit. You will find a sample of the output at https://gist.github.com/florianl/5937147

@danrl
Copy link
Copy Markdown

danrl commented Jul 11, 2013

Any news?

@danrl
Copy link
Copy Markdown

danrl commented Jul 22, 2013

This is still the right place to ask, isn't it? Should we have used the mailing list instead? @florianl how was your last patch integrated?

Maybe it's just the holidays and we are way too inpatient :)

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.

3 participants