Installer fails on some systems: glob() is not multiplatform.

From the PHP manual http://www.php.net/glob (function glob() ):

-This function isn't available on some systems (e.g. old Sun OS).
-The GLOB_BRACE flag is not available on some non GNU systems, like Solaris.

So, we should use an alternative solution.

The attached patch fixes this by using another alternative technique to detect database drivers.

The same patch, but closing the dir handle.

Where else do we require glob() ? If a module require it, then it can use a simple similar approach to the one proposed in this patch. We also have a file_scan_directory function that can be used for similar purposes.

1.

$dir = DRUPAL_ROOT . './includes/database';

Should be:

$dir = DRUPAL_ROOT . '/includes/database';

2. This should really use file_scan_directory directly. It does not currently probably because of Crell willingness to keep the database layer completely separate from the rest of Drupal. We should probably generalize fsd() and move it to bootstrap.inc.

Damien, not sure if file_scan_directory can be used to scan directories. It recurse through directories, but only returning files. To detetect database drivers we are looking for directories, not for files. What else do you suggest?

Well, here is a patch that uses file_scan_directory. Work as expected.

This patch applies cleanly to HEAD and works as expected on mac os x leopard and ubuntu.

Committed to CVS HEAD. Thanks!