pysmartnode icon indicating copy to clipboard operation
pysmartnode copied to clipboard

add docker feature to make froozen mpy files

Open dasTholo opened this issue 5 years ago • 2 comments

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

dasTholo avatar Feb 16 '20 10:02 dasTholo

Danke für den PR. Hier fallen mir ein paar Punkte auf, die mir unklar sind:

  1. Die Beschreibungen sind so kurz gehalten, dass es z.T. schwer nachzuvollziehen ist, was du eigentlich machst.
  2. Das Submodul ist ein deiner Fork nicht korrekt eingebunden und kann nicht angeklickt werden
  3. 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.
  4. Du veränderst Skripte, die lokal laufen sollen. Besser wäre es, neue Skripte speziell für die Docker-Umgebung zu erstellen.
  5. Deine Skripte enthalten absolute Pfadangaben (~/PycharmProjects/py-node/tools/esp32/dockercontainer-mpy-esp32-cc/mods) und einige ToDo

kevinkk525 avatar Feb 16 '20 14:02 kevinkk525

  1. Stimmt! Sobald 2. abgeklärt ist, werde ich das nachholen
  2. 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
  3. Weil ich deine nur angepasst habe ;)

Bei den Bash scripten hab ich mich von deinen Bash scripten inspirieren lassen.

dasTholo avatar Feb 16 '20 18:02 dasTholo