From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Christine Lemmer-Webber Newsgroups: gmane.lisp.guile.devel Subject: Re: Guile 3 and wip-elisp/Emacs Date: Thu, 21 Oct 2021 22:59:38 -0400 Message-ID: <87sfwt69fk.fsf@dustycloud.org> References: <87mtogpwvi.fsf@elephly.net> <47-61683200-11-6332ae00@255428996> <875ytx5wvg.fsf@dustycloud.org> <875yts7jcp.fsf@dustycloud.org> <87tuhc642o.fsf@gnu.org> <874k9c5ulu.fsf@dustycloud.org> <87y26mkw9g.fsf@terpri.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="16478"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: mu4e 1.6.6; emacs 27.2 Cc: guile-devel@gnu.org To: Robin Templeton Original-X-From: guile-devel-bounces+guile-devel=m.gmane-mx.org@gnu.org Fri Oct 22 05:11:28 2021 Return-path: Envelope-to: guile-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 1mdkxr-00047O-Lj for guile-devel@m.gmane-mx.org; Fri, 22 Oct 2021 05:11:27 +0200 Original-Received: from localhost ([::1]:56520 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mdkxq-0003KV-Gp for guile-devel@m.gmane-mx.org; Thu, 21 Oct 2021 23:11:26 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:51740) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mdkxP-0003JN-VM for guile-devel@gnu.org; Thu, 21 Oct 2021 23:10:59 -0400 Original-Received: from dustycloud.org ([50.116.34.160]:38584) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mdkxN-0001ZM-Kn for guile-devel@gnu.org; Thu, 21 Oct 2021 23:10:59 -0400 Original-Received: from twig (localhost [127.0.0.1]) by dustycloud.org (Postfix) with ESMTPS id 9659026600; Thu, 21 Oct 2021 23:10:55 -0400 (EDT) In-reply-to: <87y26mkw9g.fsf@terpri.org> Received-SPF: pass client-ip=50.116.34.160; envelope-from=cwebber@dustycloud.org; helo=dustycloud.org X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: guile-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Developers list for Guile, the GNU extensibility library" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guile-devel-bounces+guile-devel=m.gmane-mx.org@gnu.org Original-Sender: "guile-devel" Xref: news.gmane.io gmane.lisp.guile.devel:20929 Archived-At: Robin Templeton writes: > Christine Lemmer-Webber writes: > >> Ludovic Court=C3=A8s writes: >> >>> Hello! >>> >>> Christine Lemmer-Webber skribis: >>> >>>> I've pushed this as origin/wip-elisp-rebased. I actually rebased it >>>> again, making some naming adjustments for myself and a couple of >>>> adjustments having talked to Robin. >>>> >>>> If nobody objects, I'd like to merge this into main. Maintainers, if >>>> you have any objections, speak now or forever hold these commits! >>> >>> I haven=E2=80=99t looked at the branch, but I think it=E2=80=99s great = to see it live >>> and it=E2=80=99s great if it can be merged! >> >> I just compiled the rebased version and will be playing with it little >> bits over the next few days to make sure it's reasonably good. >> >>> Some things to pay attention to before merging to =E2=80=98main=E2=80= =99, since it >>> corresponds to the current 3.0 stable branch: >>> >>> =E2=80=A2 Make sure no backward incompatibilities are introduced in >>> preexisting modules; >>> >>> =E2=80=A2 Make sure the ABI of libguile-3.0.so and that of public mod= ules >>> is unchanged, or is changed compatibly; >> >> There are, I think, two commits that could use review, but I am NOT the >> right person to do this. >> >> 4e96211eb666751b8666beb918bf3108aa1c725b intern arbitrary constants >> 433fc448ddb018767906f8808203c9668c68cd83 multiple obarrays > > I'll take a look at these... > >> [...] (and maybe the "guile-private-ref" and "allow >> arbitrary constants in cps" commits look relevant too). > > and these, but concur that Andy is the best person to review them, and I > agree that Andy should approve the merge overall in any case. > > Still reviewing the wip-elisp-rebased branch in my spare time; so far I > haven't found any noteworthy problems, not that I was expecting to :) > > A minor point: IMHO the "(Best-ability ChangeLog annotation ...)" lines > aren't the ideal way to credit you, in terms of commit message > formatting. I'd prefer using git 'trailers' for in-commit credit, so > that it's obvious to both humans and git that it's commit metadata. I'm > not sure there's a conventional git trailer for this...something like > 'ChangeLog-by:' would be probably be clear enough. But if you prefer the > current formatting and it works with Guile's ChangeLog generator, I > don't mind leaving it as-is (except perhaps "s/Best-ability //" after > I've reviewed everything :)) WDYT? I'm fine changing it. The original reason for the "best ability" notes was that I was more or less making up text to make it compatible with the changelog style based off of reading your code. Honestly the attribution part isn't the most important to me but the "ChangeLog-by is an interesting idea". Most importantly, I want to get it in! :) > Thanks, > Robin