From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-1.mimecast.com (us-smtp-delivery-1.mimecast.com [207.211.31.81]) by mx.groups.io with SMTP id smtpd.web11.5125.1591726640044009466 for ; Tue, 09 Jun 2020 11:17:20 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=anTTrxGT; spf=pass (domain: redhat.com, ip: 207.211.31.81, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1591726639; h=from:from:reply-to: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=ZJqlEfB+rDWfAJjpiHBLfFe33VSRGlLBjSoKz98d9rc=; b=anTTrxGT2w1123TWCT9i8CT0pttaX8WIQSMQkRWH/RInbn/TFdxYhLISlhqa5Z1sSki7jD F97G/kbUKlD6dqdui0q5nlj/30ZlGyKV/1tqDMtna7Wy7Fe1IaNWvd/O+QOW+/VBwgNcEG YAR1viwAbyZD94Yb+UxIWq6uXfkH7Yw= 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-427-o9vvXO_9OMC5_3FQnJ80Ag-1; Tue, 09 Jun 2020 14:17:12 -0400 X-MC-Unique: o9vvXO_9OMC5_3FQnJ80Ag-1 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 552C2464; Tue, 9 Jun 2020 18:17:09 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-194.ams2.redhat.com [10.36.112.194]) by smtp.corp.redhat.com (Postfix) with ESMTP id 73CBA1002388; Tue, 9 Jun 2020 18:17:02 +0000 (UTC) Subject: Re: [edk2-devel] [PATCH 0/3] Maintainers.txt: restore lexicographical order between package / subsystem headers From: "Laszlo Ersek" To: edk2-devel-groups-io Cc: Andrew Fish , Anthony Perard , Ard Biesheuvel , David Woodhouse , Eric Dong , Hao A Wu , Jiewen Yao , Jordan Justen , Julien Grall , Leif Lindholm , Liran Alon , =?UTF-8?Q?Marc-Andr=c3=a9_Lureau?= , Michael Kinney , Nikita Leshenko , =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , Sami Mujawar , Stefan Berger , Supreeth Venkatesh Reply-To: devel@edk2.groups.io, lersek@redhat.com References: <20200603160627.3594-1-lersek@redhat.com> Message-ID: Date: Tue, 9 Jun 2020 20:17:01 +0200 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: <20200603160627.3594-1-lersek@redhat.com> X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit On 06/03/20 18:06, Laszlo Ersek wrote: > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2778 > Repo: https://pagure.io/lersek/edk2.git > Branch: sort_maintainers > > We originally meant to keep the subsystem (package) titles sorted, in > "Maintainers.txt". > > However, as of 7191dd3c5990 ("ArmPkg/PlatformBootManagerLib: reject > 'default' parity and stop bit count", 2020-06-03), we have some disorder > introduced. > > This patch series restores lexicographical order between the subsystem > (package) headers. > > The package / subsystem headers can be collected with the following > small script: > > $ dos2unix < Maintainers.txt \ > | sed -n -e '/^EDK II Packages:$/,$p' \ > | tail -n +3 \ > | grep -E -v '^($|[LMRWTSFX]:)' > > Where > > - "dos2unix" strips CRs, > > - "sed" removes the first part of the file that precedes the "EDK II > Packages:" heading, > > - "tail" removes that heading plus the underline on the next line, > > - and "grep" removes the empty lines and the section entries. > > What remains is the section headers. > > Here's a comparison between the outputs of this script, before (that is, > at commit 7191dd3c5990) and after the series is applied: > >> @@ -1,42 +1,42 @@ >> ArmPkg >> ArmPlatformPkg >> ArmVirtPkg >> ArmVirtPkg: modules used on Xen >> BaseTools >> CryptoPkg >> DynamicTablesPkg >> EmbeddedPkg >> EmulatorPkg >> FatPkg >> FmpDevicePkg >> IntelFsp2Pkg >> IntelFsp2WrapperPkg >> MdeModulePkg >> MdeModulePkg: ACPI modules >> -MdeModulePkg: ACPI S3 modules >> +MdeModulePkg: ACPI modules related to S3 >> MdeModulePkg: BDS modules >> MdeModulePkg: Console and Graphics modules >> MdeModulePkg: Core services (PEI, DXE and Runtime) modules >> MdeModulePkg: Device and Peripheral modules >> MdeModulePkg: Firmware Update modules >> MdeModulePkg: HII and UI modules >> MdeModulePkg: Management Mode (MM, SMM) modules >> MdeModulePkg: Reset modules >> MdeModulePkg: SMBIOS modules >> MdeModulePkg: UEFI Variable modules >> MdePkg >> NetworkPkg >> OvmfPkg >> -OvmfPkg: Xen-related modules >> -OvmfPkg: TCG- and TPM2-related modules >> OvmfPkg: CSM modules >> OvmfPkg: MptScsi and PVSCSI driver >> +OvmfPkg: TCG- and TPM2-related modules >> +OvmfPkg: Xen-related modules >> PcAtChipsetPkg >> SecurityPkg >> ShellPkg >> SignedCapsulePkg >> SourceLevelDebugPkg >> +StandaloneMmPkg >> UefiCpuPkg >> UefiPayloadPkg >> UnitTestFrameworkPkg >> -StandaloneMmPkg > > Cc: Andrew Fish > Cc: Anthony Perard > Cc: Ard Biesheuvel > Cc: David Woodhouse > Cc: Eric Dong > Cc: Hao A Wu > Cc: Jiewen Yao > Cc: Jordan Justen > Cc: Julien Grall > Cc: Leif Lindholm > Cc: Liran Alon > Cc: Marc-André Lureau > Cc: Michael Kinney > Cc: Nikita Leshenko > Cc: Philippe Mathieu-Daudé > Cc: Sami Mujawar > Cc: Stefan Berger > Cc: Supreeth Venkatesh > > Thanks, > Laszlo > > Laszlo Ersek (3): > Maintainers.txt: retitle "MdeModulePkg: ACPI S3 modules" > Maintainers.txt: restore order of OvmfPkg sections > Maintainers.txt: move StandaloneMmPkg to the right spot > > Maintainers.txt | 54 ++++++++++---------- > 1 file changed, 27 insertions(+), 27 deletions(-) > Merged as commit range 6ff7c838d092..6aa48ab791ec, via . Thanks Laszlo