Pretty much all of these are removing the 'url' property from Drupal.org-hosted Git repos and specifying 'version' property on Git clones (see http://drupal.org/node/1432190). I found a couple of other things that are probably legitimate bugs ([] is in the wrong place for views_attach; you're doing version = [git hash] in addition to revision = [git hash])

There are three outstanding issues:

1) Acquia Search:

projects[acquia_search][type] = "module"
projects[acquia_search][download][type] = "get"
projects[acquia_search][download][url] = "http://network.acquia.com/files/downloads/acquia-search-current.tar.gz"
projects[acquia_search][subdir] = "acquia"

'projects' is intended only for Drupal.org-hosted projects. This is under discussion at #1427762: Allow modules/themes not hosted on Drupal.org to be packaged into distribution .make files.. But then the question is why this project isn't at http://drupal.org/project/acquia_search?

2) pulling from a sandbox:

projects[tidy_node_links][version] = 1.x-dev
projects[tidy_node_links][download][type] = "git"
projects[tidy_node_links][download][url] = "http://git.drupal.org/sandbox/ezrag/1344006.git"
projects[tidy_node_links][download][revision] = "6.x-1.x"
projects[tidy_node_links][type] = "module"
projects[tidy_node_links][subdir] = "contrib"

The packaging script currently doesn't allow pulling code from git sandboxes. This is being tracked at #1432326: Support for git clones from Drupal.org sandboxes. But is there a reason this isn't a full project?

3) The GetID3 library will get rejected until #1432328: Add getid3 library to whitelist is resolved.

CommentFileSizeAuthor
commons-drupalorg.patch6.81 KBwebchick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ezra-g’s picture

Title: Changes to .make file to allow Commons to work with new Drupal.org packaging system » Initial changes to .make file to allow Commons to work with new Drupal.org packaging system
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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