Hardware
/
litex
mirror of https://github.com/enjoy-digital/litex.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

core/naxriscv: Don't use os.system to execute sbt 

The use of os.system together with cd is a problem because it changes the
CWD for the whole python process. This breaks for example --csr-csv.
This commit is contained in:
Christian Klarhorst 2022-11-11 14:14:47 +01:00
parent c1885b333f
commit c1c4910d67
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
litex/soc/cores/cpu/naxriscv/core.py
View File

@ -260,8 +260,7 @@ class NaxRiscv(CPU):
cmd = f"""cd {ndir} && sbt "runMain naxriscv.platform.LitexGen {" ".join(gen_args)}\"""" cmd = f"""cd {ndir} && sbt "runMain naxriscv.platform.LitexGen {" ".join(gen_args)}\""""
print("NaxRiscv generation command :") print("NaxRiscv generation command :")
print(cmd) print(cmd)
if os.system(cmd) != 0: subprocess.check_call(cmd, shell=True)
raise OSError('Failed to run sbt')
def add_sources(self, platform): def add_sources(self, platform):