Drupal.org
Issues

Allow .make files in install profiles

Posted by alex_b on September 24, 2009 at 5:18pm
Project:Drupal.org infrastructure
Component:Packaging
Category:feature request
Priority:normal
Assigned:dww
Status:closed (fixed)

Issue Summary

We are currently pulling together an install profile for Managing News: http://drupal.org/project/managingnews http://www.managingnews.com

We're planning to package Managing News with a http://drupal.org/project/drush_make based script. This will require us to add a .make file to the profile, which is currently not possible (see cvs commit response below).

I'd like to request to allow .make files in profiles besides .txt and .profile files.

** Access denied: this file is not allowed:
** contributions/profiles/managingnews/managingnews.make
** The only files allowed in contributions/profiles are .txt and .profile
** Packaged files (.gz, .tar, .tgz, and .zip) are not allowed anywhere.
** License files (LICENSE.txt) are not allowed, since everything committed
** to the Drupal CVS repository must be licensed under the GPL.
** You should never commit .svn/* or .bzr/* files to the CVS repository.
cvs commit: Pre-commit check failed
cvs [commit aborted]: correct above errors first!
Login or register to post comments

Comments

#1

Posted by Dave Reid on September 24, 2009 at 5:49pm

We're also going to need to open this up to .install, etc since install profiles as essentially modules in D7.

#2

Posted by dww on September 24, 2009 at 8:59pm
Assigned to:Anonymous» dww
Status:active» fixed

.make and .install are both allowed now. Updated the error message, too.

#3

Posted by alex_b on September 25, 2009 at 3:30pm

dww: thanks for the quick response.

And the first .make file went up: http://cvs.drupal.org/viewvc.py/drupal/contributions/profiles/managingne...

#4

Posted by dww on September 26, 2009 at 4:28am
Status:fixed» active

actually: why does this have to be a separate file? Why not just put this metadata in each profile's .info file?

#5

Posted by dww on November 19, 2009 at 11:01pm
Status:active» fixed

Well, alex_b never answered, but meanwhile, I've come up with about a dozen reasons why we want the build info in a separate .make file instead of in the .info file. ;) Back to fixed.

#6

Posted by alex_b on November 19, 2009 at 11:29pm

dww: sorry, I've completely missed your question above. My email notifications weren't properly configured.

/me fixing notifications now.

#7

Posted by dww on November 20, 2009 at 4:51am

@alex_b: no worries. If you wanted to review/test/RTBC #638030: Hide *.make files via .htaccess that'd more than make up for it. ;)

Cheers,
-Derek

#8

Posted by System Message on December 4, 2009 at 5:00am
Status:fixed» closed (fixed)

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

nobody click here