Re: python CFFI bindings integration into notmuch build/test

2019-11-17 Thread David Bremner
Floris Bruynooghe writes: > > I still have a bunch of non-existing messages with weird unicode message > IDs in them, but no associated filename just an empty string. AFAIK > notmuch_database_remove_message() needs a filename though, I've tried > removing the empty file which does not fail but

Re: python CFFI bindings integration into notmuch build/test

2019-11-17 Thread Floris Bruynooghe
On Sat 16 Nov 2019 at 10:51 -0500, David Bremner wrote: > Floris Bruynooghe writes: >> Anyway, this looks good. Would you like some changes, e.g. the rename >> to notmuch2 or so as patches? What's the next step. > > If you could look at the rename that would be great. Patches on top of >

Re: python CFFI bindings integration into notmuch build/test

2019-11-16 Thread David Bremner
Floris Bruynooghe writes: > Anyway, this looks good. Would you like some changes, e.g. the rename > to notmuch2 or so as patches? What's the next step. If you could look at the rename that would be great. Patches on top of wip/cffi, or a ref for me to pull both sound fine. > Kind of

Re: python CFFI bindings integration into notmuch build/test

2019-11-16 Thread David Bremner
Tomi Ollila writes: > On Thu, Nov 14 2019, Floris Bruynooghe wrote: > >> Hi, >> >> Thanks for carrying on with this! >> >> I'm a little confused with how to follow this, is the current state of >> the code somewhere in a repo/branch where I can try things out and make >> changes from? > > In

Re: python CFFI bindings integration into notmuch build/test

2019-11-14 Thread Floris Bruynooghe
On Thu 14 Nov 2019 at 23:24 +0100, Floris Bruynooghe wrote: > On Thu 14 Nov 2019 at 22:20 +0200, Tomi Ollila wrote: >> In git://notmuchmail.org/git/notmuch David has ref origin/wip/cffi >> which contains related changes -- You can fetch the code while waiting >> for more collaboration

Re: python CFFI bindings integration into notmuch build/test

2019-11-14 Thread Floris Bruynooghe
On Thu 14 Nov 2019 at 22:20 +0200, Tomi Ollila wrote: > In git://notmuchmail.org/git/notmuch David has ref origin/wip/cffi > which contains related changes -- You can fetch the code while waiting > for more collaboration instructions from David. Aha, thanks! Floris

Re: python CFFI bindings integration into notmuch build/test

2019-11-14 Thread Tomi Ollila
On Thu, Nov 14 2019, Floris Bruynooghe wrote: > Hi, > > Thanks for carrying on with this! > > I'm a little confused with how to follow this, is the current state of > the code somewhere in a repo/branch where I can try things out and make > changes from? In git://notmuchmail.org/git/notmuch

Re: python CFFI bindings integration into notmuch build/test

2019-11-14 Thread Floris Bruynooghe
Hi, Thanks for carrying on with this! I'm a little confused with how to follow this, is the current state of the code somewhere in a repo/branch where I can try things out and make changes from? On Tue 05 Nov 2019 at 22:22 -0400, David Bremner wrote: > Tomi Ollila writes: > > >> alternative:

Re: python CFFI bindings integration into notmuch build/test

2019-11-05 Thread David Bremner
Tomi Ollila writes: > alternative: > >... >print('Invoking: {}'.format(' '.join(cmd))) > >def preexec_fn(): os.environ['NOTMUCH_CONFIG'] = str(cfg_fname) > >proc = subprocess.run(cmd, timeout=5, preexec_fn=preexec_fn) >... > > The unix fork ...

Re: python CFFI bindings integration into notmuch build/test

2019-11-05 Thread Tomi Ollila
On Sun, Nov 03 2019, David Bremner wrote: > > @@ -32,10 +33,11 @@ def notmuch(maildir): > """ > cfg_fname = maildir.path / 'notmuch-config' > cmd = ['notmuch'] + list(args) > -print('Invoking: {}'.format(' '.join(cmd))) > +env = os.environ.copy() > +

Re: python CFFI bindings integration into notmuch build/test

2019-11-04 Thread David Bremner
David Bremner writes: > This series is on top of a slightly modified version [1] of > > id:20191008210312.20685-2-f...@devork.be > > If you want to try out the new bindings, you can try the branch wip/cffi in > the repo > https://git.notmuchmail.org/git/notmuch > > One current annoyance is

python CFFI bindings integration into notmuch build/test

2019-11-03 Thread David Bremner
This series is on top of a slightly modified version [1] of id:20191008210312.20685-2-f...@devork.be If you want to try out the new bindings, you can try the branch wip/cffi in the repo https://git.notmuchmail.org/git/notmuch One current annoyance is that the version of pytest on travis