On 30/04/2022 01:10, Paul Eggert wrote:
On 4/29/22 07:22, Max Nikulin wrote:
From my point of view, it is better to rewrite `org-compile-time' to
treat the case when there were no file prior to the call as that the
file has been updated without comparison of timestamps
Yes, that sounds
On 4/29/22 07:22, Max Nikulin wrote:
It was still working in most real-life cases.
Yes, the current code breaks only in fine-grained cases. Most of the
time it'll work fine since people rarely compile the same file twice in
the same second.
From my point of view, it is better to rewrite
On 29/04/2022 05:27, Paul Eggert wrote:
On 4/27/22 09:55, Stefan Monnier wrote:
Instead of rounding the times to whole seconds, wouldn't it make more
sense to check that the difference is larger than 1s?
org-file-newer-than-p is intended to work on filesystems like HFS+ that
store just the
On 4/27/22 09:55, Stefan Monnier wrote:
Instead of rounding the times to whole seconds, wouldn't it make more
sense to check that the difference is larger than 1s?
org-file-newer-than-p is intended to work on filesystems like HFS+ that
store just the seconds part of the last-modified time.
> - (not (time-less-p (cl-subseq (nth 5 (file-attributes file)) 0 2)
> - (cl-subseq time 0 2)
> + (not (time-less-p (org-time-convert-to-integer
> + (nth 5 (file-attributes file)))
> + (org-time-convert-to-integer
Thanks for reporting that. Fixed in Emacs master via the attached.
For the more general issue I'm planning to add a builtin boolean
variable current-time-list soon, that is t for (HIGH LOW MICROSEC
PICOSEC) format, nil for (TICKS . HZ) format.From 3abb3681b57d7c8ca7fa808addb0a10b6b109cab Mon
The change also breaks org-file-newer-than-p function that triggered the
debugger while loading my init that uses org babel. I was able to use the
example of the patch that Paul Eggert provided earlier for the desktop-save to
add the time-convert to “fix” org-file-newer-than-p as shown below.
