From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ot1-x333.google.com (mail-ot1-x333.google.com [IPv6:2607:f8b0:4864:20::333]) by sourceware.org (Postfix) with ESMTPS id 1A8E0396E44B for ; Wed, 7 Jul 2021 16:39:16 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 1A8E0396E44B Received: by mail-ot1-x333.google.com with SMTP id i12-20020a05683033ecb02903346fa0f74dso2798764otu.10 for ; Wed, 07 Jul 2021 09:39:16 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=37QycA8e/EqQKYsPk/nfmSSPzEd6ScPbhaO1Ef7tFiE=; b=adMu1Pn1beppnzyBpPanb8FxOrHyDtsRjNRQjcS0m82m0vDnPZQBs+DxnOJna+4Leu pQqhF436UYfBPw055G3suVm40SPqfjKnVocWL5fPj47AgzbbIhfvSiUz7AW2CTGnK8pk hweuf0wPtwrPsVcRFsv1UKFryYrB8Q6yVuL72qJmdo/oo6c80R+CDtB57SuI1kIHZDgb bkfUwygEHT12wwneZeyO2aeUWmibai5AXVtV2AeE8AKk2Vhc+sepLFfElxLZKH4Htlxt DMgkyDjjrAaQyEJ3tU7G4JbuKSRE2X8Z8RE6DhGvXx92HjtFWIVxcByLpGgnBWyzqPlY a5YA== X-Gm-Message-State: AOAM532nA9TWvn0oBvF+CGtFpsTkIiJX/6aAFxGC2zvtKQ5owuFZUamY CLAmdLZjnCTdzKpgDKBekjuaYylGo4Y= X-Google-Smtp-Source: ABdhPJz4RBYP4puu+7YWjIM1d10OtYHjp9fVul16SaP68RdFimH3HG9ckFWNo9CCRj6Jzy1nMyc/7A== X-Received: by 2002:a9d:3a34:: with SMTP id j49mr20275771otc.102.1625675955383; Wed, 07 Jul 2021 09:39:15 -0700 (PDT) Received: from [192.168.0.41] (75-166-102-22.hlrn.qwest.net. [75.166.102.22]) by smtp.gmail.com with ESMTPSA id c64sm4320302oif.30.2021.07.07.09.39.14 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 07 Jul 2021 09:39:15 -0700 (PDT) Subject: Re: where is PRnnnn required again? To: Jonathan Wakely Cc: Marek Polacek , gcc mailing list References: <9121724e-e741-9bad-a39d-d6ac49422589@gmail.com> From: Martin Sebor Message-ID: Date: Wed, 7 Jul 2021 10:39:14 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, KAM_SHORT, NICE_REPLY_A, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 07 Jul 2021 16:39:17 -0000 On 7/6/21 4:09 PM, Jonathan Wakely wrote: > > > On Tue, 6 Jul 2021, 22:45 Martin Sebor via Gcc, > wrote: > > On 7/6/21 3:36 PM, Marek Polacek wrote: > > On Tue, Jul 06, 2021 at 03:20:26PM -0600, Martin Sebor via Gcc wrote: > >> I came away from the recent discussion of ChangeLogs requirements > >> with the impression that the PRnnnn bit should be in the subject > >> (first) line and also above the ChangeLog part but doesn't need > >> to be repeated again in the ChangeLog entries.  But my commit > >> below was rejected last Friday with the subsequent error.  Adding > >> PR middle-end/98871 to the ChangeLog entry let me push the change: > >> > >> https://gcc.gnu.org/g:6feb628a706e86eb3f303aff388c74bdb29e7381 > >> > >> I just had the same error happen now, again with what seems like > >> a valid commit message.  Did I misunderstand something or has > >> something changed recently? > >> > >> Martin > >> > >> commit 8a6d08bb49c2b9585c2a2adbb3121f6d9347b780 (HEAD -> master) > >> Author: Martin Sebor > > >> Date:   Fri Jul 2 16:16:31 2021 -0600 > >> > >>      Improve warning suppression for inlined functions [PR98512]. > >> > >>      Resolves: > >>      PR middle-end/98871 - Cannot silence -Wmaybe-uninitialized at > >> declaration si > >> te > >>      PR middle-end/98512 - #pragma GCC diagnostic ignored > ineffective in > >> conjunct > >> ion with alias attribute > > > > This should be just > > > >       PR middle-end/98871 > >       PR middle-end/98512 > > > > , no? > > Does it matter if there's text after the PR ...? > > > > Yes. With extra text the whole line is just treated as arbitrary text, > not a "PR component/nnnn" string. So with the extra text it won't be > added to the ChangeLog file, and won't match the PR in the subject line. > >   I managed to push > > https://gcc.gnu.org/pipermail/gcc-cvs/2021-July/350316.html > > that uses the same style earlier today > > > But will it add the PR numbers to the ChangeLog? I think the answer is > no (in which case you could edit the ChangeLog tomorrow if you want them > to be in there). It updated Bugzilla but it didn't add the PR numbers to the ChangeLog entries. I still don't (obviously) understand the rules the hook uses for what to update or the rationale for them. It seems as though the PR in the subject is used to update only Bugzilla but not also update the ChangeLogs (why not?) The PR component/nnnn part that's supposed to come before the ChangeLog is used to update ChangeLog entries but seems to be ignored if it's followed by any text (why?) These unnecessary gotchas aren't documented anywhere. Martin