Skip to content

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

Merged
merged 2 commits into from
Jul 17, 2017

Conversation

dunglas
Copy link
Owner

@dunglas dunglas commented Jul 6, 2017

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.

@@ -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)

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).

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

Copy link
Owner Author

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+.

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. 😆)

@dunglas dunglas merged commit 3dea202 into master Jul 17, 2017
@dunglas dunglas deleted the autoinstall branch July 17, 2017 06:47
fabpot added a commit to symfony/flex that referenced this pull request Jun 26, 2019
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)
VitaliyMinenko pushed a commit to VitaliyMinenko/notofication-service that referenced this pull request Sep 16, 2024
Just type "docker-compose up" to install a Flex project (major refactoring)
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