From: Chen Qi <[email protected]>

Things have changed since these two recipe were added. Now
https://github.com/docker/docker has been redirected to
https://github.com/moby/moby. This means the contents of
docker-ce_git.bb and docker-moby_git.bb are essentially
the same except for some comments.

So let's merge them into one and keep the merged recipe's name
docker-moby, as the repo's name is moby.

The PROVIDES and RPROVIDES are added 'docker-ce' for backward
compatibility.

Signed-off-by: Chen Qi <[email protected]>
---
 conf/layer.conf                              |  1 -
 recipes-containers/docker/docker-ce_git.bb   | 59 --------------------
 recipes-containers/docker/docker-moby_git.bb | 27 ++++-----
 recipes-containers/docker/docker.inc         |  8 ++-
 4 files changed, 17 insertions(+), 78 deletions(-)
 delete mode 100644 recipes-containers/docker/docker-ce_git.bb

diff --git a/conf/layer.conf b/conf/layer.conf
index 88ac390..9a26bcc 100644
--- a/conf/layer.conf
+++ b/conf/layer.conf
@@ -41,7 +41,6 @@ require conf/distro/include/virt_security_flags.inc
 
 PREFERRED_PROVIDER_virtual/runc ?= "runc-opencontainers"
 PREFERRED_PROVIDER_virtual/containerd ?= "containerd-opencontainers"
-PREFERRED_PROVIDER_virtual/docker ?= "docker-ce"
 
 # Sanity check for meta-virtualization layer.
 # Setting SKIP_META_VIRT_SANITY_CHECK to "1" would skip the bbappend files 
check.
diff --git a/recipes-containers/docker/docker-ce_git.bb 
b/recipes-containers/docker/docker-ce_git.bb
deleted file mode 100644
index 6781c77..0000000
--- a/recipes-containers/docker/docker-ce_git.bb
+++ /dev/null
@@ -1,59 +0,0 @@
-HOMEPAGE = "http://www.docker.com";
-SUMMARY = "Linux container runtime"
-DESCRIPTION = "Linux container runtime \
- Docker complements kernel namespacing with a high-level API which \
- operates at the process level. It runs unix processes with strong \
- guarantees of isolation and repeatability across servers. \
- . \
- Docker is a great building block for automating distributed systems: \
- large-scale web deployments, database clusters, continuous deployment \
- systems, private PaaS, service-oriented architectures, etc. \
- . \
- This package contains the daemon and client, which are \
- officially supported on x86_64 and arm hosts. \
- Other architectures are considered experimental. \
- . \
- Also, note that kernel version 3.10 or above is required for proper \
- operation of the daemon process, and that any lower versions may have \
- subtle and/or glaring issues. \
- "
-
-#
-# https://github.com/docker/docker-ce-packaging.git
-#  common.mk:
-#    DOCKER_CLI_REPO    ?= https://github.com/docker/cli.git
-#    DOCKER_ENGINE_REPO ?= https://github.com/docker/docker.git
-#    REF                ?= HEAD
-#    DOCKER_CLI_REF     ?= $(REF)
-#    DOCKER_ENGINE_REF  ?= $(REF)
-#
-# These follow the tags for our releases in the listed repositories
-# so we get that tag, and make it our SRCREVS:
-#
-
-SRCREV_docker = "219f21bf07502b447095649b5a2764661737f164"
-SRCREV_libnetwork = "25ec449c45d2075c85fb3688ef4c1730be0466e0"
-SRCREV_cli = "569dd73db13099a7c3104d73aa15117b359045bc"
-SRCREV_FORMAT = "docker_libnetwork"
-SRC_URI = "\
-       
git://github.com/docker/docker.git;branch=23.0;name=docker;protocol=https \
-       
git://github.com/docker/libnetwork.git;branch=master;name=libnetwork;destsuffix=git/libnetwork;protocol=https
 \
-       
git://github.com/docker/cli;branch=23.0;name=cli;destsuffix=git/cli;protocol=https
 \
-       file://0001-libnetwork-use-GO-instead-of-go.patch \
-       file://docker.init \
-        file://0001-dynbinary-use-go-cross-compiler.patch;patchdir=src/import \
-        file://0001-cli-use-external-GO111MODULE-and-cross-compiler.patch \
-       "
-
-DOCKER_COMMIT = "${SRCREV_docker}"
-
-require docker.inc
-
-# Apache-2.0 for docker
-LICENSE = "Apache-2.0"
-LIC_FILES_CHKSUM = 
"file://src/import/LICENSE;md5=4859e97a9c7780e77972d989f0823f28"
-
-DOCKER_VERSION = "23.0.2-ce"
-PV = "${DOCKER_VERSION}+git${SRCREV_docker}"
-
-CVE_PRODUCT = "docker mobyproject:moby"
diff --git a/recipes-containers/docker/docker-moby_git.bb 
b/recipes-containers/docker/docker-moby_git.bb
index cd6a741..94c72e7 100644
--- a/recipes-containers/docker/docker-moby_git.bb
+++ b/recipes-containers/docker/docker-moby_git.bb
@@ -18,21 +18,18 @@ DESCRIPTION = "Linux container runtime \
  subtle and/or glaring issues. \
  "
 
-# Notes:
-#   - This docker variant uses moby and the other individually maintained
-#     upstream variants for SRCREVs
-#   - It is a true community / upstream tracking build, and is not a
-#     docker curated set of commits or additions
-#   - The version number on this package tracks the versions assigned to
-#     the curated docker-ce repository. This allows compatibility and
-#     functional equivalence, while allowing new features to be more
-#     easily added.
-#   - This could be called "docker-moby" or just "moby" in the future, but
-#     that would require the creation of a virtual/docker dependency, which
-#     is possible, but overkill at the moment (while we wait for the upstream
-#     to stop changing).
-#   - The common components of this recipe and docker-ce do need to be moved
-#     to a docker.inc recipe
+#
+# https://github.com/docker/docker-ce-packaging.git
+#  common.mk:
+#    DOCKER_CLI_REPO    ?= https://github.com/docker/cli.git
+#    DOCKER_ENGINE_REPO ?= https://github.com/docker/docker.git
+#    REF                ?= HEAD
+#    DOCKER_CLI_REF     ?= $(REF)
+#    DOCKER_ENGINE_REF  ?= $(REF)
+#
+# These follow the tags for our releases in the listed repositories
+# so we get that tag, and make it our SRCREVS:
+#
 
 SRCREV_moby = "219f21bf07502b447095649b5a2764661737f164"
 SRCREV_libnetwork = "25ec449c45d2075c85fb3688ef4c1730be0466e0"
diff --git a/recipes-containers/docker/docker.inc 
b/recipes-containers/docker/docker.inc
index 4a4bc4f..5cd8bdc 100644
--- a/recipes-containers/docker/docker.inc
+++ b/recipes-containers/docker/docker.inc
@@ -31,15 +31,17 @@ RDEPENDS:${PN} += "virtual-containerd 
${VIRTUAL-RUNTIME_container_runtime}"
 
 RRECOMMENDS:${PN} = "kernel-module-dm-thin-pool kernel-module-nf-nat 
kernel-module-nf-conntrack-netlink kernel-module-xt-addrtype 
kernel-module-xt-masquerade"
 
-PROVIDES += "virtual/docker"
-
-# we want all the docker variant recpes to be installable via "docker"
+# make it installable via "docker"
 PACKAGE_NAME = "docker"
 RPROVIDES:${PN} += "docker"
 RPROVIDES:${PN}-dbg += "docker-dbg"
 RPROVIDES:${PN}-dev += "docker-dev"
 RPROVIDES:${PN}-contrip += "docker-dev"
 
+# For docker-ce backward compatibility
+PROVIDES += "docker-ce"
+RPROVIDES:${PN} += "docker-ce"
+
 inherit pkgconfig
 PACKAGECONFIG ??= "docker-init seccomp"
 PACKAGECONFIG[seccomp] = "seccomp,,libseccomp"
-- 
2.40.0

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#8014): 
https://lists.yoctoproject.org/g/meta-virtualization/message/8014
Mute This Topic: https://lists.yoctoproject.org/mt/98550499/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-virtualization/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to