[PATCH 0/3] build fixes

2010-06-15 Thread Dirk Hohndel
On Mon, 14 Jun 2010 17:17:00 +0300, Felipe Contreras  wrote:
> Hi,
> 
> Is anything wrong with these patches?

cworth - our fearless leader - is extremely busy with his day job right
now. He tends to respond to all the patches on the mailing list as soon
as he finds time; sometimes within minutes, sometimes it takes weeks... 
you appear to have hit one of the slow spots.

/D

-- 
Dirk Hohndel
Intel Open Source Technology Center


[PATCH] remove message archiving from show-advance-and-archive

2010-06-15 Thread Sebastian Spaeth
On 2010-06-09, Dirk Hohndel wrote:
> Oh - and I really want a way to do surgery on threads. Merge threads to
> fix Blackberry users breaking threads. And split threads for
> hijackers... 

Which will open other cans of worms :-)
Have fun splitting the below thread between msg 1 and 2. Where will msg
3 end up? :-) (OK, OK, *usually* you should get a reply-to header...)

Msg id1
  Blah...

Msg id2
References: id1
  Bloh...

Msg id3
References: id1 id2
  Bleh...
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: 
<http://notmuchmail.org/pipermail/notmuch/attachments/20100615/373dffb6/attachment.pgp>


[PATCH] remove message archiving from show-advance-and-archive

2010-06-15 Thread Sebastian Spaeth
> For that, I think I want the current '|' binding to pipe the current
> message and then a new binding ("M-|" ?) to pipe every (open) message in
> the thread.
+1 

> Which makes me think that other operations should work similarly. '+'
> and '-' should change tags on the current message (as they do currently)
> and then new "M-+" and "M--" could change tags on all (open) messages in
> the thread.
+1 

> That would highlight the current 'a' as out of place since it's
> currently archiving every message in the thread. So I'd then fix it to
> be 'a' for the current message and "M-a" for every (open) message in the
> thread.

It would indeed be out of place, but I would like to voice support for
the status-quo. In General I like to press as few keys as possible
(since I use emacs my meta keys are getting worn out quite quickly).

and I usually archive whole threads rather than single messages, so I
would vote for "a" as archive thread. Is it inconsistent? Yes. Does it
let me press fewer keys? Yes...

Anyway, I can always customize it the way I like it best :-)

Sebastian
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: 
<http://notmuchmail.org/pipermail/notmuch/attachments/20100615/cac121fa/attachment.pgp>


Python bindings and Xapian exceptions

2010-06-15 Thread Sebastian Spaeth
>   A Xapian exception occurred finding message: The revision being read
>   has been discarded - you should call Xapian::Database::reopen() and
>   retry the operation.
> It makes the Python bindings almost useless to me.

Not sure, if the python bindings should simply drop and reopen a
database connection in that case? But I am not sure if libnotmuch.so
still simply exits on such an error case.

The proper fix, and a reason why I am not immediately hacking around in
the python bindings is that notmuch the library would actually return a
proper error value rather than print to stderr and quit.

Happy to receive suggestions on how to handle this best from the python
side of things.

Sebastian
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: 
<http://notmuchmail.org/pipermail/notmuch/attachments/20100615/a63b69e4/attachment.pgp>


Python bindings and Xapian exceptions

2010-06-15 Thread David Edmondson
On Tue, 15 Jun 2010 11:03:55 +0200, "Sebastian Spaeth"  wrote:
> >   A Xapian exception occurred finding message: The revision being read
> >   has been discarded - you should call Xapian::Database::reopen() and
> >   retry the operation.
> > It makes the Python bindings almost useless to me.
> 
> Not sure, if the python bindings should simply drop and reopen a
> database connection in that case? But I am not sure if libnotmuch.so
> still simply exits on such an error case.
> 
> The proper fix, and a reason why I am not immediately hacking around in
> the python bindings is that notmuch the library would actually return a
> proper error value rather than print to stderr and quit.
> 
> Happy to receive suggestions on how to handle this best from the python
> side of things.

Unfortunately I'm an ignorant about the best solution.

Having the Python layer hide the fact that the database was re-opened
seems as though it might be dangerous (in case I mix state from before
and after the re-opening), but I can't give a concrete example of
something that would fail.

Given that exception based programming is common in Python, simple
raising the exception and allowing the application to handle it appears
to make sense.

dme.
-- 
David Edmondson, http://dme.org
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: 
<http://notmuchmail.org/pipermail/notmuch/attachments/20100615/4aaf9188/attachment.pgp>


Re: Python bindings and Xapian exceptions

2010-06-15 Thread Sebastian Spaeth
   A Xapian exception occurred finding message: The revision being read
   has been discarded - you should call Xapian::Database::reopen() and
   retry the operation.
 It makes the Python bindings almost useless to me.

Not sure, if the python bindings should simply drop and reopen a
database connection in that case? But I am not sure if libnotmuch.so
still simply exits on such an error case.

The proper fix, and a reason why I am not immediately hacking around in
the python bindings is that notmuch the library would actually return a
proper error value rather than print to stderr and quit.

Happy to receive suggestions on how to handle this best from the python
side of things.

Sebastian


pgpUhij3YDFKm.pgp
Description: PGP signature
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: [PATCH] remove message archiving from show-advance-and-archive

2010-06-15 Thread Sebastian Spaeth
 For that, I think I want the current '|' binding to pipe the current
 message and then a new binding (M-| ?) to pipe every (open) message in
 the thread.
+1 

 Which makes me think that other operations should work similarly. '+'
 and '-' should change tags on the current message (as they do currently)
 and then new M-+ and M-- could change tags on all (open) messages in
 the thread.
+1 

 That would highlight the current 'a' as out of place since it's
 currently archiving every message in the thread. So I'd then fix it to
 be 'a' for the current message and M-a for every (open) message in the
 thread.

It would indeed be out of place, but I would like to voice support for
the status-quo. In General I like to press as few keys as possible
(since I use emacs my meta keys are getting worn out quite quickly).

and I usually archive whole threads rather than single messages, so I
would vote for a as archive thread. Is it inconsistent? Yes. Does it
let me press fewer keys? Yes...

Anyway, I can always customize it the way I like it best :-)

Sebastian


pgp4SSJqlKkbs.pgp
Description: PGP signature
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: [PATCH] remove message archiving from show-advance-and-archive

2010-06-15 Thread Sebastian Spaeth
On 2010-06-09, Dirk Hohndel wrote:
 Oh - and I really want a way to do surgery on threads. Merge threads to
 fix Blackberry users breaking threads. And split threads for
 hijackers... 

Which will open other cans of worms :-)
Have fun splitting the below thread between msg 1 and 2. Where will msg
3 end up? :-) (OK, OK, *usually* you should get a reply-to header...)

Msg id1
  Blah...

Msg id2
References: id1
  Bloh...

Msg id3
References: id1 id2
  Bleh...


pgpHcICvFPRQc.pgp
Description: PGP signature
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: Python bindings and Xapian exceptions

2010-06-15 Thread David Edmondson
On Tue, 15 Jun 2010 11:03:55 +0200, Sebastian Spaeth sebast...@sspaeth.de 
wrote:
A Xapian exception occurred finding message: The revision being read
has been discarded - you should call Xapian::Database::reopen() and
retry the operation.
  It makes the Python bindings almost useless to me.
 
 Not sure, if the python bindings should simply drop and reopen a
 database connection in that case? But I am not sure if libnotmuch.so
 still simply exits on such an error case.
 
 The proper fix, and a reason why I am not immediately hacking around in
 the python bindings is that notmuch the library would actually return a
 proper error value rather than print to stderr and quit.
 
 Happy to receive suggestions on how to handle this best from the python
 side of things.

Unfortunately I'm an ignorant about the best solution.

Having the Python layer hide the fact that the database was re-opened
seems as though it might be dangerous (in case I mix state from before
and after the re-opening), but I can't give a concrete example of
something that would fail.

Given that exception based programming is common in Python, simple
raising the exception and allowing the application to handle it appears
to make sense.

dme.
-- 
David Edmondson, http://dme.org


pgp9NLrqXna70.pgp
Description: PGP signature
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: [PATCH 0/3] build fixes

2010-06-15 Thread Dirk Hohndel
On Mon, 14 Jun 2010 17:17:00 +0300, Felipe Contreras 
felipe.contre...@gmail.com wrote:
 Hi,
 
 Is anything wrong with these patches?

cworth - our fearless leader - is extremely busy with his day job right
now. He tends to respond to all the patches on the mailing list as soon
as he finds time; sometimes within minutes, sometimes it takes weeks... 
you appear to have hit one of the slow spots.

/D

-- 
Dirk Hohndel
Intel Open Source Technology Center
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch