Re: [notmuch] Bug with commit 2e96464f9705be4ec772280cad71a6c9d5831e6f

Subject: Re: [notmuch] Bug with commit 2e96464f9705be4ec772280cad71a6c9d5831e6f

Date: Sat, 16 Jan 2010 02:42:39 +0100 (CET)

To: Ali Polatel

Cc: notmuch@notmuchmail.org

From: racin@free.fr


I still confirm the bug.

The problem is due to relying on non-standardized fields of directory entries (i.e. d_type), which don't behave the
same on reiserfs than on ext2 (I use reiserfs). 

The following ugly patch "solves" my problem.

diff --git a/notmuch-new.c b/notmuch-new.c
index b740ee2..42d9f89 100644
--- a/notmuch-new.c
+++ b/notmuch-new.c
@@ -359,7 +359,7 @@ add_files_recursive (notmuch_database_t *notmuch,
 
            if (! S_ISREG (st.st_mode))
                continue;
-       } else if (entry->d_type != DT_REG) {
+       } else if (entry->d_type != DT_REG && entry->d_type != 0) {
            continue;
        }

With this patch, mails are added by the database, but some directories also have d_type set to 0
and can slip through.

It seems that the only portable way to test for directory types is stat, which is expensive... I don't known
what's the best way to go.

Matthieu


----- Mail Original -----
De: "Ali Polatel" <alip@exherbo.org>
À: notmuch@notmuchmail.org
Envoyé: Mercredi 13 Janvier 2010 11h21:52 GMT +01:00 Amsterdam / Berlin / Berne / Rome / Stockholm / Vienne
Objet: Re: [notmuch] Bug with commit 2e96464f9705be4ec772280cad71a6c9d5831e6f

Ali Polatel yazmış:
> racin@free.fr yazmış:
> > Hello,
> > 
> > I just updated notmuch and now notmuch new cannot update my mail anymore... It tells me that there are
> > 700 files found, but tells that there's no new mail.
> > 
> > I did a git bisect, which tells me the first bad commit is commit 2e96464f9705be4ec772280cad71a6c9d5831e6f.
> > 
> > I did not try to use the new xapian database or to update xapian; maybe this is the problem.
> > 
> > I tested with several tools to get mail in the maildir format, including mb2md and getmail, and I always get the problem.
> Same problem here, I tried upgrading xapian to xapian-core-1.1.3_svn13824, the problem persists.
> Here's what happens here:
> 3074 alip@harikalardiyari> rm -fr .maildir/.notmuch
> 3075 alip@harikalardiyari> notmuch new
> Found 210302 total files (that's not much mail).
> No new mail.
> 3076 alip@harikalardiyari> notmuch search from:alip@exherbo.org
> 3077 alip@harikalardiyari>
> 

After doing git bisect which gave random results and testing notmuch
using cworth's notmuch-test script on another file system I figured out
my problem was due to file system corruption.
Sorry for the noise...

-- 
Regards,
Ali Polatel

_______________________________________________
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch
patch (application/octet-stream)

Thread: