Allow .inc files in contributions/profiles

mikey_p - June 5, 2008 - 23:52
Project:Drupal.org infrastructure
Component:CVS
Category:feature request
Priority:normal
Assigned:Unassigned
Status:patch (reviewed & tested by the community)
Description

Files ending in .inc are not allowed in the profile directory. This should be fixed to allow .profile, .txt and .inc as well.

#1

mikey_p - June 11, 2008 - 21:03

#2

webchick - June 16, 2008 - 01:08

As noted in IRC, the way to change this would be to patch the $xcvs['commit_deny_files'] variable in http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/cvslog/xcvs.... The commented out version is the one used on drupal.org.

If this request is accompanied by a patch, it'd be much easier for dww or someone to implement.

#3

mikey_p - June 25, 2008 - 20:32
Status:active» patch (code needs review)

Here ye go.

In all honestly I don't know how one is supposed to know that a infrastructure specific settings just happens to be stored as a comment in a module committed to CVS, but thanks to angie, here's a patch.

AttachmentSize
cvslog-xcvs-267313-003.patch1.12 KB

#4

aclight - June 25, 2008 - 20:36
Status:patch (code needs review)» patch (code needs work)

The code comment also needs to be changed to reflect the change in file extensions allowed.

#5

mikey_p - June 25, 2008 - 20:39
Status:patch (code needs work)» patch (code needs review)

Done

AttachmentSize
cvslog-xcvs-267313-005.patch1.32 KB

#6

aclight - June 25, 2008 - 20:50
Status:patch (code needs review)» patch (reviewed & tested by the community)

Here's the same patch without the change in the CVS metadata.

Code looks good. Untested however.

I'd like this to be committed and used on d.o because #231640: Auto-generate some release nodes introduces a .inc for the drupalorg_testing profile.

AttachmentSize
cvslog-xcvs-267313-006.patch1.14 KB

#7

mikey_p - June 25, 2008 - 20:54

Also, thought I'd throw out another example of the COD distribution which has taken to placing data/code in .txt files to workaround this limitation (since they have ALOT of data).

http://cvs.drupal.org/viewvc.py/drupal/contributions/profiles/cod/cck/

 
 

Drupal is a registered trademark of Dries Buytaert.