all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [BUG] Partially broken Org mode when remote setupfile is unavailable
@ 2024-03-19 10:46 Max Nikulin
  2024-03-20 12:15 ` Ihor Radchenko
  0 siblings, 1 reply; 8+ messages in thread
From: Max Nikulin @ 2024-03-19 10:46 UTC (permalink / raw)
  To: emacs-orgmode

Hi,

I have tried the following .org file

--- 8< ---
#+setupfile: /dav:localhost#8000:/setup-dav-123.org

* Heading

[[https://orgmode.org/][Link]]
--- >8 ---

Having no webdav server running. Expected error:
> Package ‘tramp-gvfs’ not supported
> File mode specification error: (user-error Package `tramp-gvfs' not supported)

Unexpected errors:
- When point is on any non-empty line, RET is broken
> save-excursion: Wrong type argument: stringp, nil
- The link is not fontified (it is parsed properly however and even may 
be opened).

In the case of unavailable http: setupfile, RET is not broken,
but the link is not fontified.

--- 8< ---
#+setupfile: http://localhost:8000/setup-http-987.org

* Heading

[[https://orgmode.org/][Link]]
--- >8 ---

I expect that inaccessible setupfiles are just ignored and Org should 
work as without "#+setupfile:" lines.

Emacs-28 Org current main HEAD.



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [BUG] Partially broken Org mode when remote setupfile is unavailable
  2024-03-19 10:46 [BUG] Partially broken Org mode when remote setupfile is unavailable Max Nikulin
@ 2024-03-20 12:15 ` Ihor Radchenko
  2024-03-27 14:33   ` Max Nikulin
                     ` (2 more replies)
  0 siblings, 3 replies; 8+ messages in thread
From: Ihor Radchenko @ 2024-03-20 12:15 UTC (permalink / raw)
  To: Max Nikulin; +Cc: emacs-orgmode


Confirmed.

Max Nikulin <manikulin@gmail.com> writes:

> I have tried the following .org file
>
> --- 8< ---
> #+setupfile: /dav:localhost#8000:/setup-dav-123.org
>
> * Heading
>
> [[https://orgmode.org/][Link]]
> --- >8 ---
>
> Having no webdav server running. Expected error:
>> Package ‘tramp-gvfs’ not supported
>> File mode specification error: (user-error Package `tramp-gvfs' not supported)
>
> Unexpected errors:
> - When point is on any non-empty line, RET is broken
>> save-excursion: Wrong type argument: stringp, nil
> - The link is not fontified (it is parsed properly however and even may 
> be opened).

This is because `file-remote-p' throws an error. I believe that it is
Emacs bug.
See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=68976

> In the case of unavailable http: setupfile, RET is not broken,
> but the link is not fontified.
>
> --- 8< ---
> #+setupfile: http://localhost:8000/setup-http-987.org
>
> * Heading
>
> [[https://orgmode.org/][Link]]
> --- >8 ---
>
> I expect that inaccessible setupfiles are just ignored and Org should 
> work as without "#+setupfile:" lines.

This is because of another bug in Emacs.
See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=58888

-- 
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] 8+ messages in thread

* Re: [BUG] Partially broken Org mode when remote setupfile is unavailable
  2024-03-20 12:15 ` Ihor Radchenko
@ 2024-03-27 14:33   ` Max Nikulin
  2024-03-30 10:02   ` Ihor Radchenko
  2024-04-09  8:07   ` Ihor Radchenko
  2 siblings, 0 replies; 8+ messages in thread
From: Max Nikulin @ 2024-03-27 14:33 UTC (permalink / raw)
  To: emacs-orgmode

On 20/03/2024 19:15, Ihor Radchenko wrote:
> Max Nikulin writes:
>> #+setupfile: /dav:localhost#8000:/setup-dav-123.org
[...]
>>> Package ‘tramp-gvfs’ not supported
>>> File mode specification error: (user-error Package `tramp-gvfs' not supported)
[...]
>> - When point is on any non-empty line, RET is broken
>>> save-excursion: Wrong type argument: stringp, nil
[...]
> This is because `file-remote-p' throws an error. [...]
> See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=68976

Thanks for the link. I would consider a workaround:

(defun org-file-remote-p (file &rest args)
   (condition-case err
       (apply #'file-remote-p file args)
     ('user-error
      (let ((file (apply #'tramp-file-name-for-operation
			#'file-remote-p file args)))
        (and (tramp-tramp-file-p file)
	  (tramp-handle-file-remote-p file))))))





^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [BUG] Partially broken Org mode when remote setupfile is unavailable
  2024-03-20 12:15 ` Ihor Radchenko
  2024-03-27 14:33   ` Max Nikulin
@ 2024-03-30 10:02   ` Ihor Radchenko
  2024-04-01 10:11     ` Max Nikulin
  2024-04-09  8:07   ` Ihor Radchenko
  2 siblings, 1 reply; 8+ messages in thread
From: Ihor Radchenko @ 2024-03-30 10:02 UTC (permalink / raw)
  To: Max Nikulin; +Cc: emacs-orgmode

Ihor Radchenko <yantar92@posteo.net> writes:

>> Unexpected errors:
>> - When point is on any non-empty line, RET is broken
>>> save-excursion: Wrong type argument: stringp, nil
>> - The link is not fontified (it is parsed properly however and even may 
>> be opened).
>
> This is because `file-remote-p' throws an error. I believe that it is
> Emacs bug.
> See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=68976

This part is now addressed by
https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=4255d5dcc

-- 
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] 8+ messages in thread

* Re: [BUG] Partially broken Org mode when remote setupfile is unavailable
  2024-03-30 10:02   ` Ihor Radchenko
@ 2024-04-01 10:11     ` Max Nikulin
  2024-04-01 11:10       ` Ihor Radchenko
  0 siblings, 1 reply; 8+ messages in thread
From: Max Nikulin @ 2024-04-01 10:11 UTC (permalink / raw)
  To: emacs-orgmode

On 30/03/2024 17:02, Ihor Radchenko wrote:
> Ihor Radchenko writes:
> 
>>> Unexpected errors:
>>> - When point is on any non-empty line, RET is broken
>>>> save-excursion: Wrong type argument: stringp, nil
>>> - The link is not fontified (it is parsed properly however and even may
>>> be opened).
>>
>> This is because `file-remote-p' throws an error. I believe that it is
>> Emacs bug.
>> See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=68976
> 
> This part is now addressed by
> https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=4255d5dcc

You are right that a workaround with `org-file-remote-p' would not help. 
On the other hand I can reproduce the issue with Emacs-28 and the error 
happens in `org--collect-keywords-1' due to the `expand-file-name' call. 
That is why I can not agree that it is addressed.




^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [BUG] Partially broken Org mode when remote setupfile is unavailable
  2024-04-01 10:11     ` Max Nikulin
@ 2024-04-01 11:10       ` Ihor Radchenko
  0 siblings, 0 replies; 8+ messages in thread
From: Ihor Radchenko @ 2024-04-01 11:10 UTC (permalink / raw)
  To: Max Nikulin; +Cc: emacs-orgmode

Max Nikulin <manikulin@gmail.com> writes:

>> This part is now addressed by
>> https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=4255d5dcc
>
> You are right that a workaround with `org-file-remote-p' would not help. 
> On the other hand I can reproduce the issue with Emacs-28 and the error 
> happens in `org--collect-keywords-1' due to the `expand-file-name' call. 
> That is why I can not agree that it is addressed.

https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=1d86651be

-- 
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] 8+ messages in thread

* Re: [BUG] Partially broken Org mode when remote setupfile is unavailable
  2024-03-20 12:15 ` Ihor Radchenko
  2024-03-27 14:33   ` Max Nikulin
  2024-03-30 10:02   ` Ihor Radchenko
@ 2024-04-09  8:07   ` Ihor Radchenko
  2024-04-09 10:30     ` Max Nikulin
  2 siblings, 1 reply; 8+ messages in thread
From: Ihor Radchenko @ 2024-04-09  8:07 UTC (permalink / raw)
  To: Max Nikulin; +Cc: emacs-orgmode

Ihor Radchenko <yantar92@posteo.net> writes:

>> --- 8< ---
>> #+setupfile: http://localhost:8000/setup-http-987.org
>>
>> * Heading
>>
>> [[https://orgmode.org/][Link]]
>> --- >8 ---
>>
>> I expect that inaccessible setupfiles are just ignored and Org should 
>> work as without "#+setupfile:" lines.
>
> This is because of another bug in Emacs.
> See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=58888

The Emacs bug has been fixed.
There is no workaround we can employ on Org side other than waiting for
new Emacs releases with the fix in place.
Closing.
Handled.

-- 
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] 8+ messages in thread

* Re: [BUG] Partially broken Org mode when remote setupfile is unavailable
  2024-04-09  8:07   ` Ihor Radchenko
@ 2024-04-09 10:30     ` Max Nikulin
  0 siblings, 0 replies; 8+ messages in thread
From: Max Nikulin @ 2024-04-09 10:30 UTC (permalink / raw)
  To: emacs-orgmode

On 09/04/2024 15:07, Ihor Radchenko wrote:
> Ihor Radchenko writes:
> 
>>> #+setupfile: http://localhost:8000/setup-http-987.org
>>>
>>> * Heading
>>>
>>> [[https://orgmode.org/][Link]]
[...]
>> See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=58888
> 
> The Emacs bug has been fixed.
> There is no workaround we can employ on Org side other than waiting for
> new Emacs releases with the fix in place.

I was considering a kludge with setting `org-resource-download-policy' 
to 'safe during loading of Org mode if its current value is 'prompt. I 
am unsure however if it is possible to detect whether the bug is fixed. 
I see several `defun' in the patch, so perhaps some of them might be 
leveraged for such test.



^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2024-04-09 10:31 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-19 10:46 [BUG] Partially broken Org mode when remote setupfile is unavailable Max Nikulin
2024-03-20 12:15 ` Ihor Radchenko
2024-03-27 14:33   ` Max Nikulin
2024-03-30 10:02   ` Ihor Radchenko
2024-04-01 10:11     ` Max Nikulin
2024-04-01 11:10       ` Ihor Radchenko
2024-04-09  8:07   ` Ihor Radchenko
2024-04-09 10:30     ` Max Nikulin

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.