From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: Creating a git branch for cond* Date: Thu, 25 Jan 2024 18:56:05 +0200 Message-ID: <86zfwtbbdm.fsf@gnu.org> References: Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="30697"; mail-complaints-to="usenet@ciao.gmane.io" Cc: ams@gnu.org, rms@gnu.org, emacs-devel@gnu.org To: Alan Mackenzie Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Thu Jan 25 17:57:22 2024 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1rT32Y-0007jX-2c for ged-emacs-devel@m.gmane-mx.org; Thu, 25 Jan 2024 17:57:22 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rT31f-0006tv-2H; Thu, 25 Jan 2024 11:56:27 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rT31d-0006tF-5v for emacs-devel@gnu.org; Thu, 25 Jan 2024 11:56:25 -0500 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rT31c-00010j-8U; Thu, 25 Jan 2024 11:56:24 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date: mime-version; bh=OfLuSb6O464snoqp8KDShClizzpoLPKl0a6aYMXA7y8=; b=qbi/B2hLIv5x xbPlfiHnkbbpAsEWW5zGMkJhZNIuo5wdsf/x1QIsENtdHoXtDQxIJZ8mjqMHm4XTp3v56EpAQd6Hq 8YkYsveNcGHacPT7OtlUJvG59iDDH2LpVyEPPRnOOxhoiKGfSMq6PrQqdfwoLLmN5P9cJEe377n+l gV/HnvBfH3YmFUEnGxS07CDguTr/T8rNK1L6GBQFlOah1wo4KdabPnpaQkOBVZ2Z/zG9xWNG7Il8V SXxPBhJOgoenQ7tRxv50w6zU65ldtdlwxt9SW6qvfeW4Vrz0fPUKAewapbAQI63nI9Z/TRiCXYNsA M5VT1iaPegJKag34rZLZUw==; In-Reply-To: (message from Alan Mackenzie on Thu, 25 Jan 2024 16:37:44 +0000) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.devel:315384 Archived-At: > Date: Thu, 25 Jan 2024 16:37:44 +0000 > Cc: rms@gnu.org, eliz@gnu.org, emacs-devel@gnu.org > From: Alan Mackenzie > > > I hope it goes to master directly, it is a single file that doesn't > > affect anything else. And _if_ existing code is updated to use cond* > > that will get much more milage. > > It does affect other things, in particular, other .el files which are > adapted to use cond*. Please don't make any changes except installing cond* and fixing it. There's no decision yet to start converting any code to use it, nor do I envision such a decision any time soon. > If that happens on master, then both cond* and > the conversion need to be perfect before the file can be committed. Once again: we are not converting anything, at least not yet. We are only discussing how best to handle installation of cond*. > What I want to do early on in cond* development is convert > lisp/emacs-lisp/macroexp.el from pcase to cond*. I don't agree with such a change, so please don't do anything like that. > Currently, pcase.el is dependent upon macroexp.el and macroexp.el is > dependent upon pcase.el. This leads to an unlovely artifice in the > bootstrapping file, loadup.el, and this has caused me grief in > another branch I'm working on. We are not going to make any such changes just because we can, or just because you have problems on some branch you'd like to avoid fixing. cond* is a new construct, and once it is in Emacs, and documented as it should be, we should collect experience with using it before we even think about converting existing code. Please don't forget that Emacs is a stable program: we don't make changes just because we can. Changes to code that was with us for years are only justified if we have a very good reason. > No doubt Eli will have something to say about this idea. ;-) He does, see above. You don't have my okay for making any changes except installing cond* and fixing it and its documentation. Btw, we should also have tests for it. > There are advantages to the branch, as well as advantages to having it > on master. Yes. But, as you say, let's hear what Richard thinks about this.