From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Fri, 04 Mar 2022 16:19:01 +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 1nQ9ht-0005MV-OU for lore@lore.pengutronix.de; Fri, 04 Mar 2022 16:19:01 +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 1nQ9hs-0005lQ-SP; Fri, 04 Mar 2022 16:19:00 +0100 Received: from mail-eopbgr30089.outbound.protection.outlook.com ([40.107.3.89] helo=EUR03-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 1nQ9h4-0005lC-Ct; Fri, 04 Mar 2022 16:18:11 +0100 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=di0H+8K6nPSlA7EWdk/VcAS7oaAr1AebGMKoiPsTDyt26RMjDsCtFkcxGaiJ6+EyZnEHQdnXVVumqSduzZlyXahNBvGIzrfms3GNlZ8aG0AtYVwDTnPdjPu+nYQktItQcJKdd7DssoSrexpQoE8kV0oVhkvIiPl4TYuhkPOPXqiqBiKqfo07mxHWarlqrqkiYvD97fRooWxxIGYsKMSVlU9iGSGGsm8MANfDET7x9OrwdghV8MOtTBMs/BLMKCIDHAcPAbpiDMGT35IddWQWxSRNFp1XpYG8eVI1U5Cki9ZjbpFTkt0fNWmMUY/iWZNkFxzu8KxLaWjBLliP65X80g== 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=EFi6nqez67QX6rYVYP9wYfDde4iFlUlvr9g4rOCy1lY=; b=jAfjhlQHHxPt1thgbBPeTDXC0JQUgx0cJfbH0TE/fIQQrxa1Bdkf6soPek55qjIAXL1gJF9HzF6YAeXKeJcIz3KUOgOgGSD9X2Im2EhSKurHcFI/xlWJlCo9S+3yV7wYCiZICN/xNRJotOWdZ4QpNC//IYjMjSdlw5E4OQYyXgIVnDaPL47SCoccp+JLiUTVRh2FPni84XupWmvC3v/U7KygL4TCh06EvxZcZ1yVswPrRtZmLsU+SamxBtS8iPGvh6gxsg0xAPI+0HhiBhyVY0N3Tnc2OfnhNEDt+pXCJDGsPPo48Ut+boAaUcEMHfVj+Gsbt4IuCTu5nQKmxiyZlA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=t2data.com; dmarc=pass action=none header.from=t2data.com; dkim=pass header.d=t2data.com; 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=EFi6nqez67QX6rYVYP9wYfDde4iFlUlvr9g4rOCy1lY=; b=gJIig8jTAMMTqPRjxW5x+9cSNCE+Pg/gsx7VVcAqRaM7t5i3IvXHd47m3A5q/RMZw7Sd2wCKB96IDohEAH05nnRfI9V7HiIJIzYVxmmWhuTMPeXL3/wpCPGrVium4h/9ML/9F5xZfgbQ3GC90O3slnZSMPvO4c2dWgGgC735SPU= 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 PAXP251MB0423.EURP251.PROD.OUTLOOK.COM (2603:10a6:102:288::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5038.14; Fri, 4 Mar 2022 15:18:08 +0000 Received: from DB9P251MB0618.EURP251.PROD.OUTLOOK.COM ([fe80::1cba:3835:dd1e:63d3]) by DB9P251MB0618.EURP251.PROD.OUTLOOK.COM ([fe80::1cba:3835:dd1e:63d3%6]) with mapi id 15.20.5038.014; Fri, 4 Mar 2022 15:18:08 +0000 Message-ID: <1640a3d2-3882-2b3f-e926-155a94099776@t2data.com> Date: Fri, 4 Mar 2022 16:16:23 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.6.0 Content-Language: en-US To: Michael Olbrich References: <20220303134028.GN12709@pengutronix.de> <7ee003fb-e6c3-34d7-d9e2-d29279369b0c@t2data.com> <9e421142-d037-6b62-79c4-2a7a9a09d120@benfm.de> <00ae8995-135e-dc67-737e-4c7d6ec3e08a@t2data.com> <20220303201138.GB26895@falbala.internal.home.lespocky.de> <20220303203533.GC26895@falbala.internal.home.lespocky.de> From: Christian Melki In-Reply-To: X-ClientProxiedBy: GV3P280CA0014.SWEP280.PROD.OUTLOOK.COM (2603:10a6:150:b::29) 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: 633dbd21-2a6e-46e4-9651-08d9fdf22ffa X-MS-TrafficTypeDiagnostic: PAXP251MB0423:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: NOYLyqKp3YDkyjihD5fgWC2XVZovkRM4O+U7ABNxOfjQ0QdgmGV44DNFX2Vra1G/4Pbb48gQjwywCZ6ytYl4nvhYcieys9xVxNkjKjUMQA9K8rYlNGXuqYpKefsARTqImIjbUzxaKCLojOjQs+94c0BMiaXwJsGjEIn5ilsHco+UotX+c6ZvLO1kNtkKBm/493RYm4TH4HHgiKokOr367eJ5mj4nhvik/d8SDJPIKoWDwXImWARgYsoDZVwIEBQywnIc99V27eOiDq5MWOpZjcakwi7lI43HPWSrVEB6DtKHLKLoUcbIZ2Zl6QQ1BUzxjpidWZi5Q0LDQCn+wSl6YREAScvsfByBuL6ZEx++rq4RLOm5e5t0dAw1ADQrzYP8FTxD4AX+SHiErno2Bf0s6XDB2oR2PDONiQmelUcHyQwlRWBzfIglYvsFyR0Bbdif08LK0X5Jbb4e1TWpdqb4EXrj25B00vP0vVnxQnLq63JaS5IIh6jdR8za9Uc1X2Q0FlD/A7Ya1/qoyxm+RBJMVzx7wWkz2agemUpBC5EKmzj9n099i42DwPVdVbH1YybMmWe0tyyQuyeF0Yir+Yk32ruxixsv3EzNqAZ/V743jTm7//DtKu5p8gF9z44Qakscqk7WDZ7jx02cvWWWGIESIJ/xXTZ7MXEiGUTJ7uIVWSmHRGwZgy9N6hlvCwtqJFe/OT1rW+KT+C4cjs6LrXhTs7lTIHilrFzbcACaKTvNSkvFbsbo9AJfpLArf9QFyrTBGOaYml//tSifwV8QdsGENPDmTc4ycprbBcF6G+oQiMs= 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)(136003)(396003)(366004)(346002)(39830400003)(376002)(4326008)(6916009)(508600001)(5660300002)(2906002)(53546011)(52116002)(6666004)(6512007)(6506007)(8676002)(38100700002)(38350700002)(31696002)(66476007)(66556008)(66946007)(86362001)(6486002)(450100002)(44832011)(186003)(36756003)(8936002)(26005)(2616005)(316002)(83380400001)(31686004)(21314003)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?R09hNEFkWTNZSmhYQis0bVhZZkp2bzlyTTAyTGZTZS96U3gyZHVHUHNkU05G?= =?utf-8?B?UXE0YmdPWmlSSUJZNTBLTDF4bkxmVC9jMXdNYXYyaFAwMzY4c1dCY3VrbTFj?= =?utf-8?B?QmNyYU5LTXB3VStkNmsxNHd5dFFWT1I3elhhOTN4dnpuTlY0MnJrT2JFU2Ja?= =?utf-8?B?Q2h5bWN5dTBEOFJSSkowMVNYdGZhRXVINFBCMnpYQzNkMWdJTFVoL21WTDBh?= =?utf-8?B?aHFVY2NyaU5lSEcrZmdoYlBpVG42OERyU09YakwxN29lcVVKQm5zZ1l4VnE2?= =?utf-8?B?cm9zRWZ5U3REWmp2d1hybG1KY3ZvbWFFekJWOGxSQXgzRU0vNnhlQi9ScS9G?= =?utf-8?B?RVNpR2RwWnVjQ3BnK1pKUldpVlArRERYRnFMdjltdm1zNU5WR2hPdUw1d1V5?= =?utf-8?B?aFBSdjRXWlUrL216aUoreFZOcEFkUWxBUFZ2N1A5ZzRURkR1U3dLODN4bXhP?= =?utf-8?B?aWNicC9QbmluQkxHWUg1NHZFN1NmczNXVHFYMzlhd3J6T1h4Wmp5OU1ldktM?= =?utf-8?B?dEpBdXY0UzVNK2xWTC9laHpsakhRWVYzckVOTElFZ29PVlQza2U2RTJ6cUpy?= =?utf-8?B?VW0xS2tDN3ZTR1pGQ3VaRGRrWldYdXZSZXEvcHE0dFUyVXN4MU5SdkNwNklk?= =?utf-8?B?NTBLUUwzNFZ1RUVMeUk0UHViQzRWWHRqenhjMXJhRmVKSkkrY3o4RXd0KzBu?= =?utf-8?B?anhjTUVZUTNJYzlvVlpwdXB0VlZiOGt3R21BaXV6amFQSkJvang4M1dBSGVZ?= =?utf-8?B?K2d1djRLYmJpc01WSEJObmJjREg4ckRibmliTU00TVkvWG9YQksrOHRHUFpT?= =?utf-8?B?RTZObFhZSHNTckp0SzRDVXZPUUEwWk1DNDUrSjNtRDZ2QWdVbTFpY00wdkZI?= =?utf-8?B?a0gwVUxEZkNiMXJDclNxOHFRbGQvS05MTUg1bGdna2RlaCs5WjFCNE5IN05i?= =?utf-8?B?a0gwN0JqUGxoSHBOMDFybG5pdFpONTYwelRHOExBK3hURi9UcnJZek42aStO?= =?utf-8?B?M2xGVXdKNmxJT3NIQVZTUlpNQlZRU3QxcG5iejhHZUJrT3hZSTdIUGt6OElu?= =?utf-8?B?Z3lmSDRBcDQwNWNaQnZrUHJPNjhEbFlDTFFvZnQ3QlVpQmRGN1NWQnE3UnlB?= =?utf-8?B?ZUhrdXl2L1hKRmhRQldEV2NPT1g1QytnamI5TFF0QXNadmZQcUlQYzRMVlZp?= =?utf-8?B?VDd6THdlRFBmU0Q1SkJKN1podHAyNHNhY2ptbGFRZFZUUHpOc3NUQWFIT2dQ?= =?utf-8?B?eHBuMTNzbzVJMkNTWmdzU1FrNytsQzdSZVkzT0NzSXFiNlF3Nncrc2NMSHJn?= =?utf-8?B?ekZBSWQ5dnR0VDZpNllkek9pNXNBWk92Y3lZMVcwNmkxcmtHWXFWY01ZSlVv?= =?utf-8?B?Zm82aVIyU0Z0cVBoaTNxUHpIZ0U3Z2FSOXJ0b3VvUXVnandscUIzV2pGN2Zl?= =?utf-8?B?WHJ5QmNxUk1sSlVTQ01JSXFWWjY3UmNMV3FHWW5VeDZLd0x1WTkxUTlDVWsy?= =?utf-8?B?MEhSL1ZmbUQ3TUZaQVBVTEpCeFQ5eC9DdVhaeEVvQ0hxVTlCdldQdmFIQ2NZ?= =?utf-8?B?T2xqcmU1eXI5Y25wZ3dyRHFYUlJpV1p5TlBDckRYaWhVdFRHN1lMeitsSGVr?= =?utf-8?B?czZFbEx0RjQ3ckYxOUw3cnFEQkN5OFFYaUJXNzNnVkh5S21jTS9MWndhL3Vw?= =?utf-8?B?SnVHcVpnQU1PNTU4enNRMitvL3RtMHBUTUFEcjMzUDBYNGNvWE53UkVwK212?= =?utf-8?B?U1RRLzhXNFN5QUpScjNkVjByVzMvaDJmeEovRWZYSUVSSmVjd2puNVNLeG1T?= =?utf-8?B?ZkI5b1hEL1RHcFk4NGpTWjNKOUE0YUF1N0NXVElLRG4wUEIvK3IzSFR0SXJ6?= =?utf-8?B?eU9udnB3VExLUERQY3loRDNTeE9vT3ZKZlFOcnczUjJSWUFuM0ZERnVNU2kv?= =?utf-8?B?bk1VZy9pOG84ajNqTThydzVwSUV5N2pGT1ZNMGNyWUkxb0Q3djI4RDVPVmhC?= =?utf-8?B?VUVsYmN2MVlpbThGdEFPWG9VQkdSTUxJZ3dqK2dGa2tTWE05bGNBWkkydTdY?= =?utf-8?B?R3BnMXZTWS94cXNVNTZ3dmt0azJoVkl2TlAxT1p0OWFrOGZPM1liMkNhZHgx?= =?utf-8?B?eTFieGRzQ0o2NWNrSVdkRVZ0QmNMNVNJUEVIcyttMlplR0lvS21XK3lUZkpC?= =?utf-8?B?WHBRejFSUjdvendybWprTUg3dlhiMDk3ZXNWVlZ6SVZhVHJFc3BqdXM2R21x?= =?utf-8?Q?iCkc3dw5/sMJYT8WRlqYMoHvxK4ml9AUFtWD4X/nWQ=3D?= X-OriginatorOrg: t2data.com X-MS-Exchange-CrossTenant-Network-Message-Id: 633dbd21-2a6e-46e4-9651-08d9fdf22ffa X-MS-Exchange-CrossTenant-AuthSource: DB9P251MB0618.EURP251.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Mar 2022 15:18:08.0387 (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: AZ1c/LxcyIRPJCFe6V4YAvvNH0Xm5ci6pkaaeG2rkMZfRYFul5uook1tmpGVeQKRzv+OgjUTvl7SdnLnzOIgahtc0c++yuChjjhH5ItpQS4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXP251MB0423 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.1 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] Patch series for package per project? 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 Cc: 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 3/4/22 9:10 AM, Michael Olbrich wrote: > Hi, > > On Thu, Mar 03, 2022 at 09:35:36PM +0100, Alexander Dahl wrote: >> On Thu, Mar 03, 2022 at 09:11:39PM +0100, Alexander Dahl wrote: >>> On Thu, Mar 03, 2022 at 08:37:45PM +0100, Christian Melki wrote: >>>> On 3/3/22 18:07, Felix Mellmann wrote: >>>>> On 03.03.22 15:22, Christian Melki wrote: >>>>>> Just one barebox version, but for different archs, with different >>>>>> patchsets. I'd like the patches to live in the same place as they've >>>>>> always done, i.e. under patches/package/... >>>>>> But patches/package/series.$platform or similar does not seem to work? >>>>>> >>>>> Would it be an option to put the patches below each of the platformdirs? >>>>> >>>>> I ran into a similiar situation when starting to migrate to a different >>>>> CPU platform. After a couple of years putting serveral "platformconfig" >>>>> files within the same directory as "ptxconfig" I'm now using separate >>>>> subdirs for each platform where separate sets of platform dependent >>>>> patches are lying. > > That would be my first suggestion as well. > Hm. My ptxdist setup seems to point to the bin/patches before projects/projectname/patches directory if i print the PATH_PATCHES variable. Default afaik? And it doesn't continue if patches are available in some other directory. So if I have generic patches in bin/patches, or if I want them overridden it won't work? I must have missed something? How is this supposed to work? >>>> Ideally not. It irks me a bit that patches have to go to a different >>>> place because they belong to another platform than the "default". Or >>>> that they have to go somewhere else altogether because they need to be >>>> split into platforms. >>>> It doesn't make much sense to me. > > I guess that's a matter of perspective. For me, either the patch stack is > shared. Then I out it in the common directory. Or each platform has a > completely separate patch stack. Then it feels natural to have them in > separate locations in each platform. > >>> If I understand you correctly, your folder patches/barebox-2022.03.0 >>> (or whatever common version you use) must look like this somehow, all >>> patches of the different series interleaved? >>> >>> 0001-apples.patch >>> 0001-foo.patch >>> 0001-this.patch >>> 0002-bar.patch >>> 0002-oranges.patch >>> 0002-that.patch >>> 0003-baz.patch >>> series.one >>> series.three >>> series.two >> >> Now, what if those series are not mutually exclusive, but certain >> patches are part of multiple series? Even worse: in different places, >> like first patch in one, and third patch in the other series? Each >> call of `git ptx-patches` would change the file numbers in the file >> names, breaking the other series? > > Exactly. Maintaining shared patches is a mess. When I have more patches and > especially if they are shared, then I work with a clone of the package > upstream git. I use 'ptxdist local-src' for development and then export a > full patch stack into the BSP. > If the patch stacks for multiple platforms share some patches then they are > in a shared branch in git, but in the BSP each platform has its own copy. > > >>> Seriously, that looks messy to me, how should anyone looking at the >>> directory know which patch belongs to which series without looking at >>> the series files? What if someone wants to copy/move all patches >>> belonging to only one series to a different place? >> >> What I would do, and in fact I did this with patches for U-Boot for >> five different boards: throw all patches into one series. Why >> separate them? Or if that's not possible, put distinct patch series >> configs/platform-XXX/patches/barebox-2022.03.0 as suggested before. > > I personally don't like having unrelated and unnecessary patches in my > patchstack... > >> Well, there's one more possibility: put the patch series to different >> top layers. ;-) > > You could also put something like this in platforms/barebox-series.in: > > config BAREBOX_SERIES > string > default "series${PTXDIST_PLATFORMSUFFIX}" > > This will be used automatically and you have a series for each platform. > Or add a prompt and explicitly set the series filename. > > Michael > _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de