search

fhem / fhem-docker

A basic Docker image for FHEM house automation system, based on Debian Linux.
https://fhem.de/
MIT License
84 stars 28 forks source link

Patch marko bullseye #35

Closed CoolTuxNet closed 3 years ago

sidey79 commented 3 years ago

Ich verstehe nicht wieso das nun merged ist.

Die Images werden meiner Ansicht nach nicht funktionieren, da Angaben im Dockerfile fehlen.

CoolTuxNet commented 3 years ago

Der Test zu mindest lief beim PR durch. Ich teste gleich mal den dev Branch

sidey79 commented 3 years ago

Der Testet aber nur amd64 und das Image wird auch funktionieren, da auf amd64 auch der build Server läuft.

Für die anderen Plattformen fehlt meines erachtens die Angabe der Zielplattform

CoolTuxNet commented 3 years ago

Die Angabe der Zielplattform steht/stand aber nicht im Dockerfile, oder?

sidey79 commented 3 years ago

Ich hatte das so gemacht, stand auch weiter oben im PR:

FROM --platform=$TARGETPLATFORM debian:buster-20210408-slim

Es ist allerdings auch möglich die Zielplattform als Parameter in buildx Aufruf zu übergeben.

Vermutlich wird das durch die action auch gemacht, sonst würde der build Prozess die Pakete nicht installieren können.

CoolTuxNet commented 3 years ago

Vielleicht können wir beide uns diesbezüglich mal zusammen setzen und per Telefon oder Videocall uns das mal in Ruhe anschauen.

CoolTuxNet commented 3 years ago

Nach einem merge in den dev gibt der Test nun einen Fehler aus

Error: buildx failed with: error: failed to solve: process "/bin/sh -c if ( [ \"${NPM_PKGS}\" != \"\" ] || [ \"${IMAGE_LAYER_NODEJS}\" != \"0\" ] || [ \"${IMAGE_LAYER_NODEJS_EXT}\" != \"0\" ] ) ; then LC_ALL=C curl --retry 3 --retry-connrefused --retry-delay 2 -fsSL https://deb.nodesource.com/setup_14.x | LC_ALL=C bash - && LC_ALL=C DEBIAN_FRONTEND=noninteractive apt-get install -qqy --no-install-recommends nodejs=14. && if [ ! -e /usr/bin/npm ]; then LC_ALL=C DEBIAN_FRONTEND=noninteractive apt-get install -qqy --no-install-recommends npm=5.8. ; fi && npm install -g --unsafe-perm --production npm && if [ \"${NPM_PKGS}\" != \"\" ]; then npm install -g --unsafe-perm --production ${NPM_PKGS} ; fi && if [ \"${IMAGE_LAYER_NODEJS_EXT}\" != \"0\" ]; then npm install -g --unsafe-perm --production alexa-cookie2 alexa-fhem gassistant-fhem homebridge homebridge-fhem tradfri-fhem ; fi && LC_ALL=C apt-get autoremove -qqy && LC_ALL=C apt-get clean && rm -rf /var/lib/apt/lists/ /tmp/ /var/tmp/ ~/.[^.] ~/.?? ~/* ; fi" did not complete successfully: exit code: 1