Skip to content

add docker feature to make froozen mpy files#12

Open
dasTholo wants to merge 4 commits intokevinkk525:masterfrom
dasTholo:feature-docker_mpy
Open

add docker feature to make froozen mpy files#12
dasTholo wants to merge 4 commits intokevinkk525:masterfrom
dasTholo:feature-docker_mpy

Conversation

@dasTholo
Copy link
Copy Markdown

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

@kevinkk525
Copy link
Copy Markdown
Owner

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

@dasTholo
Copy link
Copy Markdown
Author

  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 and others added 2 commits February 18, 2020 20:37
init sh script files to handle with mpy-docker
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants