cyrusimap / bugzilla-migrate-test

0 stars 1 forks source link

reconstruct should recurse smarter #697

Open brong opened 21 years ago

brong commented 21 years ago

From: Rob Siemborski Bugzilla-Id: 2125 Version: 2.4.17 Owner: Ken Murchison

brong commented 21 years ago

From: Rob Siemborski

Possibly related to Bug #2124.

Reconstruct should recruse into all subdirectories and look for #####. files in addition to cyrus.header files, since a cyrus.header file may have been inadvertantly removed, but message files may still remain.

Also, recursion should go as deep as possible (apparently not doing so now, but may be releated to 2124).

brong commented 21 years ago

From: Rob Siemborski

the lack of a recursion was an unrelated issue.

brong commented 11 years ago

From: Andrew Morgan

Still present in v2.4.17.

Here is an example of how to tickle this bug:

cd /var/spool/cyrus/mail/m/user/morgan/

mkdir -p empty/test

chmod -R 700 empty

chown -R cyrus:mail empty

touch empty/test/cyrus.header

chmod 600 empty/test/cyrus.header

chown cyrus:mail empty/test/cyrus.header

su cyrus -c '/usr/local/cyrus/bin/reconstruct -x -f user.morgan'

user.morgan

su cyrus -c '/usr/local/cyrus/bin/reconstruct -r -x -f user.morgan.empty'

(no output)

su cyrus -c '/usr/local/cyrus/bin/reconstruct -r -x -f user.morgan.empty.test'

(no output)

None of these reconstruct attempts put the mailbox user.morgan.empty.test in the mailbox list.

Even if I put a message file in the empty/test/ directory, reconstruct refuses to find the mailbox.

However, if I create a cyrus.header in the empty/ directory, it works:

touch empty/cyrus.header

chmod 600 empty/cyrus.header

chown cyrus:mail empty/cyrus.header

su cyrus -c '/usr/local/cyrus/bin/reconstruct -x -f user.morgan'

user.morgan discovered user.morgan.empty user.morgan.empty discovered user.morgan.empty.test

brong commented 10 years ago

From: Bron Gondwana

Batch moving bugs that won't be in 2.5