[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3
From: |
Stefan Monnier |
Subject: |
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3 |
Date: |
Tue, 11 Jun 2024 09:16:17 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
>> Note that org-num.el where `org-num-skip-tags' is defined does have
>> (require 'cl-lib).
>>
>> IMHO, there is some problem either with loaddefs generation or with the
>> fact that loaddefs file is compiled at all.
Code that's in an ;;;###autoload cookie is copied verbatim to the
loaddefs file, so it cannot assume that all the earlier code in the file
(e.g. the `(require 'cl)` at the beginning of the file) has been executed.
> No, the problem is that this variable uses cl-lib stuff for safep
> functions. IMO, this is not a good idea, and the above situation is
> precisely the evidence for that.
In theory, `org-loaddefs.el` could be arranged to require `cl-lib` (we
don't want that in `loaddefs.el` but it'd be OK for `org-loaddefs.el`).
In practice, the generation of `org-loaddefs.el` currently doesn't offer
an easy way to insert some ad-hoc code like `(require 'cl-lib)` at
the beginning. We should fix that because it's a relatively common need
(especially since the relative ordering of elements from different
files is not guaranteed).
> Stefan, WDYT about this issue?
While you're at it, please don't autoload the vars:
autoload only the safety declaration.
Stefan
- Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, (continued)
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Eli Zaretskii, 2024/06/10
- Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Ihor Radchenko, 2024/06/11
- Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Eli Zaretskii, 2024/06/11
- Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Robert Pluim, 2024/06/11
- Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Ihor Radchenko, 2024/06/14
- Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Eli Zaretskii, 2024/06/14
- Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Robert Pluim, 2024/06/14
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3,
Stefan Monnier <=
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Ihor Radchenko, 2024/06/12
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Stefan Monnier, 2024/06/12
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Ihor Radchenko, 2024/06/13
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Eli Zaretskii, 2024/06/13
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Stefan Monnier, 2024/06/13
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Ihor Radchenko, 2024/06/14
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Ihor Radchenko, 2024/06/13
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Eli Zaretskii, 2024/06/13
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Ihor Radchenko, 2024/06/15
Re: master 5a125fb5a97 1/2: Update to Org 9.7.3, Eli Zaretskii, 2024/06/15