Off late, developers have (accidentally) been committing files and directories unrelated to their projects. For e.g., Eclipse files such as .project, .settings, .cache etc., .cvsignore, .svn, thumbs.db. If possible, these commits should be denied by the server. A whitelist of files beginning with a period should be a good start (.htaccess, .htpasswd etc. should be allowed; the rest disallowed).

This could be a project module feature request too. If so, please reassign.

Thanks,
-K

CommentFileSizeAuthor
#1 CVS_junk.png18.32 KBZen

Comments

Zen’s picture

Zen commented
StatusFileSize
new CVS_junk.png18.32 KB

Attached image shows some erroneous commits: .settings, mass_contact and pathsearch all look dodgy.

-K

dww’s picture

dww
we/he/they
commented
Status: Needs review » Active

this is definitely not a cvs or project* issue, there's already a knob for this: $xcvs['commit_deny_files']. so, we just need to make it more restrictive if that's what we want. current value is:

$xcvs['commit_deny_files'] = array('@^contributions/profiles.*(?<!\.profile|\.txt)$@', '@^.*\.(gz|tgz|tar|zip)$@');

if someone wants to come up with a better regexp, just post it here and i'll review it and install it in our repo.

thanks,
-derek

damien tournoud’s picture

damien tournoud commented
Status: Active » Closed (won't fix)

Don't apply anymore now that we are migrating to Git.

Component: CVS » Other