olofk / fusesoc

Package manager and build abstraction tool for FPGA/ASIC development
BSD 2-Clause "Simplified" License
1.14k stars 235 forks source link

fix missing workdir when no_export #619

Closed skokvermon closed 1 year ago

skokvermon commented 1 year ago

Fixes a bug When using no_export, there is an error when creating edalizer yaml

Traceback (most recent call last): File "/home/skok/.local/bin/fusesoc", line 11, in load_entry_point('fusesoc', 'console_scripts', 'fusesoc')() File "/home/skok/pkgs/fusesoc/fusesoc/main.py", line 746, in main fusesoc(args) File "/home/skok/pkgs/fusesoc/fusesoc/main.py", line 736, in fusesoc args.func(cm, args) File "/home/skok/pkgs/fusesoc/fusesoc/main.py", line 293, in run run_backend( File "/home/skok/pkgs/fusesoc/fusesoc/main.py", line 427, in run_backend edalizer.to_yaml(edam_file) File "/home/skok/pkgs/fusesoc/fusesoc/edalizer.py", line 507, in to_yaml return utils.yaml_fwrite(edam_file, self.edam) File "/home/skok/pkgs/fusesoc/fusesoc/utils.py", line 152, in yaml_fwrite with open(filepath, "w") as f:

simply fixed with a mkdir when no_export flag is given

olofk commented 1 year ago

Thanks for this. I believe however that 6e9e0e30dac3d should have fixed this already. Could you please check, or reopen otherwise

olofk commented 1 year ago

Sorry. I was a bit too quick, but hopefully ce42b32 should fix this. I chose this way instead of using os.system, since that doesn't work on Windows