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

cpu_interface: select_triple silently returns None #115

Closed
JohnSully opened this issue Oct 5, 2018 · 5 comments
Closed

cpu_interface: select_triple silently returns None #115

JohnSully opened this issue Oct 5, 2018 · 5 comments

Comments

@JohnSully
Copy link

Related to this change here:
b528a00

Instead of a useful error message compilation fails with:
File "core_nopll.py", line 247, in
main()
File "core_nopll.py", line 226, in main
vns = builder.build(build_name="litedram_core", regular_comb=False)
File "/home/john/repos/litex/litex/soc/integration/builder.py", line 156, in build
self._generate_includes()
File "/home/john/repos/litex/litex/soc/integration/builder.py", line 72, in _generate_includes
define(k, v)
File "/home/john/repos/litex/litex/soc/integration/builder.py", line 70, in define
variables_contents.append("{}={}\n".format(k, _makefile_escape(v)))
File "/home/john/repos/litex/litex/soc/integration/builder.py", line 27, in _makefile_escape
return s.replace("\", "\\")
AttributeError: 'NoneType' object has no attribute 'replace'

@JohnSully
Copy link
Author

Note: you get this when it can't find the toolchain. It should give an error message saying that like the old code did.

@enjoy-digital
Copy link
Owner

Thanks, i don't see how you get this, can you provide a minimal code to reproduce?

@JohnSully
Copy link
Author

JohnSully commented Oct 6, 2018 via email

@JohnSully
Copy link
Author

Also for full explicitness, the expected behavior in this case is an error message saying it can't find the toolchain. The original code used to have a mechanism for that.

@enjoy-digital
Copy link
Owner

Fixed via 3b27d2a.

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