[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: u-boot-am335x-boneblack -> u-boot-am335x-evm-boneblack
From: |
Vagrant Cascadian |
Subject: |
Re: u-boot-am335x-boneblack -> u-boot-am335x-evm-boneblack |
Date: |
Thu, 22 Dec 2022 14:01:44 -0800 |
On 2022-12-22, Maxim Cournoyer wrote:
> Vagrant Cascadian <vagrant@debian.org> writes:
>
>> Wondering what necessitated this change from the old variable name to a
>> new name...
>>
>> commit c04528d2a2597d79278833f3607c806278253446
>> Author: Maxim Cournoyer <maxim.cournoyer@gmail.com>
>> Date: Tue Dec 20 21:25:27 2022 -0500
>>
>> gnu: u-boot-am335x-evm-boneblack: Fix variable name.
>>
>> * gnu/packages/bootloaders.scm (u-boot-am335x-boneblack): Rename to...
>> (u-boot-am335x-evm-boneblack), to match the package name.
>> * gnu/bootloader/u-boot.scm (u-boot-beaglebone-black-bootloader): Adjust
>> accordingly.
>> ...
>> diff --git a/gnu/packages/bootloaders.scm b/gnu/packages/bootloaders.scm
>> index bd9f7bb577..c8b8adbc93 100644
>> --- a/gnu/packages/bootloaders.scm
>> +++ b/gnu/packages/bootloaders.scm
>> @@ -889,7 +889,7 @@ (define*-public (make-u-boot-package board triplet
>> (define-public u-boot-malta
>> (make-u-boot-package "malta" "mips64el-linux-gnuabi64"))
>>
>> -(define-public u-boot-am335x-boneblack
>> +(define-public u-boot-am335x-evm-boneblack
>> (make-u-boot-package
>> "am335x_evm" "arm-linux-gnueabihf"
>> ;; Patch out other device trees to build an image small enough to fit
>>
>> The u-boot-am335x-boneblack was named to match the original target that
>> was removed from upstream, adapting the upstream am335x-evm to fit into
>> a smaller gap in the partition tables... (e.g. 2MB partition offset
>> instead of 4MB offset required by the default am335x-evm board
>> configuration).
>
> The problem was that the *name* of the package was
> "u-boot-am335x-evm-boneblack", as computed by the MAKE-U-BOOT-PACKAGE
> procedure, which includes the board argument in its name (it's been like
> this since its inception in 862e38d5518, 2017).
>
> If the previous variable name should have been its name, the name field
> would have needed to be overridden to it (or perhaps we could introduce
> a #:name argument that would take precedence over any cleverness).
>
> I noticed of the problem when trying to build the package; "guix build
> u-boot-am335x-boneblack" would tell me it didn't exist.
>
> I considered making a deprecated alias but decided against, because in
> the past we didn't when moving/renaming packages *variables*.
Odd. I was certainy able to build u-boot-am335x-boneblack from commit
6b99afeef89233b71d113a63cf04a6b4b49a4679 when it was introduced in 2019,
though it has been quite some time since I tested it...
I will take a guess that it was commit
c2c1dfdf5760873f1db86d14873f725a105f7feb which removed the "name" bit:
(define-public u-boot-am335x-boneblack
- (let ((base (make-u-boot-package
- "am335x_evm" "arm-linux-gnueabihf"
- ;; Patch out other device trees to build an image small enough
- ;; to fit within typical partitioning schemes where the first
- ;; partition begins at sector 2048.
- #:configs '("CONFIG_OF_LIST=\"am335x-evm am335x-boneblack\""))))
- (package
- (inherit base)
- (name "u-boot-am335x-boneblack")
live well,
vagrant
signature.asc
Description: PGP signature
- u-boot-am335x-boneblack -> u-boot-am335x-evm-boneblack, Vagrant Cascadian, 2022/12/21
- Re: u-boot-am335x-boneblack -> u-boot-am335x-evm-boneblack, pelzflorian (Florian Pelz), 2022/12/22
- Re: u-boot-am335x-boneblack -> u-boot-am335x-evm-boneblack, Maxim Cournoyer, 2022/12/22
- Re: u-boot-am335x-boneblack -> u-boot-am335x-evm-boneblack,
Vagrant Cascadian <=
- Re: u-boot-am335x-boneblack -> u-boot-am335x-evm-boneblack, Maxim Cournoyer, 2022/12/26
- Re: u-boot-am335x-boneblack -> u-boot-am335x-evm-boneblack, Vagrant Cascadian, 2022/12/27
- [PATCH] gnu: u-boot-am335x-boneblack: Revert to old name., Maxim Cournoyer, 2022/12/28
- Re: [PATCH] gnu: u-boot-am335x-boneblack: Revert to old name., Vagrant Cascadian, 2022/12/28
- Re: [PATCH] gnu: u-boot-am335x-boneblack: Revert to old name., Maxim Cournoyer, 2022/12/28