Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

go.dev: known licenses are not recognised #36758

Closed
kortschak opened this issue Jan 25, 2020 · 7 comments
Closed

go.dev: known licenses are not recognised #36758

kortschak opened this issue Jan 25, 2020 · 7 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. pkgsite

Comments

@kortschak
Copy link
Contributor

This is a refiling of an issue that was opened before go.dev issues were handled in the open (#35595).

The underlying cause appears to be google/licensecheck#6 and possibly in some cases google/licensecheck#4.

@ALTree ALTree added pkgsite NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Jan 25, 2020
@rogpeppe
Copy link
Contributor

Here's an example of a license that's not recognised and probably should be:

https://github.com/gogo/protobuf/blob/master/LICENSE

Also, because this seems to be a decent place to record this, many of the Canonical open source contributions are under an "LGPL with static linking exception" license, which isn't recognised.

For example, the popular juju/ratelimit package (license here), which is imported by ~600 other packages, is unavailable on pkg.go.dev because the license isn't recognised.

@julieqiu
Copy link
Member

julieqiu commented Jan 29, 2020

We're aware of issues with github.com/juju/ratelimit and github.com/gogo/protobuf - license issues for both of these packages should be fixed soon.

/cc @jba

@jba
Copy link
Contributor

jba commented Jan 31, 2020

As of today, gonum.org/v1/gonum, github.com/gogo/protobuf, github.com/juju/ratelimit all are correctly identified, as should all modules using the Canonical variation of LGPL 3.0.

Closing this, but reopen for other licenses you think should be detected.

We also think we are doing the right thing for vendoring, at least for https://pkg.go.dev/k8s.io/kubernetes/third_party/forked/gonum/graph, but I'd like to discuss that in a separate issue.

@jba jba changed the title go.dev: known licenses are not recognised and the site misrepresents license status of vendored packages go.dev: known licenses are not recognised Jan 31, 2020
@jba jba closed this as completed Jan 31, 2020
@kortschak
Copy link
Contributor Author

kortschak commented Jan 31, 2020

This is still not working correctly: see https://pkg.go.dev/modernc.org/cc which is a BSD 3 clause.

Before the action of redirecting currently working online documentation for package to pkg.go.dev, go.dev should be very sure that package providers are not harmed by reducing their visibility. I don't think go.dev can guarantee that given this failure and the broader issue of the more restrictive licensing regime that is in place on go.dev.

@kortschak
Copy link
Contributor Author

kortschak commented Jan 31, 2020

I also strongly disagree that you are doing the correct thing with vendored code. You are displaying host license on the vendored code package, which is irrelevant and not displaying vendored code licenses on the host module. This is facing exactly the wrong direction.

If you want to discuss this in another issue, please open that and cc me there (or return the text of the issue title to the correct form).

@jba
Copy link
Contributor

jba commented Jan 31, 2020

This is still not working correctly: see https://pkg.go.dev/modernc.org/cc which is a BSD

I didn't mention that package because it's not listed in this issue. However, the CL for it has been merged and will be in the next release.

@kortschak
Copy link
Contributor Author

It is explciitly noted in the issue linked in the OP.

@golang golang locked and limited conversation to collaborators Jan 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. pkgsite
Projects
None yet
Development

No branches or pull requests

6 participants