Download & Extend

Provide configuration integration and variable overrides

Project:AWS SDK for PHP
Version:7.x-3.x-dev
Component:Code
Category:task
Priority:normal
Assigned:boombatower
Status:closed (fixed)

Issue Summary

There is a module that does this already.
http://drupal.org/project/creeper
Why not work with them?

Comments

#1

The other module does not integrate with libraries api, no make file, etc and when I looked had no d7 plans didn't even seem like it was useful not to mention poorly named. Having releases to wrap each update to the library from Amazon is handy and allows drush make scripts to build everything automatically while allowing for specific versions to be specified.

I am happy to collaborate, about the only thing useful the other module provides is the variables for configuration although I need to look at how the library detects that file as it may not be very useful.

If the two are to to merged (both projects are tiny) I think a logical name would be better then creeper. Feel free to cross-post this in the creeper queue or move this issue over there.

#2

Started a general discussion: http://groups.drupal.org/node/139744.

#3

Title:There is a module that does this already. Why not work with them?» Support configuring SDK using creeper approach

After discussion it appear we will merge in variables support and then move all modules that depend on creeper to awssdk project.

Sounds like creeper just provides a template aws.config.inc.php file with variable_get()'s. Curious if we can do anything using libraries API to detect when library is being loaded and automatically include a template version of the file.

#4

Title:Support configuring SDK using creeper approach» Provide configuration integration and variable overrides
Assigned to:Anonymous» boombatower
Status:active» needs review

The following patch works beautifully and will automatically be loaded along with SDK.

AttachmentSize
1114744-configuration.patch 2.92 KB

#5

Status:needs review» fixed

Committed, along with hook_requirments(). :)

#6

Since we have no idea how long it will be before a new AWSSDK is release I went ahead and updated the current stable release.

#7

Status:fixed» closed (fixed)

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

nobody click here