[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [POLL] Should we accept breaking changes to get rid of Org libraries
From: |
Bastien Guerry |
Subject: |
Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading? |
Date: |
Tue, 08 Aug 2023 06:19:43 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Ihor Radchenko <yantar92@posteo.net> writes:
> Max Nikulin <manikulin@gmail.com> writes:
>
>>> Sure. This is not by itself a big deal. A number of Elisp libraries,
>>> including built-in Emacs libraries are loaded with side effects.
>>
>> It is still violation of conventions:
>>
>> (info "(elisp) Coding Conventions")
>> https://www.gnu.org/software/emacs/manual/html_node/elisp/Coding-Conventions.html
>>> D.1 Emacs Lisp Coding Conventions
>>>
>>> Simply loading a package should not change Emacs’s editing behavior.
>>> Include a command or commands to enable and disable the feature, or to
>>> invoke it.
>>>
>>> This convention is mandatory for any file that includes custom
>>> definitions. If fixing such a file to follow this convention requires an
>>> incompatible change, go ahead and make the incompatible change; don’t
>>> postpone it.
>
> This is convincing.
> I am then CCing Bastien, as, despite the Elisp convention, following it
> will break https://bzg.fr/en/the-software-maintainers-pledge/
FWIW, in this case, the mistake lies in breaking the Emacs Lisp coding
convention first. When the breaking change is a side-effect of fixing
a bug, it is unavoidable.
--
Bastien
- Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?,
Bastien Guerry <=
- Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?, Ihor Radchenko, 2023/08/08
- Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?, Ihor Radchenko, 2023/08/11
- Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?, Bastien Guerry, 2023/08/12
- Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?, Samuel Wales, 2023/08/12
- Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?, Ihor Radchenko, 2023/08/13
- Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?, Samuel Wales, 2023/08/13