-
Notifications
You must be signed in to change notification settings - Fork 66
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
[BUG] kicad8 erc crashed with json.decoder.JSONDecodeError #599
Comments
Hi @vermut ! |
We don't have kicad_common.json in our repo. Should I add empty one? Yes, |
This is in the docker image, are you sure you want a fresh image? In a fresh image I get:
And the file contains valid JSON. Just try:
I tried using the same command you put here:
And didn't get your error. I got an error related to the fact that I just downloaded a .ZIP file and didn't clone the repo. |
I see file present in docker, but the command still failing for me. I'll try tomorrow on ZIP, and after that on Linux - maybe Windows is messing up some important permission. |
Hi @vermut ! |
I ran it manually (from docker bash) and kicad_common is present BEFORE I run kibot but empty AFTER I run it. What's going on?..
|
I disabled DRC - ERC worked, but next step broke with same error. Something between steps is doing something.
|
You can try running with high verbosity levels (i.e. -vvvv) |
I tried:
And all worked well |
Also tried it as root using your command line:
And got:
Before and after running the ERC. Note that KiAuto will move away the KiCad config renaming to use a reproducible config, and then will rename it again to the original name. Something must be really wrong to truncate a file during a rename operation. |
Kicad commands doesn't harm the json file.
|
Of course, the problem must be in an operation done by eeschema_do and pcbnew_do |
Yes, this works on usual Linux machine. Under pure WSL docker it gave me this:
So this time it didn't rename back. Possibly a race condition, but otherwise I had enough of this, we can close it with "windows being windows" resolution. |
- Saves time and problems - Could be a workaround for Windows issues (INTI-CMNB/KiBot#599)
I pushed a patch to mess with config files only when using the GUI. |
Now run_drc and run_erc are deprecated in favor of drc and erc that uses KiCad CLI directly. So this issue, even when looks like a Windows glitch, shouldn't be a problem. |
Confirm, works flawlessly even on my devil's spawn OS. |
Files are at ClemensElflein/OpenMower#83
Worked in V7 (
main
branch).You've helped me already. This is V6 imported into V7 imported into V8. So there might be weird things inside. I just can't decipher where to look at - I thought there are no JSONs in play.
The text was updated successfully, but these errors were encountered: