* Error during PDF export but the PDF file is exported
@ 2021-10-29 12:43 Pascal Quesseveur
2022-10-16 10:35 ` Ihor Radchenko
0 siblings, 1 reply; 7+ messages in thread
From: Pascal Quesseveur @ 2021-10-29 12:43 UTC (permalink / raw)
To: emacs-orgmode
Hello,
I use Emacs 27.1 on Windows and Org 9.3.
From time to time I work on files mounted from a remote host which is
late compared to my pc. Then I encounter errors when producing files
from Org:
File xxx wasn't produced
But the file is produced and is up-to-date. The error message comes
from org-compile-file. In that function the test is to compare the
date at the start of function with the date of the produced file. I
don't know if it's still the case in recent versions, but I think it
could be changed. For example, instead of using the date at the start
of the function, we could use the date of the file if it already
exists
(time (if (file-exists-p output)
(file-attribute-modification-time (file-attributes output))
'(0 0 0 0)))
--
Pascal Quesseveur
pquessev@gmail.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Error during PDF export but the PDF file is exported
2021-10-29 12:43 Error during PDF export but the PDF file is exported Pascal Quesseveur
@ 2022-10-16 10:35 ` Ihor Radchenko
[not found] ` <821qr6n7ue.fsf@gmail.com>
0 siblings, 1 reply; 7+ messages in thread
From: Ihor Radchenko @ 2022-10-16 10:35 UTC (permalink / raw)
To: pquessev; +Cc: emacs-orgmode
Pascal Quesseveur <pquessev@gmail.com> writes:
> From time to time I work on files mounted from a remote host which is
> late compared to my pc. Then I encounter errors when producing files
> from Org:
>
> File xxx wasn't produced
>
> But the file is produced and is up-to-date. The error message comes
> from org-compile-file. In that function the test is to compare the
> date at the start of function with the date of the produced file. I
> don't know if it's still the case in recent versions, but I think it
> could be changed. For example, instead of using the date at the start
> of the function, we could use the date of the file if it already
> exists
>
> (time (if (file-exists-p output)
> (file-attribute-modification-time (file-attributes output))
> '(0 0 0 0)))
Confirmed.
Sorry for the late reply.
This kind of use-case is indeed not considered by our code.
However, your suggestion will only provide a partial fix - when an older
file already exists in the remote folder.
May be there a way to get time on remote server?
--
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Error during PDF export but the PDF file is exported
[not found] ` <82zgdsno6w.fsf@gmail.com>
@ 2022-10-19 10:21 ` Ihor Radchenko
2022-10-19 17:24 ` Pascal Quesseveur
0 siblings, 1 reply; 7+ messages in thread
From: Ihor Radchenko @ 2022-10-19 10:21 UTC (permalink / raw)
To: pquessev; +Cc: emacs-orgmode
[Adding Org mailing list back to the loop. Someone forgot reply all in
the earlier messages]
Pascal Quesseveur <pquessev@gmail.com> writes:
>>"IR" == Ihor Radchenko <yantar92@posteo.net> a écrit :
>
> IR> No, if there is an error when running LaTeX. No PDF file might be
> IR> produced in some scenarios. And we need to clearly communicate such
> IR> scenarios to the user.
>
> Here is a patch to org-macs.el (from Emacs 28.1) that works for me. I
> have checked with files on a server whose current date was behind my
> PC. I also checked with a sync server and with local files.
Thanks!
I think that creating a temporary file locally is excessive. You could
guard the logic behind `file-remote-p'.
Also, can you prepare a proper patch? See
https://orgmode.org/worg/org-contribute.html#first-patch
--
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Error during PDF export but the PDF file is exported
2022-10-19 10:21 ` Ihor Radchenko
@ 2022-10-19 17:24 ` Pascal Quesseveur
2022-10-20 5:12 ` Ihor Radchenko
0 siblings, 1 reply; 7+ messages in thread
From: Pascal Quesseveur @ 2022-10-19 17:24 UTC (permalink / raw)
To: Ihor Radchenko; +Cc: emacs-orgmode
>"IR" == Ihor Radchenko <yantar92@posteo.net> writes:
IR> I think that creating a temporary file locally is excessive.
Perhaps, but I couldn't find another way.
IR> You could guard the logic behind `file-remote-p'.
FMU file-remote-p works onnly when there is a handler on path for
operation file-remote-p, which is not true in the general
case. Otherwise that function returns nil.
OTOH it is possible to reserve the creation of a temporary file only
when the file is not already present.
--
Pascal Quesseveur
pquessev@gmail.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Error during PDF export but the PDF file is exported
2022-10-19 17:24 ` Pascal Quesseveur
@ 2022-10-20 5:12 ` Ihor Radchenko
[not found] ` <98da3e0e-21d1-d342-ae20-5bd9887b668f@gmail.com>
0 siblings, 1 reply; 7+ messages in thread
From: Ihor Radchenko @ 2022-10-20 5:12 UTC (permalink / raw)
To: pquessev; +Cc: emacs-orgmode
Pascal Quesseveur <pquessev@gmail.com> writes:
>>"IR" == Ihor Radchenko <yantar92@posteo.net> writes:
>
> IR> I think that creating a temporary file locally is excessive.
>
> Perhaps, but I couldn't find another way.
I think we are miscommunication. What I wanted to say is that
`current-time' is good enough if we export to local file. The whole
route with creating temporary file and getting its attributes is only
needed in remote directory.
> IR> You could guard the logic behind `file-remote-p'.
>
> FMU file-remote-p works onnly when there is a handler on path for
> operation file-remote-p, which is not true in the general
> case. Otherwise that function returns nil.
If file-remote-p does not work, I doubt that we can export to that
directory at all.
> OTOH it is possible to reserve the creation of a temporary file only
> when the file is not already present.
It is also an option.
--
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Error during PDF export but the PDF file is exported
[not found] ` <98da3e0e-21d1-d342-ae20-5bd9887b668f@gmail.com>
@ 2022-10-21 3:29 ` Ihor Radchenko
2022-10-21 8:05 ` Pascal Quesseveur
0 siblings, 1 reply; 7+ messages in thread
From: Ihor Radchenko @ 2022-10-21 3:29 UTC (permalink / raw)
To: Max Nikulin; +Cc: pquessev, emacs-orgmode
Max Nikulin <manikulin@gmail.com> writes:
> On 20/10/2022 12:12, Ihor Radchenko wrote:
>>
>> What I wanted to say is that
>> `current-time' is good enough if we export to local file.
>
> Since the patch was not posted to the mailing list, I may be wrong due
> to some rather wild assumption.
>
> Paul Eggert convinced me that generally it is a bad idea to compare wall
> (system) time and file time. They may have different resolution or may
> have significant offset. Notice that the following pending patch
> modifies time handling in `org-compile-file', however I have not tested
> it for remote files:
>
> Max Nikulin. [PATCH v2] org-macs.el: Do not compare wall time and file
> modification time. Sun, 9 Oct 2022 15:18:04 +0700.
> https://list.orgmode.org/thu03t$16vt$1@ciao.gmane.io
>
> I am sorry that I did not join to this thread earlier.
You are right, that patch should fix this problem as well.
For some reason, I only remembered that patch as something that concerns
tangling. Now, applied.
Pascal, could you please try the latest main and see if the problem is fixed?
--
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Error during PDF export but the PDF file is exported
2022-10-21 3:29 ` Ihor Radchenko
@ 2022-10-21 8:05 ` Pascal Quesseveur
0 siblings, 0 replies; 7+ messages in thread
From: Pascal Quesseveur @ 2022-10-21 8:05 UTC (permalink / raw)
To: Ihor Radchenko; +Cc: Max Nikulin, emacs-orgmode
>"IR" == Ihor Radchenko <yantar92@posteo.net> writes:
IR> Pascal, could you please try the latest main and see if the
IR> problem is fixed?
Unfortunately I can't try at the moment. I have followed the link
https://list.orgmode.org/thu03t$16vt$1@ciao.gmane.io
and FWIU it should work fine as it takes the file modification time
instead of current-time in org-compile-file as I did in my first
attempt.
--
Pascal Quesseveur
pquessev@gmail.com
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-10-21 8:51 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-29 12:43 Error during PDF export but the PDF file is exported Pascal Quesseveur
2022-10-16 10:35 ` Ihor Radchenko
[not found] ` <821qr6n7ue.fsf@gmail.com>
[not found] ` <871qr6r7op.fsf@localhost>
[not found] ` <82zgdsno6w.fsf@gmail.com>
2022-10-19 10:21 ` Ihor Radchenko
2022-10-19 17:24 ` Pascal Quesseveur
2022-10-20 5:12 ` Ihor Radchenko
[not found] ` <98da3e0e-21d1-d342-ae20-5bd9887b668f@gmail.com>
2022-10-21 3:29 ` Ihor Radchenko
2022-10-21 8:05 ` Pascal Quesseveur
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).