PHP requirements need to be added to the README

geodaniel - February 5, 2007 - 16:36
Project:Mapbuilder
Version:5.x-1.x-dev
Component:Documentation
Category:task
Priority:normal
Assigned:jpulles
Status:needs review
Description

Please could you add dependency information to the README.txt file, e.g. cURL/libcurl and Mapbuilder. The Mapbuilder one is sort of implied, but I didn't realise about curl.

#1

jpulles - February 6, 2007 - 19:01
Assigned to:Anonymous» jpulles
Status:active» fixed

Requirements paragraph added to README.txt.

#2

lambert - February 20, 2007 - 18:38
Title:Add dependency information to documentation» PHP requirements need to be added to the README

Currently, this only runs under PHP5.

Simplest and best would be the phpinfo() file, in case any modules need to be compiled in (or so we can be confident that no modules need to be).

Thanks!

#3

lambert - February 22, 2007 - 02:14
Title:PHP requirements need to be added to the README» PHP requirements need to be added to the README is NOT fixed

Right now it reads:

This module requires:
- the curl extension for php (e.g. package php5-curl under debian linux)
- the community mapbuilder library

It might be useful to expand it like so:

This module requires:

1. PHP5. It will not run under PHP4.
2. PHP5 must be configured with --with-xml and --with-curl
3. PHP5 must be configured with one of the following graphics libraries: GD .... [Here I'm just working from errors thrown on the status page when I enabled the module]

******

The requirement for the mapbuilder library isn't really a systems requirement like the above, so it should be explained in its own section.

******

Finally, ALL the configuration files that need to be adjusted should be listed. Also, would be useful if the XML in the config files was commented or IDed, so the documentation could cross reference it.

Thanks for the module, when I get it going it will be great, but right now....

#4

geodaniel - February 22, 2007 - 10:14
Title:PHP requirements need to be added to the README is NOT fixed» PHP requirements need to be added to the README
Status:fixed» active

#5

jpulles - February 22, 2007 - 12:01
Status:active» needs review

I don't build PHP myself, so I don't know which configuration options to use. Please have a look at this patch.

AttachmentSize
README.txt_3.patch 512 bytes

#6

lambert - February 22, 2007 - 15:47

Hi, thanks. The patch is:

PHP 5 with support for DOM functions

I would say instead:

PHP 5 with support for curl and DOM

just to be parallel to the two --with* configuration options that are about to follow.

 
 

Drupal is a registered trademark of Dries Buytaert.