Re: org-persist cache for remote files

2021-12-21 Thread Ihor Radchenko
Lele Gaifax writes: > Hi Ihor, > > I had some trouble testing your patches, and the outcome is still unclear to > me: apparently (again, cannot be sure this is not caused by "improper" > application of the patches) the new version of org-persist, with default value > for o-p-remote-files (100),

Re: org-persist cache for remote files

2021-12-21 Thread Lele Gaifax
Hi Ihor, I had some trouble testing your patches, and the outcome is still unclear to me: apparently (again, cannot be sure this is not caused by "improper" application of the patches) the new version of org-persist, with default value for o-p-remote-files (100), removes any remote file from the p

Re: org-persist cache for remote files

2021-12-19 Thread Ihor Radchenko
Lele Gaifax writes: > I couldn't test yet, but I saw a couple of other glitches, sorry for not > noticing first. Thanks for the proof reading. I updated the patch accordingly. Best, Ihor >From 557e7cec6dd22b09468b82650feed2020f97d781 Mon Sep 17 00:00:00 2001 Message-Id: <557e7cec6dd22b09468b82

Re: org-persist cache for remote files

2021-12-18 Thread Lele Gaifax
Lele Gaifax writes: >> + :group 'org-persist >> + :type '(choice (const :tag "Never" nil) >> + (const :tag "Always" t) >> + (number :tag "Keep note more than X files") > > Now that you explained the meaning, maybe better to be explicit here, > mentioning "X remot

Re: org-persist cache for remote files

2021-12-18 Thread Lele Gaifax
Ihor Radchenko writes: > Lele Gaifax writes: > >> In the meanwhile, here below some notes: > > Thanks! See the updated patch. Great, you're so fast! :-) I couldn't test yet, but I saw a couple of other glitches, sorry for not noticing first. > ... > +(defcustom org-persist-remote-files 100

Re: org-persist cache for remote files

2021-12-18 Thread Ihor Radchenko
Lele Gaifax writes: > In the meanwhile, here below some notes: Thanks! See the updated patch. Best, Ihor >From 92878fab54c26800562409f3c686f065b7523a61 Mon Sep 17 00:00:00 2001 Message-Id: <92878fab54c26800562409f3c686f065b7523a61.1639838789.git.yanta...@gmail.com> From: Ihor Radchenko Date:

Re: org-persist cache for remote files

2021-12-18 Thread Lele Gaifax
Ihor Radchenko writes: > Lele Gaifax writes: > >>> Maybe we should better make this a user option? >> >> Yes, that's what I had in mind: when (say) org-persist-do-not-auto-gc-remotes >> is enabled, the cleanup procedure would ignore remote Org documents and a new >> explicit (interactive) org-pe

Re: org-persist cache for remote files

2021-12-18 Thread Ihor Radchenko
Lele Gaifax writes: >> Maybe we should better make this a user option? > > Yes, that's what I had in mind: when (say) org-persist-do-not-auto-gc-remotes > is enabled, the cleanup procedure would ignore remote Org documents and a new > explicit (interactive) org-persist-gc-remotes would take care

Re: org-persist cache for remote files

2021-12-17 Thread Lele Gaifax
Ihor Radchenko writes: > Lele Gaifax writes: > >> I wonder if there is some mechanism I could use to either prevent caching >> of non-local documents or to avoid the check on existence in the >> org-persist-gc. > > I can easily make a change that always garbage-collect non-local > documents with

org-persist cache for remote files (was: org-persist)

2021-12-17 Thread Ihor Radchenko
Lele Gaifax writes: > Digging down, I found that it was due to the persistent cache, that contained > several entries about remote (Tramp) documents: upon closing Emacs, the > org-persist-gc function is triggered, and the index is traversed to remove > dangling entries, performing a file-exist-p