Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-30 Thread Nicolas Goaziou
Matt Lundin writes: > Thanks for the helpful information. I think the cache would be a nice > way to go, especially if it were combined a timestamp check. E.g., only > files that have been updated since the last publishing should be queried > for titles; otherwise, use the cached file. Unfortuna

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-30 Thread Matt Lundin
Nicolas Goaziou writes: > Hello, > > Matt Lundin writes: > >> Clearly, this is still very inadequate, but it is an improvement. I >> would love to use the built in site-map functions, but they are simply >> to slow for any larger projects. >> >> Could we do something like this to speed it up? >>

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-30 Thread Nicolas Goaziou
Hello, Matt Lundin writes: > Clearly, this is still very inadequate, but it is an improvement. I > would love to use the built in site-map functions, but they are simply > to slow for any larger projects. > > Could we do something like this to speed it up? > > (with-temp-buffer > (insert-file

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Bastien
Matt Lundin writes: >> Please go ahead for this too. > > Gladly! Applied, thanks! -- Bastien

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Bastien
Hi Matt, Matt Lundin writes: > Please *do not apply* the previous patch. Instead, I've attached an > optimized version. Applied, thanks. > All in all this patch + commit 507244d56b055e7595ba94fe89d45c7ddb2559df > modestly improves the performance of org-publish-org-sitemap. On a > directory of

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Matt Lundin
Matt Lundin writes: > Bastien writes: > >> Matt Lundin writes: >> >>> So to be safe, we could do the following in org-publish-find-date and >>> org-publish-find-title... >>> >>> (org-export-with-buffer-copy (org-export-get-environment)) >>> >>> What do you think? >> >> Yes, this sounds right,

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Matt Lundin
Bastien writes: > Hi Matt, > > Matt Lundin writes: > >> The documentation for org-agenda-inhibit-startup says that the default >> is t. However, the default is nil. >> >> , >> | Inhibit startup when preparing agenda buffers. >> | When this variable is `t' (the default), the initialization of

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Matt Lundin
Bastien writes: > Hi Matt, > > Matt Lundin writes: > >> So to be safe, we could do the following in org-publish-find-date and >> org-publish-find-title... >> >> (org-export-with-buffer-copy (org-export-get-environment)) >> >> What do you think? > > Yes, this sounds right, please go ahead. > > I

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Bastien
Hi Matt, Matt Lundin writes: > The documentation for org-agenda-inhibit-startup says that the default > is t. However, the default is nil. > > , > | Inhibit startup when preparing agenda buffers. > | When this variable is `t' (the default), the initialization of > | the Org agenda buffers is

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Bastien
Hi Matt, Matt Lundin writes: > So to be safe, we could do the following in org-publish-find-date and > org-publish-find-title... > > (org-export-with-buffer-copy (org-export-get-environment)) > > What do you think? Yes, this sounds right, please go ahead. I applied your fix in the maint branc

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Matt Lundin
Matt Lundin writes: > Bastien writes: > >> Hi Matt, >> >> Matt Lundin writes: >> >>> This patch fixes a bug in which org-publish makes the following call... >>> (let ((org-inhibit-startup t) (org-mode))) >> >> Applied, thanks! > > This could be solved by wrapping org-export-get-environment with

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Matt Lundin
Matt Lundin writes: > Bastien writes: > >> Hi Matt, >> >> Matt Lundin writes: >> >>> This patch fixes a bug in which org-publish makes the following call... >>> (let ((org-inhibit-startup t) (org-mode))) >> >> Applied, thanks! > > Oops... I see now that org-publish-find-date and org-publish-fin

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Matt Lundin
Bastien writes: > Hi Matt, > > Matt Lundin writes: > >> This patch fixes a bug in which org-publish makes the following call... >> (let ((org-inhibit-startup t) (org-mode))) > > Applied, thanks! Oops... I see now that org-publish-find-date and org-publish-find-title call org-export-get-environm

Re: [O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-29 Thread Bastien
Hi Matt, Matt Lundin writes: > This patch fixes a bug in which org-publish makes the following > call... Applied, thanks! -- Bastien

[O] [PATCH] Remove unecesssary invocations of org-mode in ox-publish

2014-07-28 Thread Matt Lundin
This patch fixes a bug in which org-publish makes the following call... (let ((org-inhibit-startup t) (org-mode))) ...in buffers that are already open. This often happens, for instance, when publishing a sitemap. But there is no need to call org-mode again in these open buffers. Moreover calling