-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
fish_update_completions throws UnicodeDecodeError #658
Comments
How old is the build of fish which you're running ? I believe this issue was fixed by #420 |
It is pretty new - the last commit is bcab703 |
Can you please post somewhere the verbose output, that is, run this:
Thanks! |
Perfect, thanks. Looks like nping.1.gz tripped us up. |
Should be fixed here: 1c55563 Thanks for reporting this |
I'm still seeing this with master HEAD.
|
@dag Maybe you should also paste the verbose output somewhere?
|
@nmeum That was from the verbose output, but skipping all the irrelevant bits. |
oops, indeed |
This happened to me as well. Same issue as dag commented like: Considering /usr/share/man/man1/xfce4-settings-manager.1.gz
Trying parser Type3
Command is xfce4-settings-manager
Trying parser Darwin man parser
Trying parser Deroffing man parser
Error parsing /usr/share/man/man1/xfce4-settings-manager.1.gz: <type 'exceptions.UnicodeDecodeError'>
Traceback (most recent call last):
File "/usr/local/share/fish/tools/create_manpage_completions.py", line 894, in parse_and_output_man_pages
if parse_manpage_at_path(manpage_path, output_directory):
File "/usr/local/share/fish/tools/create_manpage_completions.py", line 810, in parse_manpage_at_path
success = parser.parseManPage(manpage)
File "/usr/local/share/fish/tools/create_manpage_completions.py", line 692, in parseManPage
built_command(options, description)
File "/usr/local/share/fish/tools/create_manpage_completions.py", line 180, in built_command
truncated_description = truncated_description[:prefix_len] + truncation_suffix
UnicodeDecodeError: 'ascii' codec can't decode byte 0xa9 in position 37: ordinal not in range(128) but with I'm running what was Ubuntu 12.04, but with xubuntu-desktop package installed. I'd be happy to test any potential patches on my system. |
Hopefully fixed again by 2f50162 |
Works for me! |
Great, thanks for the quick check! |
Worked for me too, thanks. As for this commit, it's on the Integration_2.0.0 branch. Is that the latest development branch and I should follow that to stay on top of things? |
It is for the next few days, until we release, at which point it will be merged into master. |
and
On |
The text was updated successfully, but these errors were encountered: