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

Comments

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Active » Needs review
StatusFileSize
new backup_migrate.libraries_support.patch2.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

ronan commented

Phenomenal, thank you!

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

akalata’s picture

akalata commented

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

ronan’s picture

ronan commented
Status: Needs review » Fixed

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

shaisamuel’s picture

shaisamuel commented

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

jordanmagnuson commented
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

ronan commented
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.