Code Monkey home page Code Monkey logo

Comments (8)

madbob avatar madbob commented on August 31, 2024 1

Aggiudicato ;-)

from gasdottong.

madbob avatar madbob commented on August 31, 2024

Non usando personalmente Docker/Sail non saprei dire, ma non mi è chiaro come il tutto possa funzionare eliminando le estensioni PHP e composer... Non sono comunque dipendenze di Sail?

from gasdottong.

PixelTux avatar PixelTux commented on August 31, 2024

Io vedo Docker come una sorta di server esterno (completo con PHP e composer) che condivide una cartella sulla mia macchina e ha gli stessi permessi dentro essa. Per creare questo "server", ho bisogno delle sue caratteristiche che sono definite nel file Dockerfile situato in 'GASdottoNG/code/vendor/laravel/sail/runtimes/8.2/'. Il punto è che la cartella vendor viene "generata" da composer, quindi quando clono la repository non esiste. Per evitare di dover installare PHP e composer sulla mia macchina locale, potrei semplicemente costruire prima il container e poi utilizzare PHP e composer che vengono installati all'interno. Quindi, quello che stavo pensando era di includere nei commit la cartella sail (vendor/laravel/sail/*) per avere accesso al file Dockerfile e allo script sail (vendor/bin/sail), così da poter utilizzare il comando sail invece dei comandi docker subito dal inizio. Questi file vengono sovrascritti durante gli aggiornamenti, quindi è facile da mantenere, in quanto vengono aggiornati automaticamente da composer in caso di modifiche. Cosa ne pensi?

from gasdottong.

PixelTux avatar PixelTux commented on August 31, 2024

Banalmente descrivere Sail come commandi Docker e Docker-Composer in stille Laravel per Laravel.

from gasdottong.

madbob avatar madbob commented on August 31, 2024

Mmm... Mi garba veramente poco committare dentro al repository dei contenuti in vendor...

Capisco il punto, ed effettivamente è contestabile il fatto che Sail sia installabile solo avendo a priori a disposizione gli strumenti di sviluppo, ma del resto Sail stesso viene proposto come uno strumento di sviluppo - non di deploy - dunque evidentemente, nella sua implementazione, non si sono posti il problema di predisporre un Docker auto-definito.

Molto dipende da come intendi il Docker per GASdotto. Se come uno strumento per agevolare lo sviluppo, è ragionevole assumere che PHP e composer siano installati sulla macchina host (se qualcuno vuol partecipare allo sviluppo di GASdotto, almeno quel minimo di strumenti li deve avere). Se come uno strumento per agevolare il deploy e l'installazione autonoma su un proprio server, sarebbe forse meglio lasciar perdere Sail e tornare - come era prima - al Dockerfile (o ad un file per Docker Compose).

from gasdottong.

PixelTux avatar PixelTux commented on August 31, 2024

Capisco perché non ti garba l'idea di committare la cartella vendor, e sinceramente sono d'accordo. Tuttavia, ho suggerito questa soluzione perché sembrava la più semplice e immediata, anche se un po' stupida.

Io lo intendevo come strumento per agevolare lo sviluppo. Principali obbiettivi:

  • poter installare il progetto soltanto usando Docker e Docker Compose
  • facile da mantenere (idealmente automatizzato)

Personalmente, sono soddisfatto di Sail. L'idea mi è venuta soltanto perché stavo sperimentando con Clear Linux e Laravel. Ho notato che potrei fare a meno di PHP e Composer se avessi Sail. Un'alternativa potrebbe essere uno script che automatizza il processo di recupero dei dati da Sail su GitHub. Comunque riflettendoci meglio, mi rendo conto che questa operazione è necessaria solo una volta e quindi potrebbe non essere così utile.

Se pensi che quest "feature" non è rilevante chiudi pure la issue. :)

from gasdottong.

PixelTux avatar PixelTux commented on August 31, 2024

Mi sembra di aver trovato la soluzione nella documentazione Laravel Installing Composer Dependencies for Existing Applications. Proverò a seguire quella procedura e vedere se funziona. Tuttavia, credo che potrebbe essere la soluzione che stavo cercando.

from gasdottong.

PixelTux avatar PixelTux commented on August 31, 2024

Provato e funziona, se sei d'accordo proved ad aggiornare README.md, unica modifica necessaria. 🙂

from gasdottong.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.