-
-
Notifications
You must be signed in to change notification settings - Fork 852
Just type "docker-compose up" to install a Flex project (major refactoring) #5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@@ -4,7 +4,7 @@ | |||
# Origin: https://github.com/composer/composer/blob/master/doc/faqs/how-to-install-composer-programmatically.md | |||
# Licence: MIT | |||
|
|||
EXPECTED_SIGNATURE=$(wget -q -O - https://composer.github.io/installer.sig) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I must repeat this again. I don't see the point of us deviating from the official sample script here. The .fetch-deps
do not end up in any of the image layers, so it has zero impact except for a very slightly longer build (does not make any practical difference).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought we have already been through this before... laughs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It reduces the complexity of the build process and the maintenance. As pointed out by Jordi, there is no point using the old method when using PHP 7+. And we're using PHP 7+.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It reduces the complexity of the build process and the maintenance.
Not really. wget
is already provided by BusyBox. Installing openssl
is required for HTTPS support. And that's it. I don't see where the complexity lies.
As pointed out by Jordi, there is no point using the old method when using PHP 7+. And we're using PHP 7+.
Agreed 100%. But we don't really gain anything by changing it. It only creates another maintenance issue, if anything. (Instead of just copy-and-paste from the official sample script, now we have our own modifications. 😆)
20d6aea
to
73d58e3
Compare
This PR was merged into the 1.3-dev branch. Discussion ---------- Docker support (Docker Compose and Dockerfile) /!\ This piece of software is experimental and may break your app. Don't use it without a proper versioning strategy. This is a part of my global plan to add a first-class Docker support in Symfony. It requires dunglas/symfony-docker#5. This PR allows recipes to updates `docker-compose.yaml` (to add services and volumes for instance) and the `Dockerfile` (to add PHP extensions for instance). I'll blog about this when I'll be back from vacation. Commits ------- 78c2bec Docker support (Docker Compose and Dockerfile)
Just type "docker-compose up" to install a Flex project (major refactoring)
This is a major refactoring of this repository. It allows to start hacking on a new Flex project by just downloading this repository and running
docker-compose up
.There is no need to have PHP installed, the Docker setup will download and install Flex and create a new project during the first run. Project's files will be available both in the container and in the host (using a volume).
All PHP extensions (like APCu) and optimizations (like performance-related
php.ini
settings) are already handled.