GNU bug report logs - #77148
[PATCH 0/9] New firmware and updates

Previous Next

Package: guix-patches;

Reported by: Efraim Flashner <efraim <at> flashner.co.il>

Date: Fri, 21 Mar 2025 08:51:06 UTC

Severity: normal

Tags: patch

Full log


View this message in rfc822 format

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Efraim Flashner <efraim <at> flashner.co.il>
Cc: 77148 <at> debbugs.gnu.org
Subject: [bug#77148] [PATCH 1/9] gnu: Add u-boot-qemu-ppce550.
Date: Mon, 24 Mar 2025 09:24:43 +0900
Hi Efraim!

Efraim Flashner <efraim <at> flashner.co.il> writes:

[...]

>> > +(define-public u-boot-qemu-ppce550
>> > +  (let ((base
>> > +          (make-u-boot-package "qemu-ppce500" "powerpc-linux-gnu"
>> > +                               ;; Disable features that require OpenSSL due
>> > +                               ;; to GPL/Openssl license incompatibilities.
>> > +                               ;; See https://bugs.gnu.org/34717 for
>> > +                               ;; details.
>> > +                               #:configs '("# CONFIG_FIT_SIGNATURE is not set"))))
>> 
>> I thought the GPL incompatibility had been resolved in recent years (see
>> https://openssl-library.org/post/2017-03-20-license/) ?
>
> I just copied it from the config above it, but I think we can drop it.

Yes!

>> > +    (package
>> > +      (inherit base)
>> > +      (arguments
>> > +       (substitute-keyword-arguments (package-arguments base)
>> > +         ((#:phases phases #~'%standard-phases)
>> > +          #~(modify-phases #$phases
>> > +              (replace 'install
>> > +                (lambda _
>> > +                  (mkdir-p (string-append #$output "/share/qemu"))
>> > +                  (copy-file "u-boot" (string-append
>> > +                                        #$output "/share/qemu/u-boot.e500")))))))))))
>> 
>> I'm not sure why we need a qemu variant; just having u-boot-ppce550 with
>> a libvirt/QEMU firmware descriptor should be enough, unless I'm missing
>> something.
>
> qemu_ppce550 is the name of the config and I figured for qemu it would
> be better to have the name of the file match the bundled firmware file.

Makes sense.  Thanks for the clarification.

-- 
Thanks,
Maxim




This bug report was last modified 79 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.