Re: Re: Org-cite: Replace basic follow-processor with transient menu?

Re: Re: Org-cite: Replace basic follow-processor with transient menu?

[Pedro A. Aranda]

 >Message: 7
 >Date: Mon, 11 Nov 2024 15:52:03 +0000
 >From: Ihor Radchenko <yantar92@posteo.net>
 >To: Tor-björn Claesson <tclaesson@gmail.com>
 >Cc: Jonas Bernoulli <jonas@bernoul.li>, emacs-orgmode@gnu.org
 >Subject: Re: Org-cite: Replace basic follow-processor with transient
 >        menu?
 >Message-ID: <87jzd9ojj0.fsf@localhost>
 >Content-Type: text/plain; charset=utf-8
 >
 >Tor-björn Claesson <tclaesson@gmail.com> writes:
 >
 >> Here is my first attempt.
 >> I have read the commit guidelines, but it is very possible that I have
 >> misunderstood or just missed something, so I'm grateful for any
 >> feedback!
 >
 >Thanks for the patch!
 >
 >See my comments below.
 >
 >> * lisp/oc-basic.el (require 'transient): Pull in transient.
 >
 >Technically, we still support Emacs 27, which does not have
 >transient. But accounting for this would be a pain and Emacs 30 is
 >probably going to be released some time around the beginning of the next
 >year. So, let's not worry about this and accept that Org 9.8-pre (next
 >release) no longer supports Emacs 27.
 >
 >If you are a user of Emacs 27 + Org main branch, and have strong
 >objections, please chime in.

Maybe we don't need to be so drastic ;-) transient is in elpa and melpa. 
Including a note in the docs for emacs-27 to install it from either 
(they are also installing org from a package repo, I guess) may 
circumvent this "issue".

Just My .2 cents

Re: Re: Org-cite: Replace basic follow-processor with transient menu?

[Ihor Radchenko]

"Pedro A. Aranda" <paaguti@gmail.com> writes:

>  >If you are a user of Emacs 27 + Org main branch, and have strong
>  >objections, please chime in.
>
> Maybe we don't need to be so drastic ;-) transient is in elpa and melpa. 
> Including a note in the docs for emacs-27 to install it from either 
> (they are also installing org from a package repo, I guess) may 
> circumvent this "issue".

I guess we can add transient.el to dependencies.
This will handle ELPA installations.

However, installing from git is not easy. Our build system is not
designed to work with third-party packages.

-- 
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>

Re: Re: Org-cite: Replace basic follow-processor with transient menu?

[Pedro Andres Aranda Gutierrez]

[-- Attachment #1: Type: text/plain, Size: 1436 bytes --]

Hi

On Tue, 12 Nov 2024 at 18:20, Ihor Radchenko <yantar92@posteo.net> wrote:

> "Pedro A. Aranda" <paaguti@gmail.com> writes:
>
> >  >If you are a user of Emacs 27 + Org main branch, and have strong
> >  >objections, please chime in.
> >
> > Maybe we don't need to be so drastic ;-) transient is in elpa and melpa.
> > Including a note in the docs for emacs-27 to install it from either
> > (they are also installing org from a package repo, I guess) may
> > circumvent this "issue".
>
> I guess we can add transient.el to dependencies.
> This will handle ELPA installations.
>

That seems to be the cleanest solution, right? Not sure, but I guess we
could even mandate a certain version that is compatible with emacs-27 if
need be and for emacs-27 setups...
emacs-27 is old, OK... but still used in older systems :-)


> However, installing from git is not easy. Our build system is not
> designed to work with third-party packages.
>
> --
> 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>
>

Best, /PA
-- 
Fragen sind nicht da, um beantwortet zu werden,
Fragen sind da um gestellt zu werden
Georg Kreisler

Headaches with a Juju log:
unit-basic-16: 09:17:36 WARNING juju.worker.uniter.operation we should run
a leader-deposed hook here, but we can't yet

[-- Attachment #2: Type: text/html, Size: 2589 bytes --]