From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.120]) by mx.groups.io with SMTP id smtpd.web10.7583.1582823866111865895 for ; Thu, 27 Feb 2020 09:17:46 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=fNz98r0r; spf=pass (domain: redhat.com, ip: 207.211.31.120, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1582823865; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=x95EfYNpPwn8x7rKxWE9YOxufD8PisSnsD7XYy2nHI8=; b=fNz98r0rg5aYn2nM4fkSJ1e/TmDTNU7qqmXei3HqUvEgpbTCtzGwZWJxwab638Fb3PK79h RdNA1nrCD9acRSYH7rWOwsFpNzdo9pjNM93TdQ7PRLwkBgy8+q6632sv01z5969nurBXae FWc/4ry+dBIfIDyouLwKkvVgW10IXyQ= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-235-Xfxi05I9PtiQWTl0SeK3JA-1; Thu, 27 Feb 2020 12:17:40 -0500 X-MC-Unique: Xfxi05I9PtiQWTl0SeK3JA-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 2FCC8800EBB; Thu, 27 Feb 2020 17:17:39 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-116-46.ams2.redhat.com [10.36.116.46]) by smtp.corp.redhat.com (Postfix) with ESMTP id 7B11A90A16; Thu, 27 Feb 2020 17:17:37 +0000 (UTC) Subject: Re: [edk2-devel] [Patch 1/1][edk2-stable202002]BaseTools: Fixed a regression issue in Makefile for incremental build To: "Feng, Bob C" , "devel@edk2.groups.io" , Andrew Fish , Leif Lindholm , "Kinney, Michael D" , "Gao, Liming" Cc: Pierre Gondois References: <20200227094705.25404-1-bob.c.feng@intel.com> <32761feabc3847c6847b3ff908ce9014@intel.com> From: "Laszlo Ersek" Message-ID: Date: Thu, 27 Feb 2020 18:17:36 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <32761feabc3847c6847b3ff908ce9014@intel.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit On 02/27/20 16:53, Feng, Bob C wrote: > [Bob] I agree the BZ status should be update in time. I don't think BZ status update is the reviewer's/maintainer's responsibility, the BZ owner should be responsible for it. Agreed. > > NOTE: GitHub.com Pull Requests would not help *at all* in the face of such sloppiness; even on GitHub.com, people have to at least *name* issue numbers in commit messages. > > - TianoCore#2563 (which tracks the regression) identifies *neither* the BZ for which the regression was introduced (2481), *nor* the faulty commit (818283de3f6d). You realize it's *completely useless* to file BZs with such negligence, right? It has no more information than "stuff broke, we need to fix it" -- but ain't that the general state of things, at all times? Are you only trying to fill a BZ quota? > [Bob] I don't agree this comments. > I added the bug reproduce steps in BZ description. I think it's enough when I submit a new BZ. I'll append the root cause and solution ( would be just patch review link) in its comments when I update the BZ status later. Yes, the patch explains the issue well. If the link had been in the BZ, I wouldn't have complained (as much). > We found this critical bug in this afternoon (PRC time) and root cause and created patch very quickly. I don't think that I did not update the BZ in time is process violation. It was not clear that you ever intended to add the link to the BZ. > I think the necessary information was provided when the patch send out. The bug description and reproduce steps are in BZ, root cause is in the patch commit message, the solution is the patch itself, test result is in the commit message. Yes. There was no link from the BZ to the patch however. And it wasn't possible to determine whether you were going to add the link later. I tend to add the link immediately after posting, so I don't forget. My experience tell me that most patch submitters that don't add the link at once forget for good. Yes, it was a generalization, sorry about that. One thing I do have to admit (because I brought up GitHub.com before) is that, on GitHub.com, if you submit a pull request, and at least one of the commit messages references an Issue (like your commit message here references TianoCore#2563), then the issue automatically gets an update. So in that regard GitHub.com does save some manual work. Laszlo