From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Tue, 30 Aug 2022 10:00:15 +0200 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 1oSwAR-001V0o-Nr for lore@lore.pengutronix.de; Tue, 30 Aug 2022 10:00:15 +0200 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 1oSwAQ-0007fY-Jy; Tue, 30 Aug 2022 10:00:14 +0200 Received: from mail-eopbgr10055.outbound.protection.outlook.com ([40.107.1.55] helo=EUR02-HE1-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 1oSwA3-0007ek-JC; Tue, 30 Aug 2022 09:59:52 +0200 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=UWF3TfO+4xH1wnNzyQSBl4EVVhdGEo0/I+QDktGmKGEGCB2KoeoiWMnzUfnGwjF5KMswWMy6kAhe8Q8NXnMF/vuo4WHdHuzy1AOlJp4hb8trBM1SkJLgAVgfjv2L9JNfvHDr9lciPIMS6D6cpJu6utYLsfe5KYzgW7KzM6e3EsxSCERh6keCZqc59+y2h3S9aTs+vlSjumU9xJYvyumDrjvoa/fVn1b8F1+F0JzsXSfsy2+QAnaT0cAz16jjxgVN3yDqRprD5Hh0PD0Tv9pofN4cCbxckVO61dokTB+Fqvlz19pq9+WQwhGptFvgmzX8nBSfB4Kw2xqyAlSc9eLgpA== 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=Aqa/GaTiMcl3jKEfKsEDHKLr7Oq1gdaMFwSYy2ZAXM0=; b=WbrwUJuykAZWVtWkqPYprRQEW/yET8+1h8aw0CW0+yM/VL9X2fcAws18kme/dULYI0ImSc/FCjq63Ex5uIVSZrb7kzv49yJysJkm9VKIQ6ODydDFjRGFUXrO4slLnL2tcRIsxv09GjNgghHiLhzSkTbl9BdAQNRJ9Q57i2AqVtWs35OVIbhqqghtlNKpPlUzWarihLAZu4wUgY7DcTADqhw39a1Be3hPn6Kx2RTpusr9LcrVtBjW+pmwPVUqpThk9RMOMGxpO1ftXONwyeqGRxFxK38J5xwZMOW8lE2n1hGIYm//iRvjd+KpTGunoRPqh2Vig4bOTCXXLqyOssTXXA== 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=Aqa/GaTiMcl3jKEfKsEDHKLr7Oq1gdaMFwSYy2ZAXM0=; b=RnIu+jdXR9nMT8f5d0jUqAbB0OURgFmw5jsVZq+WHLx5zSMiZaBPZL9ImSF2LcWpQTsV9voO0FBqTAxZoSkbSnFZrsqbeMPtRfjPvesqVUeXmwXTAZkS/CiNF0I5aDvxeUTiERLtDiRI0NY74pi/3E3oMjybydJMuTcl22ujDzs= 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 AS8P251MB0935.EURP251.PROD.OUTLOOK.COM (2603:10a6:20b:570::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5566.21; Tue, 30 Aug 2022 07:59:49 +0000 Received: from DB9P251MB0618.EURP251.PROD.OUTLOOK.COM ([fe80::a130:b2a7:9ffa:f1b6]) by DB9P251MB0618.EURP251.PROD.OUTLOOK.COM ([fe80::a130:b2a7:9ffa:f1b6%7]) with mapi id 15.20.5566.021; Tue, 30 Aug 2022 07:59:49 +0000 Message-ID: <50128bcc-2dec-4dc4-7fde-10d1d969cc81@t2data.com> Date: Tue, 30 Aug 2022 09:58:03 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.1.2 Content-Language: en-US To: Michael Olbrich References: <20220826133537.1843791-1-christian.melki@t2data.com> <2026849.Kn9BfFgYnX@ada> <979bfe94-2c54-bba5-a1e9-4337f863f7b0@t2data.com> From: Christian Melki In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: OL1P279CA0016.NORP279.PROD.OUTLOOK.COM (2603:10a6:e10:12::21) 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: b0b3a488-4999-4079-00ca-08da8a5d9cae X-MS-TrafficTypeDiagnostic: AS8P251MB0935:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: +FlWAKcXcGtGw3qJWAu6BzbTy+8YYQ+uriHHvcrAWDAlW3agSl4JliVyqnwrCx3czHZVAM0Q5wcr5MS1ihwPAIK+aNh4qzkz3rrSaBT4aaI5Op1MnVtZTDWffNQC6yj6xmvQQjS+1zGjjPHR33l7yFlASfgI3J+hLVsjadHAkq6HbTrii8fmq3yk0gyek+9FJFn3LD2z3dSK28xGgQpKq4TTXViWfdVo8kta5rjl9hFHeyO1+qKhBlb4fNEcUCm3B3WWKmlsnGQRR50bGqhASP+oZOU9WR54XrWWRgeCjRkYreIs6uHUGR4QxorGZuIWVog87ckb1mWrUSTzCtc1kcwuqb8BBj1rLVONGPTo2v9tbFM0gw7/ITkVRxr5ywnRCGdTf3rsvc8CThaby7R2uW8xZk0VW9quJZtjzzSYBtCWLkWc6vo4RwrS8KDv2sI5n/ufMx6zds6rdxmCflbdjB60sDe6PLApxhHV5Ys9tlsgnwNSSQuzVUTwQhFauCBzF2XMfohdq30qayMP2L/VfxxrGsdAn6M62ifRxP2VKTDIZAP04OLvU/wfVlkoKBxQ4cUg1Wbybyu58IRPtCOGq5DNZJ9gWDcyF33L2Mordb2uMWJ4jQrLL2xzFEaSTigEZf2H2n+YNK1/tniYlch3OQOanpOxCdP2YPuDc9qkZ6zatGnodUBJCwrYcsTaT/Imbyg+brGuOSsh90pUtMcRJfDoNbms3gxg64vJqto9l5fRf9dBom4uU2rIoWSD4X2bRA/DRVbp0mfl1aOsC0kb8CnlRde/BiImtXxmoZW34XDHI3Ky+6phCypmMb5xCmFs 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:(13230016)(376002)(366004)(396003)(346002)(136003)(39830400003)(66946007)(66476007)(66556008)(8676002)(6486002)(4326008)(966005)(83380400001)(53546011)(2906002)(6506007)(6666004)(8936002)(6512007)(5660300002)(26005)(478600001)(44832011)(41300700001)(31686004)(316002)(2616005)(31696002)(86362001)(6916009)(36756003)(186003)(450100002)(38100700002)(43740500002)(45980500001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SUVBcTVzbW84VzRwcDA3TEFUL21VODM1MU90Mldhd3QrTVMwcHBGeHU1U0hY?= =?utf-8?B?MkJyMGhBTFhUWUs4eXZQa2l1eDlsWmtJQVVGSjk3bTR0ays4UTliRnE5OU9z?= =?utf-8?B?NU1qZnVVaDROLzhmbTQzWFlBSHZjckgvaC9HNzBKcm5qSEdZTm1YaG16WlNR?= =?utf-8?B?V2Q2bld1YUdsL0NqT1RwVGtxcytvOUl5UGVxbjFuMWExc2FTOE94Zk42ZHF5?= =?utf-8?B?Zk1GU3JOY0hwUVoyaHFaMUlFRUN2Y1pKN1JqdGJFU0ROMU00QVJyYm5CS1Bj?= =?utf-8?B?U0hsQjFIWDMrbTN5Tlk4Q3lJSHhjc0JqZ2N1OHpKRlZCMFpGM3FqWTc5SlNZ?= =?utf-8?B?clZNSGZPR0E3cUNzNkJMK1VlYlkrTHp0aFJoR0g3N2FJTHBLVHF0VmlRaFVQ?= =?utf-8?B?MDFxQktyUWZkdnNIRGJXamt4dGdDNXlTQTk0TUhMK2Z5RDAyNndmUFI0aUtz?= =?utf-8?B?TWQ4dnJyWjNPamMrTFFVQlozbURZR05tcEVmUzh3K3hFSW9rTGwrQmpTZ01t?= =?utf-8?B?cE1NZjMwaTdhWW1Zb1NGd1FWQVdGNm1vWXNObE1HLy9DelIxcHVOdVhNYzNF?= =?utf-8?B?R2tucGM1b2lrRDhjQlV4S3NabCtKL0Rmc1pLb05zY1BWZ1oyc2pTWlNFTTlw?= =?utf-8?B?a3ZLcFBrYTVmaUZMMWxlcS91c2N3UlRlYWt3cCtGNjdVTk5kVHhmdFhOdU5J?= =?utf-8?B?czIvdS93Szd0TlNBdjJVbzRzaFUxb2tYeDI5MUNNNFFxYlVFWXdmbGZTU01a?= =?utf-8?B?MU5XVXZQaG5VODViY3JTcjZXaHBES29VVkVUQytHQS81VmxhT0tGSExTZnJk?= =?utf-8?B?QnViTmxsaXMrZE5sY3VsRTNKNCsxcHNDVm50WWZBK0lLM1I5TmRyc25HV3BF?= =?utf-8?B?QkV4L3NPRzRuOFJvYUV5ZzJaN1drQU5zeDhCNFNOTWplcVh2ZWlhOVRneW5K?= =?utf-8?B?bkVZOGFFTnNMYzd2bm5oOVVaUHdmbU5CdlkvY2Y0cUdOTGNydjlwemVQM25L?= =?utf-8?B?NnpSVnhQTUU0bmN4R1JabjZDTUY1MCtDRm9lSHMyc2tGcVBVRjRHbWp5K3Fk?= =?utf-8?B?emt2anFzcHdSa3FtVDlRenVqN1h4L2Rlcm9kSElYS1E0T3pXQitqZ004RGg0?= =?utf-8?B?eEM2R1JCYVdxZEMwYTNaTUNWUWl2TytOenlDdkszdzRPaWZDOHhlMXNyMlNE?= =?utf-8?B?Vyt1VmVVdHZKbmhBTDBHL0k4SjlVYUFweHUrMkRrSEEyRUVkbXd6OEdnc01Q?= =?utf-8?B?cE5HRWtwU0gwb1BUN05LQ0prWHRhL3ZjbENGZnNta3VwSGNRQW02VS9hM2Y4?= =?utf-8?B?ck1SaFM5TzY0bU1NRDNYbFpPL0crYXE4U2dkL0RPbXdKcGZOZkgraEV2bWpu?= =?utf-8?B?TksyNDVKVW56Mm01WUYwL2ZHR3dqYmRFZTBramVXdkYxMmh3UFZQRTlzRUtE?= =?utf-8?B?aWdQVDZZd1N2ZFVhYzAwOGZqZVgzVnY5QjdaYWpxanFhOGJiVFZBY1lINXdz?= =?utf-8?B?UFFucnVkRi9CbVhQL1NiVGxWZy9WQkxoalFvUng1ZEYrVEVJQU5hamJwSDFP?= =?utf-8?B?QmJncnl2LzhZUTFJOGFzYnJIVlhnYk5SY2VoU1c2M045NE1QTTU1M0g1TGFO?= =?utf-8?B?VFZFK2dOaDUxUnVsdFlMOVJvdXhpRWFuRkx0VWwvUk1yaGV5WmQrZkhtbWhw?= =?utf-8?B?QXhWUHIyL3J2NGhqSGs4RXI0cTI2R3RSUU83UllYdEZyMnpDOWMxSklYTTFh?= =?utf-8?B?NDlrTC9wanZ2dXNvK0lFSC8vaVo1MlpQMWkvMy9mVVVVaHo4NHE5SzV0dFZo?= =?utf-8?B?VStVVC9pQWd5OWYxS2x5enRZdFYrdGs3Y1J0dDNGcHhwUmtsSzFLUlI1dVNV?= =?utf-8?B?bU54Sjltcit3Q2J6alBJSlkwNmt6QmJQSWw0ODRaVVJXYit2b0ZwN1hiZzNq?= =?utf-8?B?dEtlc0h3NlhzUHF2ODI4Mi9CRG5kZUhKVmo2ZktURm1PQjBsbGNjTWgwamJn?= =?utf-8?B?cFpnbmtHaldwRjhmc3lrdHdTRnpRMXFHNGpZM1dqd1pkbHowOWtGR0Q1RkVQ?= =?utf-8?B?WjZIdGlYZWR1QXpBOVhJK29RMzhSY1RhN1R5UDcxMWs2VnUwdXViRkNRK2VF?= =?utf-8?B?d1JVOXBNSWVXK1hFUkJ4TGtwNjVoNGVBbERKVGhTTE9Ecmo5RDlUR3NObWtC?= =?utf-8?B?T1E9PQ==?= X-OriginatorOrg: t2data.com X-MS-Exchange-CrossTenant-Network-Message-Id: b0b3a488-4999-4079-00ca-08da8a5d9cae X-MS-Exchange-CrossTenant-AuthSource: DB9P251MB0618.EURP251.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Aug 2022 07:59:49.2837 (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: BtP+DSC/2DC6N38WBqjJiIge4EInlllAzM+Uf3PJ0nAf/V87ABRO7C2W9+8zffaz7Va0PykueYtxLeGfxDxM0rKLfkN0OfFUcHry4rsQKrw= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8P251MB0935 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,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.2 Subject: Re: [ptxdist] [PATCH 1/3] u-boot: Move u-boot URL to github. 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 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 8/30/22 9:23 AM, Michael Olbrich wrote: > On Mon, Aug 29, 2022 at 06:57:03PM +0200, Christian Melki wrote: >> On 8/29/22 10:49, Alexander Dahl wrote: >>> Hello Christian, >>> >>> Am Freitag, 26. August 2022, 15:35:35 CEST schrieb Christian Melki: >>>> * Make URL configurable. Also. This aids in pointing >>>> ptxdist to various vendor specific releases. >>>> * Make compression format .tar.gz. Default >>>> github contains such releases for really old releases, >>>> so it should be fine. >>>> * Bump default version to something more recent. >>>> Just cosmetics really. >>> >>> What's the motivation for moving this to a mirror of the official sources? >>> According to https://u-boot.readthedocs.io/en/latest/build/source.html the >>> main and official forge is https://source.denx.de/u-boot/u-boot and >>> additionally https://ftp.denx.de/pub/u-boot/ still contains all release >>> tarballs up to the most recent RCs. This is also mentioned in the README in >>> top level U-Boot source. >>> >>> Greets >>> Alex >>> >> >> The point beeing similarity between various vendor releases and vanilla >> u-boot releases. But since the URL is configurable.. whichever really. >> Github API has the advantage of being able to tell you about new >> releases, should such a tool ever be adapted to ptxdist. > > Nack. Our u-boot package is already complex enough. I don't want to support > whatever vendors do in their downstream repositories as well. Please create > a local new package for that. I would be happy to accept a patch for a > template to make this easier. > > Michael > I understand, but don't agree. Whatever board you take nowdays you'll almost always get pointers to vendor repos for some or all of the platform bits. So external repos is often the same usual suspects. kernel, bootloaders, tf-a, op-tee etc. Platform bits. 80-90% of the time, the ptxdist handling is fine, but needs another url. Not being able to point ptxdist there is a bit annoying. Duplicating complex platform handling just for url changing is also annoying. Yes. I can copy the tarball, name them as vanilla like, place them locally and add even more confusion. But I prefer being able to tell that this is not a vanilla URL. Rant mode off. :) /C >> I actually tried asking about github as an official source on irc, but >> nobody cared to answer. >> >> regards, >> C >> >>>> >>>> Signed-off-by: Christian Melki >>>> --- >>>> platforms/u-boot.in | 11 +++++++++-- >>>> rules/u-boot.make | 4 ++-- >>>> 2 files changed, 11 insertions(+), 4 deletions(-) >>>> >>>> diff --git a/platforms/u-boot.in b/platforms/u-boot.in >>>> index 769237081..8c7931d13 100644 >>>> --- a/platforms/u-boot.in >>>> +++ b/platforms/u-boot.in >>>> @@ -9,13 +9,20 @@ menuconfig U_BOOT >>>> >>>> if U_BOOT >>>> >>>> +config U_BOOT_URL >>>> + string >>>> + default "https://github.com/u-boot/u-boot/releases/tag" >>>> + prompt "U-Boot URL" >>>> + help >>>> + Enter the U-Boot URL you want to use. >>>> + >>>> config U_BOOT_VERSION >>>> string >>>> - default "2019.01" >>>> + default "v2022.01" >>>> prompt "U-Boot version" >>>> help >>>> Enter the U-Boot version you want to build. Usually something like >>>> - "2019.01". >>>> + "v2022.01". >>>> >>>> config U_BOOT_MD5 >>>> string >>>> diff --git a/rules/u-boot.make b/rules/u-boot.make >>>> index cadca4a6b..1ba6d47b0 100644 >>>> --- a/rules/u-boot.make >>>> +++ b/rules/u-boot.make >>>> @@ -19,8 +19,8 @@ PACKAGES-$(PTXCONF_U_BOOT) += u-boot >>>> U_BOOT_VERSION := $(call ptx/config-version, PTXCONF_U_BOOT) >>>> U_BOOT_MD5 := $(call ptx/config-md5, PTXCONF_U_BOOT) >>>> U_BOOT := u-boot-$(U_BOOT_VERSION) >>>> -U_BOOT_SUFFIX := tar.bz2 >>>> -U_BOOT_URL := https://ftp.denx.de/pub/u-boot/$(U_BOOT).$(U_BOOT_SUFFIX) >>>> +U_BOOT_SUFFIX := tar.gz >>>> +U_BOOT_URL := $(PTXCONF_U_BOOT_URL)/$(U_BOOT_VERSION).$(U_BOOT_SUFFIX) >>>> U_BOOT_SOURCE := $(SRCDIR)/$(U_BOOT).$(U_BOOT_SUFFIX) >>>> U_BOOT_DIR := $(BUILDDIR)/$(U_BOOT) >>>> U_BOOT_BUILD_DIR := $(U_BOOT_DIR)$(call ptx/ifdef, >>>> PTXCONF_U_BOOT_BUILD_OOT,-build) >>> >>> >>> >>> >> >> >> >