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

CommentFileSizeAuthor
#4 1114744-configuration.patch2.92 KBboombatower

Comments

boombatower’s picture

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.

boombatower’s picture

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

boombatower’s picture

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.

boombatower’s picture

Title: Support configuring SDK using creeper approach » Provide configuration integration and variable overrides
Assigned: Unassigned » boombatower
Status: Active » Needs review
StatusFileSize
new2.92 KB

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

boombatower’s picture

Status: Needs review » Fixed

Committed, along with hook_requirments(). :)

boombatower’s picture

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.

Status: Fixed » Closed (fixed)

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