Putting PHP libraries inside module directories causes all kinds of source control and drush hell.

It would be great if this module could support the libraries api: http://drupal.org/project/libraries (and if it's not there, just look in sites/all/libraries anyway)

CommentFileSizeAuthor
#1 backup_migrate.libraries_support.patch2.67 KBrfay
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rfay’s picture

Status: Active » Needs review
FileSize
2.67 KB

And here's a patch to do it. This supports current installations, library locations provided by libraries api, and also just sites/all/libraries.

ronan’s picture

Phenomenal, thank you!

I'll get this committed and backported as soon as I have a minute.

akalata’s picture

Thanks @rfay! I was able to apply #1 to 6.x-2.x-dev without issue.

ronan’s picture

Status: Needs review » Fixed

Commited to both 6.x and 7.x branch. Thanks!

shaisamuel’s picture

This is great.

The documentation and the error message when the library is missing should update to reflect the new and better position.

Status: Fixed » Closed (fixed)

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

JordanMagnuson’s picture

Component: Code » Documentation
Status: Closed (fixed) » Needs work

Documentation and error messages still need to be updated.

If the user has libraries api installed, module should advise to place S3.php in sites/all/libraries/s3-php5-curl.

ronan’s picture

Issue summary: View changes
Status: Needs work » Fixed

This is better documented and has a better error now.

Status: Fixed » Closed (fixed)

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