Closed (fixed)
Project:
Panopoly
Version:
7.x-1.0-beta2
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
19 Mar 2012 at 22:59 UTC
Updated:
4 Jan 2014 at 01:39 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
ebeyrent commentedComment #2
floptikal commentedI tried to move manually, but it still seems to fail....any idea?
Comment #3
stan turyn commentedIf 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.
Comment #4
pirog commentedshould point out that it is solrphpclient/SolrPhpClient and not phpsolrclient/PhpSolrClient
Comment #5
stan turyn commentedSpot on, pirog. My bad, wasn't being attentive.
Comment #6
socialnicheguru commentedwhere do you make these changes?
Comment #7
ezra-g commentedThe 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.
Comment #8
ezra-g commentedComment #9
socialnicheguru commentedEven 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.
Comment #10
ezra-g commented@SocialNicheGuru It's a patch to Drush make, so it only takes effect when the distribution is re-packaged.
Comment #11
jimthunderbird commentedI follow INSTALL.txt and download the latest SolrPhpClient, put it under sites/all/libraries directory and it works.
Comment #12
pirog commentedagree 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.
Comment #13
jimthunderbird commentedAgree 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.
Comment #14
pirog commentedright, 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.
Comment #15
pirog commentedalso, 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 .
Comment #16
playfulwolf commentedgetting the same error on WAMP install. (beta2)
Comment #17
brandenlhamilton commentedI 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
Comment #18
dsy73 commentedThe 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
search_api_solr.module
Comment #19
populist commentedThanks 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.