Make all .ini files has common prefix

drupal-docker / php

Dockerized PHP for Drupal :droplet::whale:

https://hub.docker.com/r/drupaldocker/php/

MIT License

38 stars 51 forks source link

Make all .ini files has common prefix #43

Closed zaporylie closed 8 years ago

zaporylie commented 8 years ago

I believe we should use common prefix for all Drupal customizations. That approach should reduce eventual php.ini negotiation problems.

Docker is already using docker-php-ext- so I think we can prepend all filenames with drupal- prefix. That should place our customizations after default docker settings for extensions.

zaporylie commented 8 years ago

If this goes in we can start using simplified syntax:

COPY drupal?.ini /usr/local/etc/php/conf.d/

slashrsm commented 8 years ago

Looks good to me. Standardization == nice.