From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Cc: huth@tuxfamily.org, qemu-devel@nongnu.org
Subject: Re: [PATCH v2 34/34] next-cube: replace boiler-plate GPL 2.0 or later license text with SPDX identifier
Date: Mon, 16 Dec 2024 10:38:21 +0000 [thread overview]
Message-ID: <Z2ADHbndSn-vrYsK@redhat.com> (raw)
In-Reply-To: <fa8ce465-e539-48ec-abf8-d7296ae42b8a@ilande.co.uk>
On Sat, Dec 14, 2024 at 08:38:06PM +0000, Mark Cave-Ayland wrote:
> On 12/12/2024 17:28, Daniel P. Berrangé wrote:
>
> > On Thu, Dec 12, 2024 at 11:46:20AM +0000, Mark Cave-Ayland wrote:
> > > Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> > > ---
> > > hw/m68k/next-cube.c | 5 +----
> > > 1 file changed, 1 insertion(+), 4 deletions(-)
> > >
> > > diff --git a/hw/m68k/next-cube.c b/hw/m68k/next-cube.c
> > > index 1e96bb02f8..3c2f3e295c 100644
> > > --- a/hw/m68k/next-cube.c
> > > +++ b/hw/m68k/next-cube.c
> > > @@ -4,10 +4,7 @@
> > > * Copyright (c) 2011 Bryce Lanham
> > > * Copyright (c) 2024 Mark Cave-Ayland
> > > *
> > > - * This code is free software; you can redistribute it and/or modify
> > > - * it under the terms of the GNU General Public License as published
> > > - * by the Free Software Foundation; either version 2 of the License,
> > > - * or (at your option) any later version.
> > > + * SPDX-License-Identifier: GPL-2.0-or-later
> > > */
> >
> > While adding a SPDX-License-Identifier alongside existing header text
> > is acceptable, my view is that we should stay away from removing existing
> > license headers. There are some difficult questions wrt interpretation
> > of the GPL in this area & avoiding opening that can of worms would be
> > nice.
>
> I remember you mentioned this before, but I wasn't sure if this would be
> mitigated by the fact that the code originated from GSoC? I'm sure I've seen
> at least one recent patch that made a similar change, but if there really
> are legal reasons not to allow changes of this type then I shall drop it
> from the series.
GSoC isn't really important. The challenging problem here is GPL clause 1
which says
"keep intact all the notices that refer to this License and to the
absence of any warranty"
there are differing opinions on how strictly to interpret the "keep intact"
language there.
While we could have a debate over this and come to some project opinion
IMHO it is a better use of our time to just not remove existing notices.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2024-12-16 10:39 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-12 11:45 [PATCH v2 00/34] next-cube: more tidy-ups and improvements Mark Cave-Ayland
2024-12-12 11:45 ` [PATCH v2 01/34] next-cube: remove 0x14020 dummy value from next_mmio_read() Mark Cave-Ayland
2024-12-12 11:45 ` [PATCH v2 02/34] next-cube: remove overlap between next.dma and next.mmio memory regions Mark Cave-Ayland
2024-12-13 19:37 ` Thomas Huth
2024-12-14 20:59 ` Mark Cave-Ayland
2024-12-12 11:45 ` [PATCH v2 03/34] next-cube: create new next.scsi container memory region Mark Cave-Ayland
2024-12-14 5:26 ` Thomas Huth
2024-12-14 12:39 ` Philippe Mathieu-Daudé
2024-12-12 11:45 ` [PATCH v2 04/34] next-cube: move next_scsi_init() to next_pc_realize() Mark Cave-Ayland
2024-12-12 11:45 ` [PATCH v2 05/34] next-cube: introduce next_pc_init() object init function Mark Cave-Ayland
2024-12-12 11:45 ` [PATCH v2 06/34] next-cube: introduce next-scsi device Mark Cave-Ayland
2024-12-12 11:45 ` [PATCH v2 07/34] next-cube: move SCSI CSRs from next-pc to the " Mark Cave-Ayland
2024-12-14 5:28 ` Thomas Huth
2024-12-12 11:45 ` [PATCH v2 08/34] next-cube: move SCSI 4020/4021 logic from next-pc device to " Mark Cave-Ayland
2024-12-14 5:32 ` Thomas Huth
2024-12-12 11:45 ` [PATCH v2 09/34] next-cube: move floppy disk MMIO to separate memory region in next-pc Mark Cave-Ayland
2024-12-12 11:45 ` [PATCH v2 10/34] next-cube: map ESCC registers as a subregion of the next.scr memory region Mark Cave-Ayland
2024-12-14 12:42 ` Philippe Mathieu-Daudé
2024-12-12 11:45 ` [PATCH v2 11/34] next-cube: move ESCC to be QOM child of next-pc device Mark Cave-Ayland
2024-12-14 5:50 ` Thomas Huth
2024-12-14 6:10 ` Thomas Huth
2024-12-14 12:45 ` Philippe Mathieu-Daudé
2024-12-14 21:32 ` Mark Cave-Ayland
2024-12-12 11:45 ` [PATCH v2 12/34] next-cube: move timer MMIO to separate memory region on " Mark Cave-Ayland
2024-12-14 13:29 ` Philippe Mathieu-Daudé
2024-12-14 22:09 ` Mark Cave-Ayland
2024-12-12 11:45 ` [PATCH v2 13/34] next-cube: move en ethernet " Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 14/34] next-cube: add empty slots for unknown accesses to next.scr memory region Mark Cave-Ayland
2024-12-14 13:32 ` Philippe Mathieu-Daudé
2024-12-14 22:27 ` Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 15/34] next-cube: remove unused " Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 16/34] next-cube: rearrange NeXTState declarations to improve readability Mark Cave-Ayland
2024-12-14 13:34 ` Philippe Mathieu-Daudé
2024-12-12 11:46 ` [PATCH v2 17/34] next-cube: convert next-pc device to use Resettable interface Mark Cave-Ayland
2024-12-14 13:34 ` Philippe Mathieu-Daudé
2024-12-12 11:46 ` [PATCH v2 18/34] next-cube: rename typedef struct NextRtc to NeXTRTC Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 19/34] next-cube: use qemu_irq to drive int_status in next_scr2_rtc_update() Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 20/34] next-cube: separate rtc read and write shift logic Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 21/34] next-cube: always use retval to return rtc read values Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 22/34] next-cube: use named gpio to set RTC data bit in scr2 Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 23/34] next-cube: use named gpio to read " Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 24/34] next-cube: don't use rtc phase value of -1 Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 25/34] next-cube: QOMify NeXTRTC Mark Cave-Ayland
2024-12-14 6:08 ` Thomas Huth
2024-12-14 21:30 ` Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 26/34] next-cube: move reset of next-rtc fields from next-pc to next-rtc Mark Cave-Ayland
2024-12-14 13:41 ` Philippe Mathieu-Daudé
2024-12-12 11:46 ` [PATCH v2 27/34] next-cube: move rtc-data-in gpio from next-pc to next-rtc device Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 28/34] next-cube: use named gpio output for next-rtc data Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 29/34] next-cube: add rtc-cmd-reset named gpio to reset the rtc state machine Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 30/34] next-cube: add rtc-power-out named gpio to trigger the NEXT_PWR_I interrupt Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 31/34] next-cube: move next_rtc_cmd_is_write() and next_rtc_data_in_irq() functions Mark Cave-Ayland
2024-12-14 6:14 ` Thomas Huth
2024-12-12 11:46 ` [PATCH v2 32/34] next-cube: rename old_scr2 and scr2_2 in next_scr2_rtc_update() Mark Cave-Ayland
2024-12-12 11:46 ` [PATCH v2 33/34] next-cube: add my copyright to the top of the file Mark Cave-Ayland
2024-12-14 13:41 ` Philippe Mathieu-Daudé
2024-12-12 11:46 ` [PATCH v2 34/34] next-cube: replace boiler-plate GPL 2.0 or later license text with SPDX identifier Mark Cave-Ayland
2024-12-12 17:28 ` Daniel P. Berrangé
2024-12-14 20:38 ` Mark Cave-Ayland
2024-12-16 10:38 ` Daniel P. Berrangé [this message]
2024-12-16 11:17 ` Thomas Huth
2024-12-22 12:56 ` Mark Cave-Ayland
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Z2ADHbndSn-vrYsK@redhat.com \
--to=berrange@redhat.com \
--cc=huth@tuxfamily.org \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.