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

UnicodeDecodeError when compiling volk_machine_avx2_64_mmx.c #180

Closed
litmaj0r opened this issue Apr 11, 2018 · 0 comments
Closed

UnicodeDecodeError when compiling volk_machine_avx2_64_mmx.c #180

litmaj0r opened this issue Apr 11, 2018 · 0 comments

Comments

@litmaj0r
Copy link
Contributor

When compiling volk with python3, it fails due to a UnicodeDecodeError. I believe this is due to utf-8 encoding not being forced, as I was able to get this to work by modifying line 167. I'll submit a PR for this.

root@6d5f03691db3:~/sdr/src/volk/build# make
[ 2%] Generating volk_machine_avx2_64_mmx.c
Traceback (most recent call last):
File "/root/sdr/src/volk/gen/volk_tmpl_utils.py", line 31, in
import volk_kernel_defs
File "/root/sdr/src/volk/gen/volk_kernel_defs.py", line 209, in
kernels = list(map(kernel_class, kernel_files))
File "/root/sdr/src/volk/gen/volk_kernel_defs.py", line 168, in init
code = codecs.open(kernel_file, 'r').read()
File "/usr/lib/python3.5/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 2460: ordinal not in range(128)
lib/CMakeFiles/volk_obj.dir/build.make:2454: recipe for target 'lib/volk_machine_avx2_64_mmx.c' failed
make[2]: *** [lib/volk_machine_avx2_64_mmx.c] Error 1
CMakeFiles/Makefile2:121: recipe for target 'lib/CMakeFiles/volk_obj.dir/all' failed
make[1]: *** [lib/CMakeFiles/volk_obj.dir/all] Error 2
Makefile:138: recipe for target 'all' failed
make: *** [all] Error 2

litmaj0r added a commit to litmaj0r/volk that referenced this issue Apr 11, 2018
noc0lour pushed a commit to litmaj0r/volk that referenced this issue Jun 29, 2018
Addresses Issue gnuradio#180
Provides a backward compatible fix to read the kernel_file.
n-west pushed a commit that referenced this issue Jul 25, 2018
Addresses Issue #180
Provides a backward compatible fix to read the kernel_file.
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