playing around with the CVS version of the location module. i came across an error resulting from a call to bccomp() which is part of bcmath, which isn't generally enabled (AFAIK) with standard PHP distributions (well, at least not Fedora).

you may want to consider this before requiring it for the next release of location.module.

btw, i'm using the 4.6 location.module and am finding it rather cool. a drupal instance i'm working on is going to need geographic awareness (and ultimately google maps integration). i'm considering location.module as a starting point. thanks!

Comments

ankur’s picture

ankur commented
Status: Active » Fixed

A fix has been committed to HEAD, DRUPAL-4-6.

The only affected file is location.module (no db changes).

To see exactly what changed, please consult:
http://cvs.drupal.org/viewcvs/drupal/contributions/modules/location/loca...

-Ankur

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
rjbrown99’s picture

rjbrown99 commented

For what it's worth, the Drupal 6 OpenID module also calls bccomp() which requires this package. On Fedora, my fix was yum install php-bcmath and then a restart of the webserver.