Re: [O] [PATCH v2] fix SETUPFILE pathname expansion with subdirectories

2015-09-19 Thread Kyle Meyer
Richard Hansen writes: > Should I use cd-absolute instead of setting default-directory > directly? It does some sanity checks (among other things) that seem > worthwhile. Looking at these checks, I don't think they are useful in this context, and some change the intended behavior. * One check

Re: [O] [PATCH v2] fix SETUPFILE pathname expansion with subdirectories

2015-09-19 Thread Richard Hansen
On 2015-09-19 18:43, Kyle Meyer wrote: > Richard Hansen writes: > >> (with-temp-buffer >> + (cd (file-name-directory file)) >> (org-mode) >> (insert (org-file-contents file 'noerror)) > >

Re: [O] [PATCH v2] fix SETUPFILE pathname expansion with subdirectories

2015-09-19 Thread Kyle Meyer
Richard Hansen writes: >(with-temp-buffer > +(cd (file-name-directory file)) > (org-mode) > (insert (org-file-contents file 'noerror)) Why not just set default-directory instead of c

[O] [PATCH v2] fix SETUPFILE pathname expansion with subdirectories

2015-09-19 Thread Richard Hansen
* lisp/org-macro.el (org-macro--collect-macros): cd to the directory containing the SETUPFILE before recursing so that relative pathnames in the SETUPFILE are expanded properly. * lisp/org.el (org--setup-collect-keywords): cd to the directory containing the SETUPFILE before recursing so that relat