Skip to content

added touch to create ini files if not existing #67

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

Closed
wants to merge 1 commit into from
Closed

added touch to create ini files if not existing #67

wants to merge 1 commit into from

Conversation

ildanno
Copy link

@ildanno ildanno commented Feb 3, 2015

Added a touch $file before every append because it fails if file does not exists

@ildanno ildanno mentioned this pull request Feb 3, 2015
@yosifkit
Copy link
Member

yosifkit commented Feb 3, 2015

I don't see this solving any problem. The redirect will create the file if it does not exist (http://www.tldp.org/LDP/abs/html/io-redirection.html), which it does happily. I ran docker-php-ext-install pdo pdo_mysql in php:5.6-fpm and was able to see that pdo_mysql was enabled in php -i and I was able to create a new PDO connecting to a mysql host.

@ghvaldez
Copy link

I'm using php-5.6-apache and adding the docker-php-ext-install pdo pdo_mysql I get the following error at build time.

  • cd /usr/src/php/ext/pdo
  • phpize
    Configuring for:
    PHP Api Version: 20131106
    Zend Module Api No: 20131226
    Zend Extension Api No: 220131226
  • ./configure
    checking for grep that handles long lines and -e... /bin/grep
    checking for egrep... /bin/grep -E
    checking for a sed that does not truncate output... /bin/sed
    checking for cc... cc
    checking whether the C compiler works... no
    configure: error: in /usr/src/php/ext/pdo': configure: error: C compiler cannot create executables Seeconfig.log' for more details

If it works in fpm then whatever was fixed there needs to be added to apache version

@md5
Copy link
Contributor

md5 commented Mar 10, 2015

@ghvaldez that should have been fixed in #73. You might want to do a docker pull php:5.6-apache to confirm you have the latest updates to the image.

@ghvaldez
Copy link

@md5 That was it. Thanks!

@ildanno
Copy link
Author

ildanno commented May 20, 2015

I've got this problem with docker 1.4 running on boot2docker, but i'm not able to reproduce this anymore. Closing.

@ildanno ildanno closed this May 20, 2015
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.

4 participants