Libraries path for requiremetns [#1488790]

Hi,

i see install requires sphinxapi.php to be copied in an unusual path ( /includes/api )

function sphinx_requirements($phase) {
  if($phase == 'runtime') {
    // TODO: Add check for admin include file name
    $t        = get_t();
    $path     = './'. drupal_get_path('module', 'sphinx') ."/includes/api";
    $api_file = $path ."/sphinxapi.php";
    if (file_exists($api_file)) {
    ...

Is there a reason we are not following best practise of using sites/all/libraries ?

Thanks for the great job!

Comments

Comment #1

nestor.mata CreditAttribution: nestor.mata commented 12 April 2012 at 14:01

Status:

Active

» Fixed

Hi Augusto,

The reason is because libraries module is actually newer than this module, it was not available when this module started.
But, this has already being integrated in the dev version and will soon be available in the next stable version.
Right now I'm working on several improvements and a Drupal 7 version.

Thanks for the sugestion.

Nestor