unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Harald Jörg" <haj@posteo.de>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: emacs-devel@gnu.org
Subject: Re: cperl-mode: Eliminating references to obsolete packages
Date: Thu, 24 Sep 2020 17:34:48 +0200	[thread overview]
Message-ID: <b636d94c-6205-0853-d99b-f22849bdcdde@posteo.de> (raw)
In-Reply-To: <jwvsgb8s2hv.fsf-monnier+emacs@gnu.org>

Stefan Monnier writes:

> [...]
>> I am unsure about the procedure: Should the backwards-compatible
>> version live in a separate branch on Emacs?
>
> I think we don't need a separate branch.  E.g. for format-prompt we can
> use a patch like the one below.  If you can't think of a simple way to
> preserve backward compatibility in a specific situation, feel free to
> ask for help,

Thanks for the patch!  It does the trick, so I apologize that I've
still doubts about the _process_.  I guess we should start making
cperl-mode.el a "dual life" project soonish - I'll better start a new
thread for that.

In detail:

The point of *this* post was to actually _eliminate_ some fboundp
conditionals in cperl-mode.el, so it feels a bit weird to introduce a
new one in the same step.

I am not particularly concerned about the use of format-prompt,
because the calling functions have other issues (cperl-info-on-command
is broken anyway, and cperl-perldoc is based on man, which means it
fails on Windows and has suboptimal formatting on all platforms).
That's outside the scope of this thread, though.

But replacement of deprecated / old stuff does occasionally happens
across the Emacs repository.  This may hit cperl-mode.el at any time,
like it did with time-convert and format-prompt.  So, every now and
then, the Emacs repository will contain a cperl-mode.el which won't
work with Emacs 26, until someone notices it and someone prepares
another backporting-patch.  Or until cperl-mode.el gets a place or
procedure where these changes don't hit.
--
Cheers,
haj




  reply	other threads:[~2020-09-24 15:34 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-23 18:37 cperl-mode: Eliminating references to obsolete packages Harald Jörg
2020-09-23 19:16 ` Stefan Monnier
2020-09-23 19:38   ` Harald Jörg
2020-09-23 19:50     ` Stefan Monnier
2020-09-24 15:34       ` Harald Jörg [this message]
2020-09-24 17:09         ` Stefan Monnier
2020-09-24 18:43           ` Harald Jörg
2020-09-24 22:20             ` Stefan Monnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b636d94c-6205-0853-d99b-f22849bdcdde@posteo.de \
    --to=haj@posteo.de \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.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).