emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* Bug: org-ellipsis does not work as a local variable [9.3.1 (release_9.3.1-95-gf93020 @ /home/mark/git-repos/org-mode/lisp/)]
@ 2020-01-16 21:57 Mark E. Shoulson
  2020-05-06  0:17 ` Nicolas Goaziou
  0 siblings, 1 reply; 2+ messages in thread
From: Mark E. Shoulson @ 2020-01-16 21:57 UTC (permalink / raw)
  To: org-mode mailing list

[-- Attachment #1: Type: text/html, Size: 11735 bytes --]

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

* Re: Bug: org-ellipsis does not work as a local variable [9.3.1 (release_9.3.1-95-gf93020 @ /home/mark/git-repos/org-mode/lisp/)]
  2020-01-16 21:57 Bug: org-ellipsis does not work as a local variable [9.3.1 (release_9.3.1-95-gf93020 @ /home/mark/git-repos/org-mode/lisp/)] Mark E. Shoulson
@ 2020-05-06  0:17 ` Nicolas Goaziou
  0 siblings, 0 replies; 2+ messages in thread
From: Nicolas Goaziou @ 2020-05-06  0:17 UTC (permalink / raw)
  To: Mark E. Shoulson; +Cc: org-mode mailing list

Hello,

"Mark E. Shoulson" <mark@shoulson.com> writes:

> The "org-ellipsis" variable is specifically marked as (potentially) a
> safe local variable, so obviously someone intended for it possibly to be
> used that, and believed that it might be useful to someone as a local
> variable.
>
> However, there is no setup that I can find which makes this work.
> Placing
>
> :  #  Local Variables:
> :  #  org-ellipsis: "XXX"
> :  #  End:
>
> at the end of the file, or setting it in the top line, makes no change
> in the ellipsis, even if you do M-x org-mode again or reload the file
> (with find-alternate-file).  Indeed, M-x org-mode clears the local value
> assignment altogether.  Setting it by hand with setq-local doesn't work
> (it is cleared when you do M-x org-mode again anyway, as mentioned.)
>
> So, is org-ellipsis really not meant ever to be a local variable?  If
> so, that likely should be documented, and certainly the :safe annotation
> on it should be removed, as it strongly implies that using it as a local
> variable is acceptable and useful.

I removed :safe keyword and specified it isn't meant to be a set as
a local variable in its docstring.

Patches are welcome to enable it, tho.

Thank you for the report.

Regards,

-- 
Nicolas Goaziou


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

end of thread, other threads:[~2020-05-06  0:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-16 21:57 Bug: org-ellipsis does not work as a local variable [9.3.1 (release_9.3.1-95-gf93020 @ /home/mark/git-repos/org-mode/lisp/)] Mark E. Shoulson
2020-05-06  0:17 ` Nicolas Goaziou

Code repositories for project(s) associated with this 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).