From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE,SPF_PASS, USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 789C7C433F5 for ; Wed, 8 Sep 2021 11:38:42 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id BE958610C8 for ; Wed, 8 Sep 2021 11:38:41 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org BE958610C8 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=sancloud.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 4DD6E83340; Wed, 8 Sep 2021 13:38:28 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=sancloud.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=sancloud.onmicrosoft.com header.i=@sancloud.onmicrosoft.com header.b="mtzOmk4L"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 014F782BB2; Wed, 8 Sep 2021 13:38:17 +0200 (CEST) Received: from GBR01-CWL-obe.outbound.protection.outlook.com (mail-cwlgbr01on072c.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe14::72c]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id C23FD8323E for ; Wed, 8 Sep 2021 13:38:10 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=sancloud.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=paul.barker@sancloud.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=UpMPA/E7r3xwtUveZOk+VCqilqURw6HIPjGyGgrsuJkXe56txBWOWPTeouOfq9JdQx1lGGruV7IEMZKFxOX+puUng8cDtrPekitQY07bQInSZmTINDes4zgRXvuSPcmPL5un6y5Z/lho+faZ+vJvq494zmp8tuguTi+cwwben2YzMSAsDndHVA8pUX+4Ae1PqCmqpawmbgiciToh+S5EoVNtYlBowZ/Zzw7T61PLwyWjtDBib3moZwp6wG2V+aTY/eUKDdIfYTvoQRfIOQD+zUXNy3avWdQY5rLU1Ldan/aaDsPGKQq0Hw9rAOVQpO8BMfCFvsnz+Ce1/82COtiQZQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=JWmz6Teh0IEJeAmCcQRSps/6cHy22knl4gfCE9PjRic=; b=fa5b+Pvl8mATJ2Qu0rZjgVNtg7jVr9mA9VE1vxUEzB1pyon6TRiG2oDAaz3yt0pwcmu3dCUxDEYKoEs3A73Fg7uUGqeDy/B12YXkk63hGB16cQnsmwBMqJcXV73vcr6+8ZmbWx37mV1SuSHQk1/jqNq3dfJqi2dIfXq9Lxphf/frpuLy3OuYWUEb6jBFyIQBueCIeJJbGIjFiGTJ8x7lNc5NCo8lIMuH4lu4JS6fr2TgUGmlG+1fHAl/uqeGmRRWoPmY7iKVCJ/Q66buidV4Q0XFDTt8tLmIZhVK8UYsVz6ayQvFZiLO04PLVD2UxIw5Pg359ubxGUW+wmMCtMOLyA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=sancloud.com; dmarc=pass action=none header.from=sancloud.com; dkim=pass header.d=sancloud.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sancloud.onmicrosoft.com; s=selector2-sancloud-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=JWmz6Teh0IEJeAmCcQRSps/6cHy22knl4gfCE9PjRic=; b=mtzOmk4LJ7xuafOIszy3n+/VlN2whb6VBu3/+DrJk/0uYJnZw0euGGvobsGr52NTRSU3W1VB10Z3tfrNvQN56id3X/K0u40f3qiFIYmZ5ik8N/2iRrOW8qeKKyafqJs4dwWY5rxzf+lFnzwhOwImy+d9lim80uI/1i2yhRTmF3o= Authentication-Results: chromium.org; dkim=none (message not signed) header.d=none;chromium.org; dmarc=none action=none header.from=sancloud.com; Received: from CWLP123MB2241.GBRP123.PROD.OUTLOOK.COM (2603:10a6:401:61::19) by CWXP123MB3045.GBRP123.PROD.OUTLOOK.COM (2603:10a6:400:42::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4478.22; Wed, 8 Sep 2021 11:38:10 +0000 Received: from CWLP123MB2241.GBRP123.PROD.OUTLOOK.COM ([fe80::d45d:1046:b6e9:de6e]) by CWLP123MB2241.GBRP123.PROD.OUTLOOK.COM ([fe80::d45d:1046:b6e9:de6e%7]) with mapi id 15.20.4500.015; Wed, 8 Sep 2021 11:38:10 +0000 From: Paul Barker To: Simon Glass , Tom Rini , u-boot@lists.denx.de Cc: Paul Barker Subject: [PATCH v4 2/2] tools: Handle PAGER containing arguments Date: Wed, 8 Sep 2021 12:38:02 +0100 Message-Id: <20210908113802.5252-3-paul.barker@sancloud.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20210908113802.5252-1-paul.barker@sancloud.com> References: <20210908113802.5252-1-paul.barker@sancloud.com> Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: LO2P265CA0495.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:13a::20) To CWLP123MB2241.GBRP123.PROD.OUTLOOK.COM (2603:10a6:401:61::19) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from badger.mshome.net (80.7.160.81) by LO2P265CA0495.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:13a::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4478.20 via Frontend Transport; Wed, 8 Sep 2021 11:38:09 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 4b0482da-96ed-4b30-fe8f-08d972bd225d X-MS-TrafficTypeDiagnostic: CWXP123MB3045: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:2958; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: IAI3AyWSzTBzJTwJR8wHkRuSK4JGKNXE8pchxps0mQYAurNC28O3eQYHt/dyb/h3VHZW1fu1eoJtg/jNPjJjwXJqjeRY/xmw/VFvKxhWv0wKZeLSIJSRVGU7rPe5zW7gAgBGnlzAvDpdCbJ0u/uKm59aDRdnpaDVmU99CquVWy9pag9fD+3bC2n+YJzn8YDmDhnPduE+9Zj44TNmyKK8/+OixSnNMoPTMeWYUI+emsaUqzBSyxAX6LCyE0j0gSWZjC6o6vzns8s70bmyYucgbY9eTSmZE1c0qNeIdGmWldOGsfZTyfTGEgH+Id2IrxbcnIUoS0vYil0K3z3mVJEvgE8oe6/oiiYxLGEpfjbgXYgJ7+OSQC2lrIGaRJewVfW4h/U6pWpOUcrGq7gRCXeuww6rHQN31o48roXfJmjvtLxYzK4C+42fhawYf6+DiocGAoYhLhOmB2A0408ICl2zBBZjhrWKokj+IdB3wZDTPqxVAkCtgv8EqeVSO2DrhmLqSrC6GbvLg5Jct6LUknFiLNBwulfDrLZWilLyEDHV58uuWxWxZot34OpEprdebGYPNu4cdA3iEB0DQIMmlsp56Cv5Mq0G6Qs1IaihvoLB2B1O+3rTryYKM4vWoSaEVTQ6V7Qrlaf3k+ZHCnvN6R8TDztVA8UKZ/uR+Ky4SCXTGE7eRFgC8fpcUoXsx4e7syTNhoCtlLP7MD/s8vocTvmwQw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CWLP123MB2241.GBRP123.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(366004)(39830400003)(346002)(136003)(376002)(396003)(1076003)(2616005)(956004)(186003)(2906002)(83380400001)(508600001)(4326008)(6506007)(6486002)(110136005)(52116002)(6666004)(8676002)(6512007)(8936002)(316002)(38350700002)(38100700002)(26005)(36756003)(66946007)(66556008)(66476007)(107886003)(5660300002)(4744005)(44832011)(86362001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?qlVrlfFDNdROpN7DcsJPRmVBSv5u8oFoZyHC+eYYwPtEuQ4xhBJayYKRytKd?= =?us-ascii?Q?rWjeEYn9MTFGkjrmOtFohf3K29YNZZbo289166ZYiWHS9f1FHdcYKbcmM1OP?= =?us-ascii?Q?sr9bucX9PENGCNIA5IqDuvGbxZteTWRPTgHIdkfarDxwzMSqeRRO8XNQiqXw?= =?us-ascii?Q?gOoMxZG2+NG242J+Q5Yz7M2CHbZsjdKz3JWNHWRc0Rm2CsW8GjggyW1+896+?= =?us-ascii?Q?IJsrhN1EckjbjlalhzLyeszOumPf6lgU1yJCSk3ONPvNEfuuK6ZceEhHvxlU?= =?us-ascii?Q?KDIKhV3Iewt0ggNlgbgSDjnc7B/ECW/l6aj/3kpTU6vGWySQqrljCQst6qeD?= =?us-ascii?Q?68sVWVrXNM49qgwiMY4cGdu8HR9DSbivAXLsKZ4grSYvw9qBgXSJFirqkDhm?= =?us-ascii?Q?iHvtKUxM8LyNqPRfaso9N0t7H8IKdu6p/myhnIMyKWEBH6R38wpDoI14dFKh?= =?us-ascii?Q?fSEAzTnvJ4XkKFX5aXl36MqHCBQIomT0ZKeEvx6m1SPrKQvb3C/d1Y06/amb?= =?us-ascii?Q?mVZxlOkaG2ItiwmYUv8zBfwZoHQ3iIZvTNN0tW5EUh3OsYtm7BIsiC6Uubot?= =?us-ascii?Q?o12AQcGERbGJC5YM9P63vanfGH5Xypi3del+B43Lcgxm5guZ6HBoY98Ik/VQ?= =?us-ascii?Q?Ta8P1j4rFKhDxpLgzugKI5ZA5UG1Z3wWNc/uFrNgFGaQE0lrax1CdinlTZyM?= =?us-ascii?Q?kmdMNDjBGC/ZMJ0sK0Zu4RS3x8383nvHhTKSCN05zIuiZ+tnsiHaq2ITAT0S?= =?us-ascii?Q?/Lkq8d2iGbGGgSP5VaVGIKhiezN/ET/8rIqo+LwodID7l0JLL7G5ih0uRNbH?= =?us-ascii?Q?YIrIlRf2K0CKbVJejpEX/8O/9RnBESjUJ4eRbmlOGOMv0lAQSsv7ODpmI5iq?= =?us-ascii?Q?1bHsyH/L/ypZxIkAxhB+4tXHRo8B3MP9jk0MHUmmLScodxmv3eBIOy7/ACtm?= =?us-ascii?Q?3zgk6K/u431dF8S1rDdO71pktmqJX0enHejLwvwl+gQa+u3BifWEB+p2dnJa?= =?us-ascii?Q?tCuEAjGXq+g7LEGjThSeDYcrMYNxrI+k394RZhIxI+VJVMbkBQZthVXXYErk?= =?us-ascii?Q?0nX00vyh2PfNRG28QH3WBiy3mq/uE2W+6bLHOqSqrJgqJPP2j47iaL2Hs2Ta?= =?us-ascii?Q?WFQMbnZqfLu1BHy8JK5hiEHd/i07TR1WrTwc+ZS/8p2hIBUvPAccrelJ1VDG?= =?us-ascii?Q?cmiq694wWr3ZTFD4RQYzPYF6NEQ4dlTK3XoiazJQ1du0+ZI8lRriWkmX+7HS?= =?us-ascii?Q?qRn0I/wo313C2oEaI0N1UCe3WsVRzlo4Jn2o5ovBwDxO9LeS1o8MifxCC1AY?= =?us-ascii?Q?0GKYrOL80VnBlgJix4NLTByX?= X-OriginatorOrg: sancloud.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4b0482da-96ed-4b30-fe8f-08d972bd225d X-MS-Exchange-CrossTenant-AuthSource: CWLP123MB2241.GBRP123.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Sep 2021 11:38:10.1676 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 840be37c-244a-450e-9bcc-2064862de1f4 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: QiSbBU+02YYZJw47rFJIFxpK9MF0gfDOH4UXX5wdeXUfQjyyTZImcYUk1rCQ8yQ62+bhcxYBU8koIbTnZi5gi+8Fn9MN6pOenbm+NAU9Q98= X-MS-Exchange-Transport-CrossTenantHeadersStamped: CWXP123MB3045 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean When printing full help output from a tool, we should be able to handle a PAGER variable which includes arguments, e.g. PAGER='less -F'. Signed-off-by: Paul Barker --- tools/patman/tools.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tools/patman/tools.py b/tools/patman/tools.py index 96882264a2f9..710f1fdcd361 100644 --- a/tools/patman/tools.py +++ b/tools/patman/tools.py @@ -5,6 +5,7 @@ import glob import os +import shlex import shutil import struct import sys @@ -588,9 +589,10 @@ def PrintFullHelp(fname): Args: fname: Path to a file containing the full help message """ - pager = os.getenv('PAGER') + pager = shlex.split(os.getenv('PAGER', '')) if not pager: - pager = shutil.which('less') + lesspath = shutil.which('less') + pager = [lesspath] if lesspath else None if not pager: - pager = 'more' - command.Run(pager, fname) + pager = ['more'] + command.Run(*pager, fname) -- 2.33.0