Please rename phpsolrclient to PhpSolrClient and move the library to sites/all/libraries - without these the install fails with 'PhpSolrClient library not found'.

Comments

ebeyrent’s picture

Priority: Normal » Critical
floptikal’s picture

I tried to move manually, but it still seems to fail....any idea?

stan turyn’s picture

If you move and rename it manually before attempting installation it should work with no problem. Make sure you renamed it correctly and make sure its permissions are OK (i.e. web server can read it) and make sure the path is correct (see README). If still no joy, check web server logs for hints.

pirog’s picture

should point out that it is solrphpclient/SolrPhpClient and not phpsolrclient/PhpSolrClient

stan turyn’s picture

Title: Please rename phpsolrclient to PhpSolrClient and move the library to sites/all/libraries » Please rename solrphpclient to SolrPhpClient and move the library to sites/all/libraries

Spot on, pirog. My bad, wasn't being attentive.

socialnicheguru’s picture

where do you make these changes?

ezra-g’s picture

Title: Please rename solrphpclient to SolrPhpClient and move the library to sites/all/libraries » Please rename solrphpclient to SolrPhpClient
Status: Active » Needs review
StatusFileSize
new832 bytes

The library should be placed in a directory called SolrPhpClient, but it would be non-standard for a profile to place its library in sites/all/libraries. Instead, I filed #1500210: _search_api_solr_solrphpclient_path() fails when installing with non-default installation profile, which should be applied to search_api_solr in conjunction with this patch.

ezra-g’s picture

Version: 7.x-1.0-beta1 » 7.x-1.0-beta2
socialnicheguru’s picture

Even after patch, when I install using Aegir I still get this:

Exception Object ( [message:protected] => SolrPhpClient library not found! Please follow the instructions in search_api_solr/INSTALL.txt for installing the Solr search module.

ezra-g’s picture

@SocialNicheGuru It's a patch to Drush make, so it only takes effect when the distribution is re-packaged.

jimthunderbird’s picture

I follow INSTALL.txt and download the latest SolrPhpClient, put it under sites/all/libraries directory and it works.

pirog’s picture

agree with ezra-g that the correct approach here is to scan profile/*/libraries instead of moving the solr client to sites/all/libraries.

if you do move the client to sites/all/libraries it should be known that this is a workaround (read: not a solution) precisely because it is a non-default installation profile and thus the libraries module is not yet enabled. This means that profile/*/libraries is not searched as it normally would be.

jimthunderbird’s picture

Agree on this one, we should consider profile as a self-contained environment so profile/*/libraries make a lot of sense, otherwise, users would need to manually put the solr client libraries in sites/all/libraries folder, a bit painful at the beginning.

pirog’s picture

right, and if we had people put the solr client in sites/all/libz just to get through the install process we would end up with two solr clients in sites/all/libs and /profile/*/libs which is not great. then, after the install process the library module takes over so you would want to remove one of the solr clients.

you might say "well just have drush make put the solr client in sites/all/libraries" to begin with but i don't think you can do this because when panopoly is built it is relative to the profile folder not the drupal root and i dont think you can back out of that folder to get to sites/all/libraries with DM. this is irrelevant anyway because we should be consistent with where we keep our libraries in an installation profile... and that location is profiles/*/libraries.

that being as it is: ezra-g's patch is the way to go.

pirog’s picture

StatusFileSize
new832 bytes

also, i think the patch in #7 should be the inverse of what it is now otherwise the solr client is still not found even with the fix in #1500210: _search_api_solr_solrphpclient_path() fails when installing with non-default installation profile .

playfulwolf’s picture

getting the same error on WAMP install. (beta2)

brandenlhamilton’s picture

I can confirm that applying the following two patches resolved the 'PhpSolrClient library not found' error message.

Patch from issue summary from http://drupal.org/node/1500210 as well as
Patch from comment #15 http://drupal.org/node/1490396#comment-5802250

dsy73’s picture

Issue tags: +apach solr

The previous patches doesn't work for me, I had to rename the library folder to "SolrPhpClient" as you can see in the second patch result:

drupal-org.make

libraries[SolrPhpClient][download][type] = get
libraries[SolrPhpClient][download][url] = http://apps.getpantheon.com/sites/all/libraries/SolrPhpClient.r60.2011-05-04.zip

search_api_solr.module

    else {
      $search_dirs[] = 'sites/all/libraries/SolrPhpClient';
      if (!empty($_GET['profile'])) {
        $search_dirs[] = 'profiles/' . $_GET['profile'] . '/libraries/SolrPhpClient';
      }
    }
populist’s picture

Status: Needs review » Fixed

Thanks for the feedback! The installation profile has been updated to use the correct capitalization and the new version of Search API will soon be updated to address #1500210: _search_api_solr_solrphpclient_path() fails when installing with non-default installation profile.

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