Uh oh!
There was an error while loading. Please reload this page.
Add LICENSE (copy of docker/docker's Apache-2.0 file) - #30
Conversation
friism
commented
Oct 10, 2017
Thanks for sending this PR. I don't think we should add a permissive license to this repo, however. The code in this repo is meant for the sole use of Docker Inc. to build packages for the Docker Community Edition products. If this code had a permissive license, that might be misunderstood to mean that it's ok for others to build and distribute "Docker" packages (it's not ok, "Docker" is protected by a trademark owned by Docker Inc.). Anyone interested in building something out of the open source components that go into Docker should use the Moby project, not this code: https://github.com/moby/moby This packaging code is made available to provide transparency for Docker CE users, and to help them troubleshoot and report issues with Docker CE packages. |
tianon
commented
Oct 10, 2017
Even though the Apache-2.0 license already applies to most of this code, since it was copied from the old https://github.com/docker/docker? |
tianon
commented
Oct 10, 2017
To put that another way, this isn't a "license" to use the Docker trademark, but merely governs how this code is allowed to be re-used, and in this case, is only reflecting the reality of the license this code is already under (due to it having been previously released with this license). |
friism
commented
Oct 10, 2017
@tianon I recognize that the upstream licenses already apply to this code, and I recognize that not having the correct license here is unhygienic. You're also correct that the trademark status of the artifacts produced by this code is strictly speaking an orthogonal concern. Still, I think it's preferable to not include a license in this repo. |
Signed-off-by: Tianon Gravi <admwiggin@gmail.com>
AkihiroSuda
commented
Oct 22, 2019
README says "This repo contains the open source scripts for packaging Docker CE products." 🤔 |
thaJeztah
left a comment
There was a problem hiding this comment.
LGTM
@silvin-lubecki or @StefanScherer PTAL
thaJeztah
commented
Mar 11, 2020
let's merge |
This repository was missing a license, but since it's been created mostly from bits of
github.com/docker/docker, I figured that license was probably appropriate for this top-level (since the individual vendored directories should have theirLICENSEfiles intact anyhow).A quick
diffto show that it's 100% identical:See also docker-archive/docker-ce#103. 👍