esppdp
esppdp copied to clipboard
Spritetm
build fails with format errors
I just tried building the project firmware using a fresh clone of esp-idf but the build fails with format errors ie -Werror=format
One example:
/home/leifj/esppdp/firmware/main/pdp11_rq.c:3286:58: warning: format '%d' expects argument of type 'int', but argument 3 has type 'uint32' {aka 'long unsigned int'} [-Wformat=]
Use the older 4.x esp-idf. esp-idf 5.x introduced some extra code checks, which lead to this error.
Best use is esp-idf 4.3, since 4.2 didn't had the 'esp_bt.h' and 4.4 introduced some different usage of the IRAM, so the project don't fit in default config.
Dear Sprite_tm,
You have done a stunning job here of stripping Simh down to produce a suitable image for the ESP32. I have looked into this in the past and given up in despair! So I built a much simpler PDP11 emulator. Anyway, the only problem with the repo is as above in that trying to build this with IDF 5.0 generates lots of errors (warnings). I have attached a rather dubious patch to sort this out. In addition, I have also changed some of the options in sdkconfig. I have attached this file as well as the format has changed from IDF 4->5. The final image now runs fine on my Wrover kit. (Google Drive) https://drive.google.com/file/d/192e8-08gp3iPm5B2T6XzKR-vhc4VgO48/view?usp=sharing EspPdp11.patch
@Isysxp Thanks for the patch! Unfortunately, it seems to be misformed, I can't get it to apply (patch says 'Only garbage was found in the patch input.' and on casual inspection at least the filenames to patch are missing)
Well, this is embarrassing. I used WinMerge to create this patch and apologies as there does not seem to be any way to use patches in this format!!!! So, I have created a fork with the updated code which I hope is OK. I would note that this is not a brilliant solution and there are still many build warnings. However, it is adequate for anyone using the latest IDF install.
Sure, if you can tell me where the fork is, I can pull from that, or you can see if you can create a pull request myself.