GNU bug report logs - #69865
29.2; Can't instrument `org-babel-exp-results' with edebug

Previous Next

Package: emacs;

Reported by: Damien Cassou <damien <at> cassou.me>

Date: Sun, 17 Mar 2024 18:05:02 UTC

Severity: normal

Tags: confirmed, patch

Found in version 29.2

Done: Michael Heerdegen <michael_heerdegen <at> web.de>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Ihor Radchenko <yantar92 <at> posteo.net>, Kyle Meyer <kyle <at> kyleam.com>
Cc: Michael Heerdegen <michael_heerdegen <at> web.de>, damien <at> cassou.me, Eli Zaretskii <eliz <at> gnu.org>, 69865 <at> debbugs.gnu.org
Subject: bug#69865: 29.2; Can't instrument `org-babel-exp-results' with edebug
Date: Mon, 10 Mar 2025 13:01:51 -0700
Ihor Radchenko <yantar92 <at> posteo.net> writes:

> Kyle Meyer <kyle <at> kyleam.com> writes:
>
>>> I wouldn't install anything more substantial as it's better to take
>>> that with the Org developers first, but small bug fixes should be
>>> fine, I think.
>>>
>>> I'm copying in Ihor, in case he has anything to add.
>>
>> Ihor's opinion is the more important one here, but, as the person who
>> does the bulk of the porting from Emacs's to Org's tree, I agree with
>> your position.  Of course it's a judgment call regarding what's a minor
>> or simple enough change, but, worst case scenario, the Org side will see
>> something that we think warrants more discussion and initiate it.
>
> Usually, we are more than happy to backport changes that span across
> multiple Emacs libraries and happen to touch upon Org mode.
> There is no reason to split such changes into multiple patches.
> AFAIU, this discussion is an example of a change that fixes multiple
> places; not just Org mode.
>
> Same for trivial changes, even if they only concern Org files.
>
> Non-trivial changes, as it is usually the case, may better be discussed
> with maintainers of the files that are affected. In the case of Org
> mode, CCing Org mailing list (preferred for the purposes of public
> archiving), Org package maintainer, or relevant Org library maintainer,
> is appropriate.

Would it be worth documenting this in admin/MAINTAINERS?




This bug report was last modified 72 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.