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

'man scrcpy' shows a instead of a -, causing errors #4528

Closed
2 tasks done
Ganton opened this issue Dec 13, 2023 · 2 comments
Closed
2 tasks done

'man scrcpy' shows a instead of a -, causing errors #4528

Ganton opened this issue Dec 13, 2023 · 2 comments

Comments

@Ganton
Copy link

Ganton commented Dec 13, 2023

  • I have read the FAQ.
  • I have searched in existing issues.

Environment

  • OS: Kubuntu 23.10
  • scrcpy version: 1.25
  • installation method: apt
  • device model: [it does not matter]
  • Android version: [it does not matter]

Describe the bug

If the user executes man scrcpy, copies the --disable‐screensaver text that is seen, and executes scrcpy --disable‐screensaver: an scrcpy: unrecognized option '--disable‐screensaver' error message is seen.

I attach a screenshot of what is seen when man scrcpy is executed and the user looks for -. It turns out that is not -:

Screenshot man

Edited: This a screenshot of the difference between both symbols (because after reporting this bug: in the bug description and title, Github has replaced the first symbol with the second):

difference

Thanks for scrcpy and its quality.

@rom1v
Copy link
Collaborator

rom1v commented Dec 13, 2023

Thank you for your report.

Fixed by cbce423.

The result probably depends on your manpage viewer or locale. On my machine, it prints a normal - anyway. Here I copied the text from the manpage and pasted:

$ echo --disable-screensaver | hd
00000000  2d 2d 64 69 73 61 62 6c  65 2d 73 63 72 65 65 6e  |--disable-screen|
00000010  73 61 76 65 72 0a                                 |saver.|
00000016

But if I copy tour text from this issue:

$ echo --disable‐screensaver | hd
00000000  2d 2d 64 69 73 61 62 6c  65 e2 80 90 73 63 72 65  |--disable...scre|
00000010  65 6e 73 61 76 65 72 0a                           |ensaver.|
00000018

@Ganton
Copy link
Author

Ganton commented Dec 13, 2023

The solution looks good. Great!

@rom1v rom1v closed this as completed in cbce423 Mar 2, 2024
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

No branches or pull requests

2 participants