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

Missing php bcmath extension in Pantheon appserver #1655

Closed
bmcclure opened this Issue Oct 11, 2016 · 9 comments

Comments

Projects
None yet
2 participants
@bmcclure
Contributor

bmcclure commented Oct 11, 2016

When trying to enable the Drupal Commerce 2.x beta2 module suite for D8 on Kalabox 2.0.1 (Windows 10), the following error is returned:

RuntimeException: The bcmath extension is required by NumberFormatter. in CommerceGuys\Intl\Formatter\NumberFormatter->__construct() (line 113 of vendor/commerceguys/intl/src/Formatter/NumberFormatter.php).

I tried using both "56" and "70" for my PHP versions (with rebuilding in between), but both have the same issue.

Is adding this extension to the container a simple matter?

@pirog pirog changed the title from Missing bcmath extension for Drupal Commerce 8.x-2.x to Missing bcmath extension in Pantheon appserver Oct 11, 2016

@pirog pirog changed the title from Missing bcmath extension in Pantheon appserver to Missing php bcmath extension in Pantheon appserver Oct 11, 2016

@pirog pirog added this to the v2.0.2 milestone Oct 11, 2016

@pirog pirog self-assigned this Oct 11, 2016

@pirog

This comment has been minimized.

Show comment
Hide comment
@pirog

pirog Oct 11, 2016

Member

@bmcclure it should be pretty easy to add this manually (and we should add it in by default). Here are some instructions to add it yourself and we will get this into the next release.

# SSH into the Kalabox2 VM
# Check docs for clarity if you dont know how to do this
docker-machine ssh Kalabox2 

# Docker exec into the appserver for your app
docker exec -i -t myapp_appserver_1 bash

# This will drop you into a shell inside your appserver container
docker-php-ext-install bcmath
docker-php-enable bcmath
exit

# Drop back out to VM
exit

# Drop back out to host
kbox restart
Member

pirog commented Oct 11, 2016

@bmcclure it should be pretty easy to add this manually (and we should add it in by default). Here are some instructions to add it yourself and we will get this into the next release.

# SSH into the Kalabox2 VM
# Check docs for clarity if you dont know how to do this
docker-machine ssh Kalabox2 

# Docker exec into the appserver for your app
docker exec -i -t myapp_appserver_1 bash

# This will drop you into a shell inside your appserver container
docker-php-ext-install bcmath
docker-php-enable bcmath
exit

# Drop back out to VM
exit

# Drop back out to host
kbox restart
@pirog

This comment has been minimized.

Show comment
Hide comment
@pirog

pirog Oct 11, 2016

Member

NOTE: WE might also want to add bz2 and calendar

Member

pirog commented Oct 11, 2016

NOTE: WE might also want to add bz2 and calendar

@bmcclure

This comment has been minimized.

Show comment
Hide comment
@bmcclure

bmcclure Oct 11, 2016

Contributor

Thank you, this fixed things for now!

Side note, docker-php-enable returned "command not found". There is a docker-php-ext-enable command, however that indicated that bcmath was already enabled. Perhaps that step is no longer needed?

Contributor

bmcclure commented Oct 11, 2016

Thank you, this fixed things for now!

Side note, docker-php-enable returned "command not found". There is a docker-php-ext-enable command, however that indicated that bcmath was already enabled. Perhaps that step is no longer needed?

@pirog pirog closed this Oct 12, 2016

@pirog

This comment has been minimized.

Show comment
Hide comment
@pirog

pirog Oct 12, 2016

Member

@bmcclure bcmath and bz2 and calendar will be in the next release

Member

pirog commented Oct 12, 2016

@bmcclure bcmath and bz2 and calendar will be in the next release

@bmcclure

This comment has been minimized.

Show comment
Hide comment
@bmcclure

bmcclure Jan 5, 2017

Contributor

@pirog I'm getting errors again (using kbox drush, curious if bcmath is still included by default?

Failed: RuntimeException: The bcmath extension is required by NumberFormatter.
Contributor

bmcclure commented Jan 5, 2017

@pirog I'm getting errors again (using kbox drush, curious if bcmath is still included by default?

Failed: RuntimeException: The bcmath extension is required by NumberFormatter.
@pirog

This comment has been minimized.

Show comment
Hide comment
@pirog

pirog Jan 5, 2017

Member
Member

pirog commented Jan 5, 2017

@bmcclure

This comment has been minimized.

Show comment
Hide comment
@bmcclure

bmcclure Jan 5, 2017

Contributor
Contributor

bmcclure commented Jan 5, 2017

@pirog

This comment has been minimized.

Show comment
Hide comment
@pirog

pirog Jan 6, 2017

Member

it does run in a different container! might be worth posting an issue over on
https://github.com/RobLoach/drush-docker

Member

pirog commented Jan 6, 2017

it does run in a different container! might be worth posting an issue over on
https://github.com/RobLoach/drush-docker

@bmcclure

This comment has been minimized.

Show comment
Hide comment
@bmcclure

bmcclure Jan 6, 2017

Contributor

Thanks! Tracking that now at RobLoach/drush-docker#43, so seems like this issue can remain closed.

Contributor

bmcclure commented Jan 6, 2017

Thanks! Tracking that now at RobLoach/drush-docker#43, so seems like this issue can remain closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment