pysmartnode
pysmartnode copied to clipboard
add docker feature to make froozen mpy files
esp32_build_image_files.sh cp files to dockercontainer-mpy-esp32-cc esp32_build_image.sh make a firmwareimage
Todo: right path like $(pwd) isn`t set, you must change path
Danke für den PR. Hier fallen mir ein paar Punkte auf, die mir unklar sind:
- Die Beschreibungen sind so kurz gehalten, dass es z.T. schwer nachzuvollziehen ist, was du eigentlich machst.
- Das Submodul ist ein deiner Fork nicht korrekt eingebunden und kann nicht angeklickt werden
- Wieso nur Skripte, um die mpy in docker zu generieren aber keine Skripte um auch die Firmware in Docker zu kompilieren? Ich denke, ein PR für docker ist nur sinnvoll, wenn auch die Firmware mit Hilfe eines Docker containers kompiliert wird.
- Du veränderst Skripte, die lokal laufen sollen. Besser wäre es, neue Skripte speziell für die Docker-Umgebung zu erstellen.
- Deine Skripte enthalten absolute Pfadangaben (~/PycharmProjects/py-node/tools/esp32/dockercontainer-mpy-esp32-cc/mods) und einige ToDo
- Stimmt! Sobald 2. abgeklärt ist, werde ich das nachholen
- Mhh?! Das muss ich mir genauer anschauen. Ich hab einen Eintrag in .gitsubmodules gemacht und local funktioniert es. Ich prüfe 3.+4. Ich hab mich da von deinen Scripten inspirieren lassen und hab die nur angepasst. Aber du hast recht. Vielleicht sollte ich das mal "komplett" umschreiben und alles ein einem Script/Dockercontainer laufen lassen
- Weil ich deine nur angepasst habe ;)
Bei den Bash scripten hab ich mich von deinen Bash scripten inspirieren lassen.