Moodle Community Sites

git mirror contains empty directories that were removed from CVS

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Won't Fix
  • Component/s: git.moodle.org
  • Labels:
    None

Description

Empty direcotries are not acceptable as they are recongized as proper plugins and cause problems. I just spotted at least /grade/report/visual, removed from CVS. Maybe we should have some empty-directories-removal tool being run automatically? Or is this a part of "common" git/cvs shift?

Activity

Hide
David Mudrak added a comment -

Ping for watchers who might be actually interested or may know some solution

Show
David Mudrak added a comment - Ping for watchers who might be actually interested or may know some solution
Hide
Jordan Tomkinson added a comment -

I'll look into making a fix for this - its a known issue with the cvs->git process

Show
Jordan Tomkinson added a comment - I'll look into making a fix for this - its a known issue with the cvs->git process
Hide
David Mudrak added a comment -

This does not seem to be easily fixable :-/ Once the directories get into the git mirror, they can't be removed as empty directory removal can't be committed...
What works for me is semi-manually removing empty directories using

$ find . -depth -type d -empty
$ find ./lib -depth -type d -empty -exec rmdir {} \;
Show
David Mudrak added a comment - This does not seem to be easily fixable :-/ Once the directories get into the git mirror, they can't be removed as empty directory removal can't be committed... What works for me is semi-manually removing empty directories using
$ find . -depth -type d -empty
$ find ./lib -depth -type d -empty -exec rmdir {} \;
Hide
David Mudrak added a comment -

The manual removing described above works and we are going to switch from CVS hopefully soon.

Show
David Mudrak added a comment - The manual removing described above works and we are going to switch from CVS hopefully soon.

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: