RemoteDebug icon indicating copy to clipboard operation
RemoteDebug copied to clipboard

Fixed moved header file sha.h

Open kvoit opened this issue 3 years ago • 4 comments

See https://github.com/espressif/esp-idf/issues/3245

kvoit avatar May 29 '22 15:05 kvoit

what prevent this PR to be merged ?

grostim avatar Sep 26 '22 11:09 grostim

either the fact that this repository clearly isn't maintained anymore, or the fact that this actually breaks compatibility with older IDF versions ... by now, I wouldn't merge it myself in the current state, but haven't brought myself to write a clean version using ESP_IDF_VERSION_MAJOR/ESP_IDF_VERSION_MINOR defines to determine what to include.

kvoit avatar Sep 26 '22 16:09 kvoit

Hi there,

Very interested in this project, I wonder if there is a kind of "official fork" still maintained where those MR have been accepted ?

gbetous avatar Jan 03 '23 07:01 gbetous

I have just implemented my above mentioned clean solution for both versions in my fork. https://github.com/kvoit/RemoteDebug.git

I have also taken care of the deprecation warning of esp/sha.h and replaced it with sha/sha_parallel_engine.h. This compiles, but I don't have the hardware here to finally test.

kvoit avatar Jan 03 '23 08:01 kvoit