Re: [O] [PATCH] org-contacts.el: Catch 'nextfile in org-contacts-db

2015-07-04 Thread phcrama . ebiz
Hi Greg,   thank you for your remarks.   I agree with all of them but like the idea that there's one central point checking what a valid org agenda file is (even if for now only the file existence is checked).  So let me propose a different patch (attached).   Best regards, -- Philippe Cr

Re: [O] [PATCH] org-contacts.el: Catch 'nextfile in org-contacts-db

2015-07-01 Thread Daimrod
phcrama.e...@gmx.com writes: > Hi Nicolas and Greg, Hi Philippe, sorry for the late reply :( > Thank you for applying my initial patch. > > Inspired by Greg's improvements to my patch, I'd like to propose the attached > (presumptuously named) better-patch that I believe fixes the TODO of my >

Re: [O] [PATCH] org-contacts.el: Catch 'nextfile in org-contacts-db

2015-06-25 Thread phcrama . ebiz
Hi Nicolas and Greg, Thank you for applying my initial patch. Inspired by Greg's improvements to my patch, I'd like to propose the attached (presumptuously named) better-patch that I believe fixes the TODO of my initial patch. Sorry it isn't quite in the format of the previous one/recommended

Re: [O] [PATCH] org-contacts.el: Catch 'nextfile in org-contacts-db

2015-06-23 Thread Nicolas Goaziou
Hello, Philippe Crama writes: > This is a small patch to catch the 'nextfile thrown by > `org-check-agenda-file' in `org-contacts-db'. Applied. Thank you. Regards, -- Nicolas Goaziou

Re: [O] [PATCH] org-contacts.el: Catch 'nextfile in org-contacts-db

2015-06-21 Thread Daimrod
Philippe Crama writes: > Hi, Hello Philippe, > This is a small patch to catch the 'nextfile thrown by > `org-check-agenda-file' in `org-contacts-db'. > > I don't know how to fix the TODO in the comment myself, sorry. At > least, with the patch, the execution doesn't abort anymore. Thanks for

[O] [PATCH] org-contacts.el: Catch 'nextfile in org-contacts-db

2015-06-14 Thread Philippe Crama
Hi, This is a small patch to catch the 'nextfile thrown by `org-check-agenda-file' in `org-contacts-db'. I don't know how to fix the TODO in the comment myself, sorry. At least, with the patch, the execution doesn't abort anymore. Regards, -- Philippe Crama ===File c:/msys64/tmp/org-mode/0001