geoPHP is now dual-licensed GPLv2 and Modified BSD, so we can have a module!

Module is here: http://drupal.org/project/geophp

No need for the libraries module or to download anything.

CommentFileSizeAuthor
#1 geophp_dep.patch5.81 KBmichaelfavia

Comments

michaelfavia’s picture

michaelfavia commented
StatusFileSize
new geophp_dep.patch5.81 KB

Quick shot at removing instructions and add dep to info file with a little module refactor thrown in for good measure.

Looked like it would work but not at a computer that can test it right now.

I basically removed the requirements sections we can now punt that to geophp to get right. and dropped all the statsic library loading in favor of geophps implementation. By adding the module as a dep i was able to avoid all the module_exists overhead calls as well.

michaelfavia’s picture

michaelfavia commented
Status: Active » Needs review

Sorry for the bugspam marking to needs review

Brandonian’s picture

Brandonian commented
Status: Needs review » Fixed

Committed to 2.x (with testing and adjustments :-)). Thanks @michaelfavia.

Next we'll need to make sure we warn for upgrades when we eventually push a 7.x-2.0 release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.