public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Thomas Wolff <towo@towo.net>
To: cygwin-apps@cygwin.com
Subject: Re: LICENSE values for non-standard OSS licenses
Date: Wed, 12 Oct 2022 10:36:02 +0200	[thread overview]
Message-ID: <e50707c7-e9b1-2584-6974-3562c48e913b@towo.net> (raw)
In-Reply-To: <d0ab5486-3662-d4ef-3a5f-da00fa7a368e@SystematicSw.ab.ca>



Am 11/10/2022 um 22:13 schrieb Brian Inglis:
> On Tue, 11 Oct 2022 09:37:23 +0100, Adam Dinwoodie wrote:
>> I'm trying to upload a new version of git-filter-repo, and took the
>> opportunity to set the LICENSE value in the cygport file.  The new value
>> looks valid according to my reading of the SPDX specification, but is
>> being rejected by calm.
>> The license for git-filter-repo is a bit complicated, because different
>> parts have different licenses, and several of them aren't "normal"
>> licenses.  The license is described at [0] and files referenced / linked
>> from there.
>> [0]: https://github.com/newren/git-filter-repo/blob/main/COPYING
>> I've encoded this as the somewhat verbose
>>     LICENSE='(MIT OR LicenseRef-inherit-git OR 
>> LicenseRef-inherit-libgit2) AND (MIT OR LicenseRef-inherit-git OR 
>> LicenseRef-inherit-libgit2 OR LicenseRef-inherit-libgit2-examples) 
>> AND GPL-2.0-only'
 From a mere Boolean perspective, this looks redundant and should be 
simplified to
     LICENSE='(MIT OR LicenseRef-inherit-git OR 
LicenseRef-inherit-libgit2) AND GPL-2.0-only'

>> The error I'm getting from calm is as follows:
>> ```
>> ERROR: invalid hints git-filter-repo-2.38.0-1-src.hint
>> ERROR: package 'git-filter-repo': errors in license expression: 
>> ['Unknown license key(s): LicenseRef-inherit-git, 
>> LicenseRef-inherit-libgit2, LicenseRef-inherit-libgit2-examples']
>> ERROR: errors while parsing hints for package 'git-filter-repo'
>> ERROR: error parsing /sourceware/cygwin-staging/home/Adam 
>> Dinwoodie/noarch/release/git-filter-repo/git-filter-repo-2.38.0-1-src.hint
>> ERROR: error while reading uploaded arch noarch packages from 
>> maintainer Adam Dinwoodie
>> SUMMARY: 5 ERROR(s)
>> ```
>> So it looks like the issue is the way I've encoded the non-standard
>> licensing options.  "LicenseRef-"(idstring) seems to be the way to
>> encode this sort scenario, per [1] and [2], but that doesn't seem to be
>> acceptable to calm.
>> [1]: 
>> https://spdx.github.io/spdx-spec/v2.3/other-licensing-information-detected/
>> [2]: https://spdx.github.io/spdx-spec/v2.3/SPDX-license-expressions/
>> Are there any suggestions about how to resolve this?  I don't think I
>> can just use the standard license strings: even if we used GPL-2.0-only
>> in place of LicenseRef-inherit-git -- incorrect as that's the license
>> *currently* used by Git, but the license for git-filter-repo explicitly
>> incorporates any future OSS license Git might use -- that still leaves
>> the problem of LicenseRef-inherit-libgit2, which is currently GPL 2.0
>> with an exception that's not covered by any of the SPDX standard
>> exceptions.
>> For now I can just remove the LICENSE values to get the build released,
>> but that seems like a temporary approach at best...
>
> To a similar issue of mine in another thread here (search license) Jon 
> replied calm uses:
>
>     https://github.com/nexB/license-expression
>
> produced by the same project/dev as scancode (which scans a codebase 
> to identify licences as part of project AboutCode), which has 
> registered an SPDX namespace for its own LicenceRefs available at:
>
>     https://scancode-licensedb.aboutcode.org/
>
> which makes me believe Cygwin should use 
> LicenseRef-scancode-public-domain or as referenced there 
> LicenseRef-PublicDomain, and license-expression should be able to use 
> the scancode list.
>


  reply	other threads:[~2022-10-12  8:36 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-11  8:37 Adam Dinwoodie
2022-10-11 20:13 ` Brian Inglis
2022-10-12  8:36   ` Thomas Wolff [this message]
2022-10-12  9:51     ` Adam Dinwoodie
2022-10-12  9:00   ` Adam Dinwoodie
2022-10-12 15:45     ` Brian Inglis
2022-10-12 20:14       ` Adam Dinwoodie
2022-10-12 17:58 ` Achim Gratz
2022-10-12 18:59   ` Adam Dinwoodie
2022-10-12 22:28     ` Brian Inglis
2022-10-13 10:32       ` Adam Dinwoodie
2022-10-15 19:27         ` Brian Inglis
2022-10-16  8:42     ` ASSI
2022-10-14 16:28 ` Jon Turney
2022-10-15 12:58   ` Adam Dinwoodie
2022-10-30 13:15     ` Jon Turney
2022-10-30 17:19     ` Brian Inglis

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=e50707c7-e9b1-2584-6974-3562c48e913b@towo.net \
    --to=towo@towo.net \
    --cc=cygwin-apps@cygwin.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).