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!

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

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

dww’s picture

dww
we/he/they
commented
Assigned: Unassigned » dww
Status: Active » Fixed

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

alex_b’s picture

alex_b commented

dww: thanks for the quick response.

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

dww’s picture

dww
we/he/they
commented
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?

dww’s picture

dww
we/he/they
commented
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.

alex_b’s picture

alex_b commented

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

/me fixing notifications now.

dww’s picture

dww
we/he/they
commented

@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

Status: Fixed » Closed (fixed)

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