From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Fri, 28 Jan 2022 10:56:45 +0100 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1nDNzp-00FqAL-DF for lore@lore.pengutronix.de; Fri, 28 Jan 2022 10:56:45 +0100 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1nDNzo-0006Wj-Lr; Fri, 28 Jan 2022 10:56:44 +0100 Received: from mail-eopbgr00040.outbound.protection.outlook.com ([40.107.0.40] helo=EUR02-AM5-obe.outbound.protection.outlook.com) by metis.ext.pengutronix.de with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nDNzW-0006WR-7J for ptxdist@pengutronix.de; Fri, 28 Jan 2022 10:56:26 +0100 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=G0/owBhkmpPjrXysXRafMfuQEcOkJP8SwliLfy5zTamSvUqtcRWszcAyhFItlU/Js2pOGhYIp6xFkyJzGC+RSPLZTjz6/mkym2QxtjJ+iNmDXsm+suQMdiwGyrN7OJtYcp8S9Gj5AcRYumck0iUzEGdpCdkliLU9EJWqdkOPVLrlAXOr5E9TwipyolwoB+4WTedK9tkdlBXbgmHRrSiu1hjJseR3xAdnyggwOGyKCSvqarsAm1RqW/w3sUEohihJ4xN2fMOkf2VoTLQkH+X1C5G2MRxC1Aa0BWbwNICvIla+LZdD9J9gWKqU9S0oshWKqfS4c15ouPPM/nrHoQpzOA== 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=Bmou1/7FWUHle2lo17uC2wgohO/Dx/ub7ioEpjDAiV4=; b=LLCB4Le8nH+MDa8Tenu98D4UutN6iQYFyYBuDW4dBGKeo+6HT9AEgwVo+ULjxlhaBj2lVYIOflKssxJ4nXAXoNVBcea7e4GSM9MLJfkVphC9Fvz+ErAIFhFGQhhpZ3ebLdjVesoEasYoc19Z1tjaHy3sU2wgEpbBM0FXYsBvhlMXm52iZB5P5LfgP9eUNK2seAUOkrTpOhzSoR5aQjUwE9dJHylw5HG9HFayWbZtLDoLWMPg07GN/dISo+AHj5yHNW7Spk8uB5k5OQs7K3+UCZrIH3W6n3T7k/fQpHbogegtlX5K0pDZVof54m7lqWFyQe37bdJdbrsGceuOCfRHCg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=t2datacom.onmicrosoft.com; s=selector1-t2datacom-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Bmou1/7FWUHle2lo17uC2wgohO/Dx/ub7ioEpjDAiV4=; b=fs2tiBRNYsFd6DQAsB8QfSdhZ1srO+VryiwpsLEX7VKUzkLj64BZA1WzvAAEAUNBfprho4UEAuHLzVskpyXHJk5w41uewyhOwSC6AeylhT0vvoDA/Qjtyj3xntH4ERL3feVpg9F3EMe7Tg2e3llmC8uKPkuiDjLo++h5Wu8hxQk= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=t2data.com; Received: from DB9P251MB0618.EURP251.PROD.OUTLOOK.COM (2603:10a6:10:334::22) by DB9P251MB0186.EURP251.PROD.OUTLOOK.COM (2603:10a6:10:2ca::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4930.19; Fri, 28 Jan 2022 09:56:24 +0000 Received: from DB9P251MB0618.EURP251.PROD.OUTLOOK.COM ([fe80::10ab:30da:9dfb:56fe]) by DB9P251MB0618.EURP251.PROD.OUTLOOK.COM ([fe80::10ab:30da:9dfb:56fe%4]) with mapi id 15.20.4930.015; Fri, 28 Jan 2022 09:56:24 +0000 Message-ID: <0c4b6abf-bf68-81bd-80b2-279bbb2fe170@t2data.com> Date: Fri, 28 Jan 2022 10:55:05 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 Content-Language: en-US To: ptxdist@pengutronix.de References: <20220125150124.6297-1-christian.melki@t2data.com> From: Christian Melki In-Reply-To: X-ClientProxiedBy: AM6P195CA0077.EURP195.PROD.OUTLOOK.COM (2603:10a6:209:86::18) To DB9P251MB0618.EURP251.PROD.OUTLOOK.COM (2603:10a6:10:334::22) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 6ee42324-f6d5-4317-80fe-08d9e244718c X-MS-TrafficTypeDiagnostic: DB9P251MB0186:EE_ X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:275; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: YEneCfj5SJYOn5280rif6KhgBZ9RxCQb8DWZoOc4tGXISbVq/hmPEQMy6bZDvmqt6DdQTwLMEdezUMvh5OgILeLH843Klt/HvL+/ou2G4AD0m97x9Re2uForwqTYfjQlHDXtam924qr9fdY/alF49Zp6KH/l2tPA2RCMB4VB/2q0ibxLEbkB/LznSXGBIPWjfB8sFKNIgn7UNJv1WcgKrmE2BpRAQ3bhq+P1rAt1X84w+CXaFSJD7CpCeT9DN01q17AXAM+cqB4naDEk7Ui32+C2UuNXBWy1fMtRVBSUEwmrWIGcYcv1kYnfSUI+tj6xIJYCHVGoye/FiZ4xHhPjaI/k2DyOOkc07EuD0fLJVKtiE+1faxv2aSmte3+HxLy8irpPTOMaGJk0QVODw9xZaGRzlM3Gv+Tva5fG23JrN3eb2Dz9w1bIO2RSLJAjDkuoKok3nTwN/Yq38LCIM2I90F10YdWtzOdrVf9J6+DbDzwKbXYjhB33aADjOgzI8J97VcNvziFhWHgOL+yochl/u/I2CdFsVHs2/jPf0TonKcBChFHXmdQsEvec+61k3bqHGcGWjGrQMTbAT8Y29LAyobRIYVD2bTjD47A0n71O0CKlTTT8s6D1X31OqQNmrHCMBOfeQL+dvaZ0IMguLEhPj/0z8k+weBGXcirAjRHzyHkSjOhfj8xwLxTXk0mTtwUZoDk20YpFO/RJvo6mo6z+8A1aVkiQ4hDxL/1uNrZnHxOFQQ9i/ZbpyVnqoCGoaMaA23p6Ltq0342Z4No8dbUyhA== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DB9P251MB0618.EURP251.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(13230001)(396003)(376002)(366004)(39830400003)(136003)(346002)(36756003)(2906002)(38350700002)(66946007)(38100700002)(66476007)(66556008)(186003)(26005)(31686004)(83380400001)(2616005)(6506007)(6666004)(6916009)(52116002)(53546011)(44832011)(5660300002)(316002)(31696002)(86362001)(508600001)(8936002)(8676002)(6486002)(6512007)(45980500001)(43740500002)(20210929001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?VDN1d01yMlYyTDh3R1hCakozUzhlZmd2N3hhR3FETmRaalFZNFVGVFBJVjJP?= =?utf-8?B?OWZzYTR4bUM3ZG9QRzhTUktmMGNzSHVrQkJNREFGQk9DbzF2ZzExcU8xSHJt?= =?utf-8?B?dHZjTEw0OTBUMlFrK0ZkdVFlVEhaaGZONlJqZmhIbE1xYVdUazhvWFlsekp6?= =?utf-8?B?S1FESm04TkhQNVdsZlVjVU03U1BNVnhkMHo1ZUl1Y0dIaE5sZTBiN25pa09q?= =?utf-8?B?L0RlYVhXWUxzVFRlWHJOYWNxTkJpNEVlUnJ4alZQajJmbjk4bTFaWHk3R2xt?= =?utf-8?B?SWFaNnhITG9BNU12S2ZyemZIZ3NFT2t5SXM4Y3B2b0JwMnVvVlg1aWRWeVpl?= =?utf-8?B?UDJ6dmJmcXRNc1NRUTM2ZWYreXMyby9OWkQ2Q3VaUythVURITndva0lobmNR?= =?utf-8?B?ZW9UaTN3OXFrNzJVVGhTcVhBRlZ2Nm85L2djRmo3d25oNnZNbnBQRDdXc0dJ?= =?utf-8?B?bm5yRlo3M1FVMXBiTVlTbTRKTDlVRFFST0ZKRWJXSjg2b0JYTTZLalA2Nlph?= =?utf-8?B?ZmtJcU1ScUNTczFPZUI2SkNucEZ0d0VpYzZDdEhmT3hYSVJqQTVuZ2h6THpM?= =?utf-8?B?eXc5Y0p4U2N1NnhRMXNzZ0p5K0paa2lYeVBtTjl4WHZ0UjkrUWhtcXlCN0Jx?= =?utf-8?B?Y29WRjZueE5KamRab01PWHE5TzYwL0NCcGVUcjJON0hXUyszR1pIdWxRWjdB?= =?utf-8?B?a2RGSjdBaVFiZEFGY2tPMzNpNG5pdGlZREYrQStzRk5UczlXQllsSDUzVzhJ?= =?utf-8?B?VWI0bXFOV2NMcTFMSVpmYnlsaStUclJnUDhsLzZVVVN0UE41SWpVVDBObGR6?= =?utf-8?B?MEttbmQrWmZNTGNIczlMdkxueHBiQTVrUTM3L2NrWlVXZ09hcFY0M0hGWWhO?= =?utf-8?B?NjhYR1FWVFQvaGRZQkplVVRtMmRNR0pER0VOdlFDSGFxUGJPaHlDdi8vcGpH?= =?utf-8?B?SWZ0V3ViNzNQVGhIMUpyNjNaV3doV29pNHJ3K1dpdWFqeEtOOXNEWDdLMm90?= =?utf-8?B?cFNvUGJJVXpIbG9KaWUxZTk2RU9xdDFXUUh3Z3dveWZMbjZNaDc3OVQzOFpJ?= =?utf-8?B?WnV2WmdvdmlRczVtaWVjS1B6ZC93QndBY092eVpzTVZOWXgwNGJQNFJKY3pS?= =?utf-8?B?OU1YeXN5ZC85dkhmcEtpOXFtelNaQW9kVHlNb0VQZzlqSHUrN3JST09lWC8v?= =?utf-8?B?aDBjdDYwSXoyeGdCKzNNTmwzeENYMTNBUERtVzJETkoxQTc3Z3ZlWGVGNUlu?= =?utf-8?B?bENScW0rbTd4QW1ITTE4RzRRdTJRZDRESURlVWxKK2lNTzlWQlRDNVFtTXJn?= =?utf-8?B?WmVuRGNLa0FFN1NzQ3p3eWNGWmZHM0xhdHR6V2kyZUpXQ3RsTktrSUM4dHZW?= =?utf-8?B?MW5jbERCL3ZoOGlwL2hEb0VwLzMrRXhab1hOcHp3Vi9uZTZRako4cHhMZjk2?= =?utf-8?B?YmZsb1p2bG5Ob3dsN3oxK0cyME5pQmtwZzQzazB5Wk9icmVUaS9TNDAwbDBN?= =?utf-8?B?VHBiVWw3RjhqSFhDK3F1SUdyQlZvaEg0Tk5BRktWTTR3Nzl5YnR2dkhSWmpq?= =?utf-8?B?d0N4WUVqbzZWdnpoZFhHQjlkckozUTZpLzZoZmdhM2VoRUVCUXI2SWlNcEdx?= =?utf-8?B?SWswUHVwS2Z3Tm9oOEhQTHMzOGcwUXFnVjRrcjdxRVA2Q2dEZXVES2oydE41?= =?utf-8?B?QXhGME5NMTJLSUhHUlhYN2FrTkF1YU5Qb3V2Sml0RTNXYUlBL0hqQm1KMHdx?= =?utf-8?B?OWJ3S1ZsR2VwQ0IwZFNndDdlNE8wWmVmcFNPMFZxc3MyQk9YWUtWdW9nMWdR?= =?utf-8?B?ek5XV3RZSDJWMCtsVDZWT1VDaVQ0Ui9iN0ZGSGd6Z1ZHcTc0aDJoSUhnQTEv?= =?utf-8?B?b0l6b1ZhTkxjc1hQY2xMdWRKV3paQWNHMExQdDVtZnNmeTRQcXlvVlI1STAv?= =?utf-8?B?ODZ3WkdSd3VkcXdoL0owRDQ3TVBtVzN2WWZhR1o4K1NQVERDZ09lZU4yRnBB?= =?utf-8?B?YnBEblMzbGxkQi9QZUhCTmt5YTRYcFNnamtSYjZ6UVZBTysxTkRqK0JRVGg2?= =?utf-8?B?WXB6dzNZb20xTDYyUGVIL2ZLUnBvTUtHVEhWdk1KVjNHYTY2SlBXaEJwd3hw?= =?utf-8?B?cDNqMTFUZ1NRTGVmZ1FoMkJLamo1RjJhVUhGMXBab2tiSVVVWjJBVGsrSmZR?= =?utf-8?B?WmtYQlJZY2VXZld0Z01VOGdzMHdFZklxZVUzSEI5KzNFSWwzYzZkV2xPN3Qx?= =?utf-8?Q?eUY9cHhxb7qXyobjz8X99vHzGxUCKl/MOzhpq04xPw=3D?= X-OriginatorOrg: t2data.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6ee42324-f6d5-4317-80fe-08d9e244718c X-MS-Exchange-CrossTenant-AuthSource: DB9P251MB0618.EURP251.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Jan 2022 09:56:24.1103 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 27928da5-aacd-4ba1-9566-c748a6863e6c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: +qQik5l3Q83y5+WxSouZHsxpRwwU/xz+O/jR7jy6LgtVM5jeiGkeT9YC605SFQj06RVw1c4Kbf7D5dEH6Z5qgslmL/qJlUnwFPukEN5oRQI= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9P251MB0186 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.ext.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-2.0 required=4.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2, SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.2 Subject: Re: [ptxdist] [Might-be-helpful-to-someone-PATCH] scripts/kconfig: (Re-)add warning about unassigned symbols. X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false On 1/28/22 10:16 AM, Michael Olbrich wrote: > On Tue, Jan 25, 2022 at 04:01:24PM +0100, Christian Melki wrote: >> Kconfig/lxdialog is inherited from the linux kernel. >> That version removed a warning regarding unassigned symbols. >> Commit f072181e6403b0fe2e2aa800a005497b748fd284, >> kconfig: drop the ""trying to assign nonexistent symbol" warning. >> >> This warning actually helps detecting symbol renames which are lost in >> translation which is helpful when doing oldconfigs etc. > > Good argument. I like it. > > When I update kconfig, then I overwrite it with the new version. Then I > apply the patches from scripts/ptx-modifications/. So any local changes > must be added there too. Can you do that? Just add exactly this patch there > and add it to the series file. Then amend the commit or create a second > one. Either way works for me. > > Michael > Will do. Btw. This obviously applies to other projects using kconfig/lxdialog. Kernel, busybox etc. I've always found that the developers do not care for unassigned symbols in defconfigs etc. I am currently trying to push this back into the kernel as it has approx 1k unassigned symbols over arch defconfigs (ARM has around 450). Even really, really old symbols. But for the embedded developer for a project, coming from an old version and upgrading kernel, busybox, whatever with oldconfig, a lot of symbols are renamed over time and functionality "lost" without as much as a notice. Sometimes that leads to time waste because you did not notice the symbol name change. Which has always bugged me. Ever since that was deleted in 2008. So probably helpful for those pesky Busybox updates too (tnx) :) >> Signed-off-by: Christian Melki >> --- >> scripts/kconfig/confdata.c | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c >> index c5f4b552b..09777fdfe 100644 >> --- a/scripts/kconfig/confdata.c >> +++ b/scripts/kconfig/confdata.c >> @@ -426,6 +426,7 @@ load: >> if (def == S_DEF_USER) { >> sym = sym_find(line + 2 + strlen(CONFIG_)); >> if (!sym) { >> + conf_warning("trying to assign nonexistent symbol %s", line + 2 + strlen(CONFIG_)); >> sym_add_change_count(1); >> continue; >> } >> @@ -468,8 +469,10 @@ load: >> * include/config/foo.h must be touched. >> */ >> conf_touch_dep(line + strlen(CONFIG_)); >> - else >> + else { >> + conf_warning("trying to assign nonexistent symbol %s", line + strlen(CONFIG_)); >> sym_add_change_count(1); >> + } >> continue; >> } >> >> -- >> 2.30.2 >> >> >> _______________________________________________ >> ptxdist mailing list >> ptxdist@pengutronix.de >> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de >> > _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de