From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id LuqAL/5CR2DUfgAAWB0awg (envelope-from ) for ; Tue, 09 Mar 2021 04:42:22 -0500 Received: by simark.ca (Postfix, from userid 112) id B570E1EF78; Tue, 9 Mar 2021 04:42:22 -0500 (EST) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on simark.ca X-Spam-Level: X-Spam-Status: No, score=0.2 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,MAILING_LIST_MULTI,RDNS_NONE,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.2 Received: from sourceware.org (unknown [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 42A551E54D for ; Tue, 9 Mar 2021 04:42:22 -0500 (EST) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 97D673870867; Tue, 9 Mar 2021 09:42:21 +0000 (GMT) Received: from mail-wm1-x329.google.com (mail-wm1-x329.google.com [IPv6:2a00:1450:4864:20::329]) by sourceware.org (Postfix) with ESMTPS id CA22A3870867 for ; Tue, 9 Mar 2021 09:42:17 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org CA22A3870867 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=embecosm.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=andrew.burgess@embecosm.com Received: by mail-wm1-x329.google.com with SMTP id b2-20020a7bc2420000b029010be1081172so5540318wmj.1 for ; Tue, 09 Mar 2021 01:42:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=embecosm.com; s=google; h=date:from:to:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=Yui7DkA7gAkkQNUNs6iJF6mhVXZnWh/iv89nB7IAwIg=; b=DeQxNVwTJhgesFcAzeB1F0lo24D0LgEvit4LI3sFPSubJ/8WELkYCCFVUS1CUruf1p vTcau3Jl7FOV/jklSmeow2+saKT3UgFBTsbKRLYXa7thfqZ/jqWuPzgWWOTD9fIg9/t3 u8nL3s64Ggc7VcA0JWgPrah64Au2Rnc/QTnqMt08LojIZK8GVXLBt/KwPqINxMMWYQ3/ n7P6IRqna3S6dNlfDvacNRXtfVND/ZfEso1H12XLvd1wyuUhs4wcI48AYo//mWCXlJeT jqL9FiGb+4rlnBQk7ZKYk17Y8LG+uNRdpGGbegsBIKL3B8qEwsdH32eltphF7uqL+PaP iMXA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=Yui7DkA7gAkkQNUNs6iJF6mhVXZnWh/iv89nB7IAwIg=; b=JDR16pwI/O6PWEe7kwWhy2QUfOCyu8B2YnE29niBXdNc19aJkPGPpcMUT51y0n7BtM 7vzFtip9kBlLj0EWObHW2rNTAXgcbnKOuVfQxvnuTEsLaue6pIDKLBUq6CTFAWGh8vtR 2mfxjvA1FDdK9YXWqMAxtdj0Bqj6+Oa2G7Kvn4+IHktPAebUKZE8H7DqHzfVGmgnjnCW YlT6AcY7ga4g7VUFFGskiZ5lk2bF8bpbzvbGzaZ5m/1YajXnNX5Nh8+Ot+q5wpuFRarQ 0/eNqhdjUFTq6/ySBvmAr7eRrbe7XMFMUI+qDiSLhl3mJ43E71ai7wc53EcbEbXmnbx7 RcJA== X-Gm-Message-State: AOAM530q5pT3wLzNuevzFdQ4s1/uL+i+rDJe/UhpP0RmdIeAc9TPdThM qagGaWhrX3KEPaMpCwiOd2XmpGzTcPW96Q== X-Google-Smtp-Source: ABdhPJzuW+Yy9zkdYzaYKqzksqlp3pnTI16M7u0rk4pN2l6eODKntLdGKD84VtqO6MnH5/lziPZp0g== X-Received: by 2002:a7b:c407:: with SMTP id k7mr3100895wmi.136.1615282936750; Tue, 09 Mar 2021 01:42:16 -0800 (PST) Received: from localhost (host86-134-238-232.range86-134.btcentralplus.com. [86.134.238.232]) by smtp.gmail.com with ESMTPSA id z2sm27315877wrm.0.2021.03.09.01.42.15 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 09 Mar 2021 01:42:16 -0800 (PST) Date: Tue, 9 Mar 2021 09:42:14 +0000 From: Andrew Burgess To: gdb-patches@sourceware.org Subject: Re: [PATCH] sim: switch to autogenerated ChangeLog files Message-ID: <20210309094214.GN1720904@embecosm.com> References: <20210110033752.6002-1-vapier@gentoo.org> <20210110034223.6268-1-vapier@gentoo.org> <20210111110544.GC1151657@embecosm.com> <313988fb-ed42-11f7-1e64-b46005580792@polymtl.ca> <20210111193830.GT7494@vapier> <20210112104746.GJ1175365@embecosm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Operating-System: Linux/5.8.13-100.fc31.x86_64 (x86_64) X-Uptime: 09:25:59 up 90 days, 14:10, X-Editor: GNU Emacs [ http://www.gnu.org/software/emacs ] X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces@sourceware.org Sender: "Gdb-patches" * Mike Frysinger [2021-03-09 00:51:15 -0500]: > On 12 Jan 2021 10:47, Andrew Burgess wrote: > > Mike Frysinger [2021-01-11 14:38:30 -0500]: > > > On 11 Jan 2021 12:00, Simon Marchi wrote: > > > > On 2021-01-11 6:05 a.m., Andrew Burgess wrote: > > > > > I think sim/ should follow the same policy as gdb/ for now. Do feel > > > > > free to raise this as a suggestion for gdb/ in general though, it > > > > > would be an interesting conversation to observe. > > > > > > > > Last time I tried the gitlog-to-changelog script on GDB, it produced > > > > horrible results. Probably because it was not designed for C++. > > > > > > > > Making a script to produce ChangeLogs for C code is already very > > > > difficult, making it work with good results for C++ would be even > > > > worst. And most importantly, I think it would be wasted development > > > > time. > > > > > > to be clear, it isn't generating entries exactly like we write. it's > > > using the git commit logs with formatted dates. so i don't think this > > > applies exactly anymore. so it's inline with the GNU's VCS principles: > > > https://www.gnu.org/prep/standards/html_node/Change-Logs.html > > > (and that also recommends just using gitlog-to-changelog). > > > > I read this page, and especially the part that talks about using > > gitlog-to-changelog, and I don't find their argument compelling. > > > > Early in the page they list some uses for ChangeLogs, which basically > > boils down to performing "software forensics". Their benefits are all > > good, but IMHO are all covered (and covered better) by what the VCS > > provide. > > > > At the end of the page they say if you don't maintain a ChangeLog then > > use a script to generate one in case people want to look at the > > ChangeLog. But they fail to explain what use a ChangeLog is in a > > release tree. The same argument could just as easily be used to > > justify including a poem in the release; it should be there in case > > someone wants to look at it. > > > > For me the question is what benefit does a ChangeLog offer in a > > release tree? When I look through the 7 points (in favour of > > ChangeLogs) raised on the above page I don't see what value any of > > those things have given only a release tree. > > > > If we really wanted to include something informative inside each > > release then I'd be most tempted to just do something like: > > > > git log --stat last-release-tag...new-release-tag > GIT-LOG > > > > That would give people a far more detailed understanding of what has > > changed. > > > > But honestly, I don't think it's unreasonable to say that if someone > > wants to dig into the source history, just clone the repo.... and let > > ChangeLogs burn! > > so how are we feeling now ? i think we agree that we don't want to hand > maintain ChangeLog files. > > then we need to decide on whether we want to include ChangeLog files for > sim/ at all in releases. i'd be fine with just dropping them entirely if > we don't want to bikeshed it anymore. we can create a sim/ChangeLog that > just reads: > Please see the git log online at: > https://sourceware.org/git/?p=binutils-gdb.git;a=history;f=sim > > if we do include them, i think it's reasonable to expect they take a form > similar to the other ChangeLog files in the tree. that would mean using > gitlog-to-changelog, or some other gnu tool that i've missed. the upside > to that over a raw `git log` is that, for old commits, it wil extract any > ChangeLog entries and inline them in the log. this would be an easy way > to bridge the old & new ways of writing log entries. My general position is that sim/ is really a sub-component of gdb, as it is only ever released as part of gdb, and as such should follow GDB policies and coding standards, just for consistency. That said, ChangeLogs suck so much, so maybe we could use this as an opportunity to drop them, and maybe set some precedent for changing gdb/ one day (hey, I can dream!). I think the one thing I would say is that auto-generated ChangeLogs are worse than what we currently have, so can we please agree to take them off the table. I've not yet seen a tool that can 100% accurately figure out the names for changed entities, nor does an auto-generated script ever say anything useful about what changed. So you're left with a list of things that _might_ have changed, with no information about _what_ changed in them. Just a complete waste of time. I guess the one thing that has surprised me is that nobody has jumped in to really defend ChangeLogs. Maybe there really is hope that we could drop these for gdb/... I think you should create a new top-level post, with a subject line that makes it clear you're planning to drop ChangeLogs for sim/. If nobody complains to that then go with your plan, just point folk to the online history. Thanks, Andrew