ydeploy icon indicating copy to clipboard operation
ydeploy copied to clipboard

tasks/setup.php copyDatabase lädt SQL dump nicht aus release Ordner hoch

Open ynamite opened this issue 2 years ago • 1 comments

Damit der Task tasks/setup.php bei mir durchläuft, musst ich die Methode copyDatabase leicht anpassen und zwar musste ich auf Zeile 211 folgende Zeile entfernen: https://github.com/yakamara/ydeploy/blob/7e04a5f1415dbf07f86ad8aa35d18637ef68c043/deployer/tasks/setup.php#L211

Das Problem ist, dass mit dieser Zeile der Export in /.build/release/ landet, die Funktion upload weiter unten aber als Quelle vom Projekt-Root ausgehnt /: https://github.com/yakamara/ydeploy/blob/7e04a5f1415dbf07f86ad8aa35d18637ef68c043/deployer/tasks/setup.php#L225

Wenn man die angegebene Zeile 211 entfernt, landet der Dump ebenfalls im korrekten Ordner, aber vom Root ausgehend (nicht im Release).

Man könnte auch umgekehrt die Zeile hier auf upload("{{current_path}}/$path", "{{release_path}}/$path"); ändern, also den current_path mitgeben, aber da bin ich unsicher, ob der SQL dump besser im Release ist oder doch im Root (Entwicklungsumgebung). Jedenfalls hat das mein Problem gelöst.

ynamite avatar Jan 12 '24 15:01 ynamite

Nach weiteren Tests denke ich, dass die zweite Variante mehr Sinn ergibt. Oder was meint ihr?

edit: allerdings krieg ich es mit weder mit der Original-Variante noch mit der zweiten Variante (oben) zum laufen ... hmpf

ynamite avatar Mar 13 '24 14:03 ynamite

Von mir angedacht war eigentlich Variante 1. Die nutze ich nun per #86.

(Die Zeile wird aber benötigt, wenn man einen anderen Server als Quelle nimmt, nicht Lokal. Insbesondere für das lokale Setup wichtig. Dementsprechend ist die Zeile nun in einem if.)

gharlan avatar May 21 '24 12:05 gharlan

Alles klar, super, vielen Dank!

ynamite avatar May 21 '24 13:05 ynamite