qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 3/4] slirp: remove reference to COPYRIGHT fil


From: Marc-André Lureau
Subject: Re: [Qemu-devel] [PATCH v2 3/4] slirp: remove reference to COPYRIGHT file
Date: Wed, 13 Mar 2019 19:07:24 +0100

Hi

On Wed, Mar 13, 2019 at 6:57 PM Eric Blake <address@hidden> wrote:
>
> On 3/13/19 11:42 AM, Marc-André Lureau wrote:
> > The slirp COPYRIGHT file is a BSD-3 license.
>
> Is it? I only see 2 clauses listed there.

In commit 2f5f89963186d42a7ded253bc6cf5b32abb45cec ("Remove the
advertising clause from the slirp license"). the 4th BSD clause was
removed.

Daniel Gasparovski & Kelly Price gave permission to license slirp
under 3-clause BSD.

So I think we should instead put back the 3rd clause in COPYRIGHT file.

What do you think?

>
> My understanding (although I may be wrong) is that 2-clause can be
> strengthened to 3-clause unilaterally, but 3-clause cannot be weakened
> to 2-clause without copyright owner buy-in.
>
> > Instead of referring to
> > another project file, the SPDX license notice present in all source
> > files states that unequivocally.
>
> Should we put this patch before 2/4?  You'd have to reword things if you
> do it in that order, though.
>
> >
> > Signed-off-by: Marc-André Lureau <address@hidden>
> > ---
>
> > +++ b/slirp/src/debug.h
> > @@ -1,9 +1,6 @@
> >  /* SPDX-License-Identifier: BSD-3-Clause */
> >  /*
> >   * Copyright (c) 1995 Danny Gasparovski.
> > - *
> > - * Please read the file COPYRIGHT for the
> > - * terms and conditions of the copyright.
>
> If we resolve the issue about slirp/COPYRIGHT being 2-clause or
> 3-clause, and we are happy with the addition of the SPDX tag in 2/4,
> then this change makes sense.  But I'm hesitant to give R-b to this one
> without resolving those questions first.
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc.           +1-919-301-3226
> Virtualization:  qemu.org | libvirt.org
>



reply via email to

[Prev in Thread] Current Thread [Next in Thread]