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

[Doc] centos8 seq command -t error ! #18517

Closed
1 of 2 tasks
startjava opened this issue Nov 17, 2022 · 2 comments · Fixed by #18526
Closed
1 of 2 tasks

[Doc] centos8 seq command -t error ! #18517

startjava opened this issue Nov 17, 2022 · 2 comments · Fixed by #18526
Labels
doc-required Your PR changes impact docs and you will update later.

Comments

@startjava
Copy link

Search before asking

  • I searched in the issues and found nothing similar.

What issue do you find in Pulsar docs?

https://pulsar.apache.org/docs/next/getting-started-standalone/#write-some-more-messages

bin/pulsar-client produce my-topic --messages "$(seq -s, -f 'Message NO.%g' -t '\n' 1 10)"

CentOS8 has error:
[ghy@localhost bin]$ pwd
/home/ghy/T/apache-pulsar/bin
[ghy@localhost bin]$ ./pulsar-client produce my-topic --messages "$(seq -s, -f 'Message NO.%g' -t '\n' 1 10)"
seq: 不适用的选项 -- t
请尝试执行 "seq --help" 来获取更多信息。

use bottom command run ok:
./pulsar-client produce my-topic --messages "$(seq -s, -f 'Message NO.%g' 1 10)"

What is your suggestion?

?

Any reference?

No response

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@startjava startjava added the doc-required Your PR changes impact docs and you will update later. label Nov 17, 2022
@startjava
Copy link
Author

@momo-jun

@labuladong
Copy link
Contributor

labuladong commented Nov 17, 2022

Seems -t option is only for macOS:

-t string, --terminator string
     Use string to terminate sequence of numbers.  The string can contain character escape sequences in backslash notation as
     defined in ANSI X3.159-1989 (“ANSI C89”).  This option is useful when the default separator does not contain a \n.

It's reasonable to remove this option in doc for common Linux. I will improve it in the next pr.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required Your PR changes impact docs and you will update later.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants