dmazzella / ucryptography

Lightweight porting of pyca/cryptography to Micropython based on ARM Mbed TLS
MIT License
7 stars 0 forks source link

ESP8266 Support #4

Closed officialpiyush closed 1 month ago

officialpiyush commented 1 month ago

Hello! Thank you for your work on the library!

I was trying to use the library with my ESP8266, I tried compiling it, but after flashing it did not show up in the modules in micropython. I used the ESP32 build commands as references, but I don't think it is supported it. Are there any plans to support ESP8266?

this is the command I used to build the firmware - make -C ports/esp8266 BOARD=ESP8266_GENERIC USER_C_MODULES="$(pwd)/usercmodule/ucryptography/micropython.cmake"

Also, the patch file seems to be outdated now, so the patch was not applied, so instead, I manually added the constants to the file

dmazzella commented 1 month ago

Hello, can you send me the output of the compilation? are there any warnings? errors?

dmazzella commented 1 month ago

please read steps from MicroPython ports/esp8266 README.md Building with a local toolchain

dmazzella commented 1 month ago

for now I am closing as unsupported platform, ucryptography needs mbedtls at version 3, and esp8266 uses axtls

officialpiyush commented 1 month ago

Hello!

can you send me the output of the compilation? are there any warnings? errors?

There were no errors, it compiles successfully, but the module was not getting added, makes sense now

please read steps from MicroPython ports/esp8266 README.md Building with a local toolchain

Yes, this is what I followed to build from scratch, just modified the make command to include the makefile for ucryptography

for now I am closing as unsupported platform, ucryptography needs mbedtls at version 3, and esp8266 uses axtls

Oh I see, that's sad, thank you anyway!

Could you please update the patchfile if possible, it seems like the source file was edited causing patch to fail, attached the screenshot (I manually patch the file now)

image

dmazzella commented 1 month ago

Could you please update the patchfile if possible, it seems like the source file was edited causing patch to fail, attached the screenshot (I manually patch the file now)

Done, thank you