From patchwork Tue Jan 4 12:34:30 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joel Holdsworth X-Patchwork-Id: 12703418 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3013FC433EF for ; Tue, 4 Jan 2022 12:35:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233005AbiADMe7 (ORCPT ); Tue, 4 Jan 2022 07:34:59 -0500 Received: from mail-mw2nam12on2050.outbound.protection.outlook.com ([40.107.244.50]:2528 "EHLO NAM12-MW2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S229502AbiADMe6 (ORCPT ); Tue, 4 Jan 2022 07:34:58 -0500 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=VhYu74PqKBUUHyraoAC13TV9B20ajnxaOpM6xvNOT0WeOmJEgxH3Smafrrllx85dn1i5LXiom91JTe3XcSqpD75cfl5TqR9/G7xbP8RtmtwoX46bp+aBMvuQWk4lNQLoaMmeiRPB9utSlRHaySh5hjzjfjQcXJ85D4N1QJCD4qBJLO1ycPuMmbliXqWWNF6fGJWPc3v3F4fE+3xp8Ox4+gp5QA8vDixJZRuzfTHMslD258/8BVNoOrGQ8GdRlljf8FaH+LYm9yoH5bt1WWFTdN3cXeJm+Ep38Wq3BVJhgJOSbsWra1xoDmZ2Fp6UvY486J7oiA9j+IR+SGsvPK/2Qw== 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:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=/gU3MF7gGQYtV4aF8/ecE33bN7jRFtySedVwflNQ9jU=; b=VphClMQpc0FsxGMgNHzcJxpBflDKj4gvtiVI70vrg59POysbnANopA/BfKjdKyvr12CWsOWz4d7Y/XP/i+itRxXKgva3q8oqQb+1BECFOwKOOF7Liz54yuXFb4Nc9iNjeteDg/98ZcY5wY430RxsHq4fv21UgN/CBnpGfaFxZ/Wkn4Apnt102QvnFlkmZZLkT2cCCkESIgSnyh5o4PD3W5i9rFJltrZZ1YauEefFgzktHsz3y0w6fA5NPAAPI2JKEviRNzFS3xNOeGlN5kVY9oSo1q6UXl+/KMZTAIAn2dxmCqu1iw+WUSc8FRC8ikzE8dmMFtaGvDEfySInCq1dsA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/gU3MF7gGQYtV4aF8/ecE33bN7jRFtySedVwflNQ9jU=; b=q08O9dWTWvoJ9kB4B3kJwSGFzF26V24wfQktJEaAch4ncKithjtT3v+mJNKd19wYMlIXZUGa2yEyk7sXNr3GiA7iC6c0uN4mUB+D948RF7LZjQHork4j8JMLV2y2Z3Kr5xwtZ7pXdrknHIMZviAmK7nDCpeQetTMnU6+CCzD2J4Du9meK66sVCnhKA8eKGJz/sX5M1ylB0bfM9PWph2tFQonLgr5VOMhIZkGz+WK/WIlZgwpq7x9x+UsU2A3e7JArqkUvayHn15Kug9mTJj4c2E8mTjdBgAorstcfbTimK1Xk9rEfNj8JJF4VlaC9HnystqK7S7kvhSmbekhBv0toQ== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from BL0PR12MB4849.namprd12.prod.outlook.com (2603:10b6:208:1c2::17) by BL0PR12MB4866.namprd12.prod.outlook.com (2603:10b6:208:1cf::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4844.15; Tue, 4 Jan 2022 12:34:57 +0000 Received: from BL0PR12MB4849.namprd12.prod.outlook.com ([fe80::516f:3a1f:4951:ebd6]) by BL0PR12MB4849.namprd12.prod.outlook.com ([fe80::516f:3a1f:4951:ebd6%5]) with mapi id 15.20.4844.016; Tue, 4 Jan 2022 12:34:57 +0000 From: Joel Holdsworth To: git@vger.kernel.org Cc: Tzadik Vanderhoof , Dorgon Chang , Joachim Kuebart , Daniel Levin , Luke Diamand , Ben Keene , Andrew Oakley , Joel Holdsworth Subject: [PATCH v3 RESEND 1/2] git-p4: remove "debug" verb Date: Tue, 4 Jan 2022 12:34:30 +0000 Message-Id: <20220104123431.1710-2-jholdsworth@nvidia.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220104123431.1710-1-jholdsworth@nvidia.com> References: <20220104123431.1710-1-jholdsworth@nvidia.com> X-ClientProxiedBy: AS8P250CA0010.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:330::15) To BL0PR12MB4849.namprd12.prod.outlook.com (2603:10b6:208:1c2::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 5aa47d29-879d-4f79-8981-08d9cf7e9ddf X-MS-TrafficTypeDiagnostic: BL0PR12MB4866:EE_ X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:2887; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 3rXcdxvM9krIILGvfJ5tjJk4RB2VOvVCYHXvMDyM2jBXZLp4jz3sZOAnaSBO+YqQAGrwwKVAZUk6Lo0mqIGBv9vDInRCrXgR3VMbawJyMUKKptgvzxo+UQYrACHUB4kT63h2jj46MnX4pGdC7L8KZwJsbOCQTCMD7ZUISwOj/AQjxDOG7drHDwQV4pV9ulapzsT3ddJSdqU5Oje/HwGuu+QKtSq/OWHl4sGDjH7T4JzAnNsT9YKmt2GVPtBMx6Sd0oGX8XSTAlvnjorxu1pU1I/tg8kJD9CTmnBb8VZPxGCtEcjMuFcV+VhquTLS4OZBZa5PFCd9zq3TsWgIFmDn5JtOuoks7ruOvUayHISe6i+q9azLzzHhUuyBAR4+D+nlOWyP+MeDAJHpsNBakS75V3i6JHP+P8fW5y+EdZuiYcLwW+WKd+BXZhjlKjSop4fQoMtCc5ir/4BYZwhcCnh73OpBenQnZ//8ywIullo9VLa5nA0uB/7O6Ib94Wm4DNauQ+jRg9Vs3+kyiDpmmSrkM/IHeEyTNZtxLpkhSbSCd45Zn5qiUAxgZGUzcaQFYo7asJSyrcg/KBMA0qvxm+qayuY2u+Fa2QHekcoloZdN9dR3sABBnd8wY/OauDFdgZlv0ENIszfIRjicjxw1A5/yNfEEtsyWkwBntNOU/dMsNZ/Fy1odBu5lkQU96lIni0RM92jMW96RLFmt/3M9AzPrSA== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR12MB4849.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(86362001)(55236004)(38350700002)(26005)(6666004)(508600001)(2616005)(8936002)(6916009)(8676002)(36756003)(38100700002)(1076003)(316002)(186003)(66946007)(83380400001)(6512007)(52116002)(5660300002)(4326008)(66476007)(6486002)(6506007)(54906003)(107886003)(66556008)(2906002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: MbXFb+jwEZqytUw+qXZQzXKAnia6bNopNJXsTW+5wfzal370Pn0WZ6Al09YFTIrd68RI7cVNy8WThSZ+D0fepPtJGtD82MicTknoBHHEYm4MFLwsW5iZwEH1rPjHTKpUOwc0/Jwkti5Q/6Ii66oFWdSWoa1piRkmmgyxD7H91FbilHydghsu/fWbxY/KoNRaHWGIquhW2vCjPeaOCT6y7WG2rpsdRL62Z7WcgoKT4Vl0yY60vBIgqPQG0uhrZJjbpYXU2qYxdwEVJEae7QhN7RnN0IyEmQ/qNvblrqFQEmLkW8I48MifPZjOmLISVeH7C64/mqo9JtmGflwSp1X79uKBadqj+skvd/W45fLbcwwuOxzUFi3DRLzrCGipWozRdL5hygeALOFTNS3l9J9CiE+5KgdjMknpz9J/Zz4ES4R3zio4eMr7wfw648t6pi5mW6SqomtSt5Nr43YCa1RR82eme/L8f0xgWHhRfY6CQctPcW1C3KXXqLOrly4JEt3511Xb9Oq3+JcIWCyFx3St+chmsUPalaHSci1vQ43bg2nmli8XtA956nk0YIwzrxeUHeLJzteIgTfTX3aseX67fFp/q49saqeI6NlprHC4oSZBXUpK2Z5VugW5jbOawF6XN3dDx/wIMMjVjDxoWvNWkHTsrsyE2X99HpQA1vt9DhxwjbT5NTPiEQdoQ/kxRKjYdcXNLqNGB91zzyZeNILiO5PdBK4TaKA4u2oH7tsj6Q3ognf6gDq7RDA5L8NYkgscghBSc8B8/X5uYUqtVMHI6sPI09E+dUL5nXSwkawfiH+EVkDnAISlMDk8vt6+9NK46hxfufCHO2zgeENRxkGGEu0KK4ZKiwYMvsQxppzStYggfPGgJR8gDjaKxrRGtyoZMLBcX4v6FSx1xwEHrSQFmQPq3KvOzlU/zmBcErty7HDYFS2592Uq/V3ZxPQ01skkiXDrIs0IwsSmTpGo38bhrZJdSy4Mq2h6/3NhY5nzATffyV67gir6TNkNCRKzg/UWLnEnukOjbgvk9J3fqxb8g4bQYpqgrBBQGtkRNGCbmv+u7mUeT3UXpsLVusKTDA9Hcjshs4LgcS/zDq4bOKeonLmM/pZYGUPZx2yBUJ80ioqZyqo0haDKB8j+fS0CWhFki/xBIlYGgy2AMIhX+7d5E3CKn5S6GedHKEkf8VuezFi8CfTm4ikCAt7C+GB8gzhV17ctAeNzuDNAuUc79w10eAgZ/kEGwFtm3uy+/BbNHCQ7WVYzU9q2Njk7EDv5G/hHS2olrrCRBKtGtOswxVyj5dbK+aNjDprRumLwL36d6W0qT8qXdWiMbHjkPgf10xuKJTCd6HaVesNRraz6+Yi6WKWSmtKzyOX3OyuHqrV3pMFWI/QN92gNolKuc9OH74Yj1LXRiCM0T1Qaps/ASPGGjdBk9LPAkjv5vmorVNa+e5iJwKCWlkUxcovMQZ3kxbGQGk7eJYWdPk1JHQhFNeDSIpt3Hlox6878CVETytWKQ+1PHiw64VtGWjUWnZrgroNKGVxe+jrT4pax9STeQaptLalqqxRkJziJCXa9Ro0eTXYIwYQXLzrKZy4tLzR/U4Q25T3k51ESOopiO6yVhgjPPhRGi+ocQiJD9XYKPgQ2XO8= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5aa47d29-879d-4f79-8981-08d9cf7e9ddf X-MS-Exchange-CrossTenant-AuthSource: BL0PR12MB4849.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Jan 2022 12:34:57.3324 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 3ZVGbNrhDidaWG5lKLz76/+qz0PSpKrTyb/ljF4lvFVYrv96Q8ACImd027KI/81t/LVdowVd8hpcXuXdVypQjg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR12MB4866 Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org The git-p4 "debug" verb is described as "A tool to debug the output of p4 -G". The verb is not documented in any detail, but implements a function which executes an arbitrary p4 command with the -G flag, which causes perforce to format all output as marshalled Python dictionary objects. The verb was implemented early in the history of git-p4, and may once have served a useful purpose to the authors in the early stages of development. However, the "debug" verb is no longer being used by the current developers (and users) of git-p4, and whatever purpose the verb previously offered is easily replaced by invoking p4 directly. This patch therefore removes the verb from git-p4. Signed-off-by: Joel Holdsworth --- git-p4.py | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/git-p4.py b/git-p4.py index 2b4500226a..b7ed8e41ff 100755 --- a/git-p4.py +++ b/git-p4.py @@ -1532,21 +1532,6 @@ def loadUserMapFromCache(self): except IOError: self.getUserMapFromPerforceServer() -class P4Debug(Command): - def __init__(self): - Command.__init__(self) - self.options = [] - self.description = "A tool to debug the output of p4 -G." - self.needsGit = False - - def run(self, args): - j = 0 - for output in p4CmdList(args): - print('Element: %d' % j) - j += 1 - print(output) - return True - class P4RollBack(Command): def __init__(self): Command.__init__(self) @@ -4363,7 +4348,6 @@ def printUsage(commands): print("") commands = { - "debug" : P4Debug, "submit" : P4Submit, "commit" : P4Submit, "sync" : P4Sync, From patchwork Tue Jan 4 12:34:31 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joel Holdsworth X-Patchwork-Id: 12703419 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DF98BC433F5 for ; Tue, 4 Jan 2022 12:35:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233013AbiADMfC (ORCPT ); Tue, 4 Jan 2022 07:35:02 -0500 Received: from mail-bn8nam12on2060.outbound.protection.outlook.com ([40.107.237.60]:38528 "EHLO NAM12-BN8-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S233008AbiADMfA (ORCPT ); Tue, 4 Jan 2022 07:35:00 -0500 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=nHl7xtnFJC6jAEL2z9U5ReALWZobjZ1y9U4KSNZS2hQV5+vbJZ+PBKvttA2TDnfxSLoHpdvXCQoeJmuAOJI7JiJJ2SyizCD3nLrnmTh7qcpE8jcOglNyQWPXe8EI9jfUQiGnNbtH8pi9f5db4nKxtZQ7iTdj8lRn0NTJPQDjcwrkg4KGx1Lg4gp73JGSJFp1Q43nEKt5PBPPWfEn8tH2XJ8bGaaPyL/q5IQ6Xmrb0oC3vYxAcI+y+OxVvblYJu/NhtFE+h/thES2xSlcWfMwqDRzXtwswqaTpueFCyvXEW/tt4cmZVoiD063iWAymsGr9pB0rVph40xj9wtparlR6A== 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:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=MMRgCVUxSYr4FxEekHbbXBu5e5eF1dO2x28eBVe479A=; b=jUamfXlT2P+vatT1a+kx8BQSAir8Xz+lggwXWOt0PRymnT+T6v9A/GWEetPTO/HrrV5sZbucdlJow6xrLHmbtcBV0KLP+vas1/hV7raJ+QGwe5pkFYk9CNVPPW4m/hkJiDGGOPmvJV92pDcQF8Dms7jFYTncXhFluxg2YNHS+7ob0v+JxuKp3NPMVm/8MpmFB5aEpHaKqwne+KQ8iM5qeWffAT5ZlF2lGZ92TGnpHsFhiFcsMZ3ZT/IBL+8syEABclYlT1vo9gMzqmtsrzAZPwZG0tp49hRCFi2hmK/gWAoYXT5CK8jyAGgCAuCfEizLpKK/5YL81MKTinrDA5083g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=MMRgCVUxSYr4FxEekHbbXBu5e5eF1dO2x28eBVe479A=; b=rJd5htYLhfo/6ZmAzoifNPD+I2PdZ4IwdcOiWAe2N72o6Ck728GIDCZPLdTSSJphThTJncXHYQgf4/fC7qTrhNUZCW+V0AN+OnBNO+PleMQuCcgnbydShriMmQI99LlbeN2nG76bdwj2d2wZZQE527EW5mIRitEQD0v7Z21vHegiczp/yY0sEdAzV5+DDSmBAdkHJU2dr6Ca3jnd3bxGmpzwOFPolhsljVAWnT8TJvVo8IzcKbKHxnO/y6sOpUzU4kqW4WJoUwrrDrVzU5tkzQVmH8BBMMHxZ1zpvG9mhxPDpLjpCYh28gjOiFJrgNtYzjFyHWbeQxh9wTiUXfeVsA== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from BL0PR12MB4849.namprd12.prod.outlook.com (2603:10b6:208:1c2::17) by BL0PR12MB4866.namprd12.prod.outlook.com (2603:10b6:208:1cf::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4844.15; Tue, 4 Jan 2022 12:34:59 +0000 Received: from BL0PR12MB4849.namprd12.prod.outlook.com ([fe80::516f:3a1f:4951:ebd6]) by BL0PR12MB4849.namprd12.prod.outlook.com ([fe80::516f:3a1f:4951:ebd6%5]) with mapi id 15.20.4844.016; Tue, 4 Jan 2022 12:34:58 +0000 From: Joel Holdsworth To: git@vger.kernel.org Cc: Tzadik Vanderhoof , Dorgon Chang , Joachim Kuebart , Daniel Levin , Luke Diamand , Ben Keene , Andrew Oakley , Joel Holdsworth Subject: [PATCH v3 RESEND 2/2] git-p4: remove "rollback" verb Date: Tue, 4 Jan 2022 12:34:31 +0000 Message-Id: <20220104123431.1710-3-jholdsworth@nvidia.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220104123431.1710-1-jholdsworth@nvidia.com> References: <20220104123431.1710-1-jholdsworth@nvidia.com> X-ClientProxiedBy: AS8P250CA0010.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:330::15) To BL0PR12MB4849.namprd12.prod.outlook.com (2603:10b6:208:1c2::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: dee13228-13d1-4167-3ed3-08d9cf7e9ed5 X-MS-TrafficTypeDiagnostic: BL0PR12MB4866:EE_ X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:1079; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: nxeiRWeKCiKeqIc292XeEJpmXwxuPzH+uRmVRbevcrmkLD93sGtwFfPHonjmGSX5wjGPe25U4iBcEQG433BhPfMzQwUF+otA2wo14f+dqZsguyEwzTjn0W7lsaJqTxdl5DBA/Vu2GqenUHRYTXWmeECPNBuBtIgUgQlSK5AgWGvmBrW/1rsqFY9bzcrkNzsGEJCzhPpWmqvIOygWfY/M5eL2NRFlANVvVFoBMkmUdrLzGVOviXWTDtHZN/tK+1jfNVQ7d6deOHcJeBloCk+IpA/N11wJmSwvVmwQgfthrn+4Y2kpXBIEGFQqnFTHk35Oj/Ajy3gt0ICVxoiD5MqZCjwiMHIPOr4rH2e34kwJgFafsaKf5+Xycpj5qBUFVyeUKIxj57bgux0WNxoOc/jUk5Uct7Abbdof1Zkhrh9wCKovFF+6NBLfQ/W7EDFE0dO4U4lQJdKXGZs4+wBYxwbA8C/dL6SBBiw/myg+6gz0LBfFmxL0VugyWoqBF/knKFoQrdOqYhsiwAgrGsi//BuePzfwa9W/xOTCzKRodWHnjy5kbUZM4mHfZ9mVv5cyNF6kMtZF8F3y8POi0pmt6U2rPZfbtXUaDDG/RhEfoSyBJvEG4aJHkcWCjEwJxlihZIDHArzVcpoj8G5pHx6w/lPBG1R2TopwC5EsRvzhozd1X75B3YvyJKiY08mC7tz/pgtFFNDrDDVWnZ9rasVkxAYyQA== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR12MB4849.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(86362001)(55236004)(38350700002)(26005)(6666004)(508600001)(2616005)(8936002)(6916009)(8676002)(36756003)(38100700002)(1076003)(316002)(186003)(66946007)(83380400001)(6512007)(52116002)(5660300002)(4326008)(66476007)(6486002)(6506007)(54906003)(107886003)(66556008)(2906002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: p/9/+K0QnLUxR+yp6lVmMdJEgiZ4eItGXba6ZBbH7UFRY4PQw/E58x+LHxjcORWJKo7bRH4G7rtGmTXt3+oZ3GOWBprn3q/bIU7/hJPu5r/9CxhsCJVTm1W2xSx2UWaJ2a+TCMVpiMJEe+syL8X4a3gcEB5r0jEMCgQrkFIp5ga+peB6vgsPZ148NTyyc4huLNv7RHvi88rEXvdlMdP66hIXeAxOqV9lYe2rZjK3OZsvZWjKcIS3odM0FeCb5Y9xFHVeEEAApxpxHdjtIVUZDOg3Z+NUrK1s25yDBbG9yKr8JNJhtKH8VG3h7ZVG7NjkCdO0t1rORq4lVbNSgcNwz9PqSVh9JqoPsqaeMzebbOYxOLjXl7qWapRKjS7u0v5aUzRGMEPjKtPKzgwDi8RM6eQgMIfffpSC1CWV4HYeJ72xPCegLmicbtonrrTWXKFJcWWTHlxLSoh0+wYTOWBpe8h7LT1Eur3E+njhn9LCr6PaTN+DfFn1XwVN9QpiQ9LOb3MSdWREIBUycIvXXISo/DO/vjRCMOZws3mfeMpguGgcNlktwkFTwcQdR9JZ3Ha63PsU3ggoS149KhX/hmtQGkCMQ+RCrjXF3g/vbRLZXN6PPJ0ilSxciuU4mXczNoCqSa+LJgGjSxBAWlHmDHh7QxOzEtMgV3rqw4S48vIONm5/Jc5Uf0buOkKKhM5XnXoOsEmNSJu1ODFTse8lDe9o1mzn1y/QBLm5Oo18X7zrA4w1WP6lxKEh21MIiKH58ZUtYzcDDxIdXl4TEBYncGBo+Xo6k7NEOrs5spQQvOqnbfDZXq/6rvO4z1PG6Wkx4eyIzisRuFExJTLphrzQD0GFFJiu+mjULVag4JXNlo7u+H8FGm4nP4dXXODF5G/axFERazkAs3DRKKdfXtYkv8adfwbCFeh+YsVqFWEl6k9eLmi3wH4ajRMc60rSLH+vRmR7VbTqSEFHrfd9CdYdJUHWEuuX54Y0FHBLbbahsufMEaew0FfdZEOBQ/6rzNdWASh0LG6M5bDfjVGOl6tkkdM2Bx0KSo6YRAiorGd/ohS87WaKLta3/fjWiSy5JTsuJQHHLTTg3y4uQa5+FtmVx2vxZTPHGnosN5rRsMlPTkye3IyKT1RsOpOtQ6KWUWsdg6F9v1388dCQivNcuOGIa3mL1H5Ii2mvlQq9y7RwNcsowJedBCmPOnzLaW9E7ii1I/bsHa3xQdVa/FAxmYz/0daja69wefSDnJlLs6X0QKCsSIaXRh31yV0BHo7I8miFk8G/PoXEoglHOzm7rH6Y+nfv4GFiMVD/wKWda1V3X7s+BXRIP+CStiY4dIkK2UkAnTh8sHs+OV8b7GIhEMI8MXnYNxkcHA++iq+jnsna0fmmv7um5GLz8RaaG4AZ7iEprfWvFJYIQs9CkREfmvVJHWbg8q9OR90vf1yaILXjLSGKqBsjSRSF8FHe51aN8uc0G5/F8EKNJOUuj+U8vTrMqg65oBG5U0yazLb1J26TsKMtWXz0ibf6XthOTOQKsP1siHOeuPaKe47Pc4mlrRd8i27y7mm6cXhFS9VAyrqrZbxoDT0+wsbOqRFDA/O2dCk9GFBLFIFxrJWBdQS3fqNjlQ13d1iBpFNUl5FNmJwGR6XmLn8= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: dee13228-13d1-4167-3ed3-08d9cf7e9ed5 X-MS-Exchange-CrossTenant-AuthSource: BL0PR12MB4849.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Jan 2022 12:34:58.9271 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: WZ5yrvbh5CsdroUsvELgeI5frR4nCd/f0Dl7arEeHEq8HSg6vTYQASjXeq+jxVkyTO0GSpH3IuBwr1DWYnRjoA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR12MB4866 Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org The "rollback" verb implements a simple algorithm which takes the set of remote perforce tracker branches, or optionally, the complete collection of local branches in a git repository, and deletes commits from these branches until there are no commits left with a perforce change number greater than than a user-specified change number. If the base of a git branch has a newer change number than the user-specified maximum, then the branch is deleted. In future, there might be an argument for the addition of some kind of "reset this branch back to a given perforce change number" verb for git-p4. However, in its current form it is unlikely to be useful to users for the following reasons: * The verb is completely undocumented. The only description provided contains the following text: "A tool to debug the multi-branch import. Don't use :)". * The verb has a very narrow purpose in that it applies the rollback operation to fixed sets of branches - either all remote p4 branches, or all local branches. There is no way for users to specify branches with more granularity, for example, allowing users to specify a single branch or a set of branches. The utility of the current implementation is therefore a niche within a niche. Given these shortcomings, this patch removes the verb from git-p4. Signed-off-by: Joel Holdsworth --- git-p4.py | 60 ------------------------------------------------------- 1 file changed, 60 deletions(-) diff --git a/git-p4.py b/git-p4.py index b7ed8e41ff..a7cb321f75 100755 --- a/git-p4.py +++ b/git-p4.py @@ -1532,65 +1532,6 @@ def loadUserMapFromCache(self): except IOError: self.getUserMapFromPerforceServer() -class P4RollBack(Command): - def __init__(self): - Command.__init__(self) - self.options = [ - optparse.make_option("--local", dest="rollbackLocalBranches", action="store_true") - ] - self.description = "A tool to debug the multi-branch import. Don't use :)" - self.rollbackLocalBranches = False - - def run(self, args): - if len(args) != 1: - return False - maxChange = int(args[0]) - - if "p4ExitCode" in p4Cmd("changes -m 1"): - die("Problems executing p4"); - - if self.rollbackLocalBranches: - refPrefix = "refs/heads/" - lines = read_pipe_lines("git rev-parse --symbolic --branches") - else: - refPrefix = "refs/remotes/" - lines = read_pipe_lines("git rev-parse --symbolic --remotes") - - for line in lines: - if self.rollbackLocalBranches or (line.startswith("p4/") and line != "p4/HEAD\n"): - line = line.strip() - ref = refPrefix + line - log = extractLogMessageFromGitCommit(ref) - settings = extractSettingsGitLog(log) - - depotPaths = settings['depot-paths'] - change = settings['change'] - - changed = False - - if len(p4Cmd("changes -m 1 " + ' '.join (['%s...@%s' % (p, maxChange) - for p in depotPaths]))) == 0: - print("Branch %s did not exist at change %s, deleting." % (ref, maxChange)) - system("git update-ref -d %s `git rev-parse %s`" % (ref, ref)) - continue - - while change and int(change) > maxChange: - changed = True - if self.verbose: - print("%s is at %s ; rewinding towards %s" % (ref, change, maxChange)) - system("git update-ref %s \"%s^\"" % (ref, ref)) - log = extractLogMessageFromGitCommit(ref) - settings = extractSettingsGitLog(log) - - - depotPaths = settings['depot-paths'] - change = settings['change'] - - if changed: - print("%s rewound to %s" % (ref, change)) - - return True - class P4Submit(Command, P4UserMap): conflict_behavior_choices = ("ask", "skip", "quit") @@ -4353,7 +4294,6 @@ def printUsage(commands): "sync" : P4Sync, "rebase" : P4Rebase, "clone" : P4Clone, - "rollback" : P4RollBack, "branches" : P4Branches, "unshelve" : P4Unshelve, }