meta-virtualization.lists.yoctoproject.org archive mirror
 help / color / mirror / Atom feed
* [meta-virtualization][kirkstone][master][PATCH v2 1/3] nerdctl: install the binaries in OE standard places
@ 2022-08-09 16:31 Jose Quaresma
  2022-08-09 16:31 ` [meta-virtualization][kirkstone][master][PATCH v2 2/3] kubernetes: " Jose Quaresma
  2022-08-09 16:31 ` [meta-virtualization][kirkstone][master][PATCH v2 3/3] kubernetes: replace tabs with spaces for identation Jose Quaresma
  0 siblings, 2 replies; 3+ messages in thread
From: Jose Quaresma @ 2022-08-09 16:31 UTC (permalink / raw)
  To: meta-virtualization; +Cc: Jose Quaresma

- Install the binaries in the default "/usr/bin" and not in "/bin"

- The env BIN_PREFIX is there from the beginning but it is not defined,
also fix a fatal QA errors installed-vs-shipped (only enabled with usrmerge):

ERROR: nerdctl-v0.18.0-r0 do_package: QA Issue: nerdctl: Files/directories were installed but not shipped in any package:
  /bin
  /bin/nerdctl
Please set FILES such that these items are packaged. Alternatively if they are unneeded, avoid installing them or delete them within do_install.
nerdctl: 2 installed and not shipped files. [installed-vs-shipped]
ERROR: nerdctl-v0.18.0-r0 do_package: Fatal QA errors were found, failing task.

Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io>
---

 v2: update commit description, drop k3s patch

 recipes-containers/nerdctl/nerdctl_git.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/recipes-containers/nerdctl/nerdctl_git.bb b/recipes-containers/nerdctl/nerdctl_git.bb
index 5d4d827..ad18f67 100644
--- a/recipes-containers/nerdctl/nerdctl_git.bb
+++ b/recipes-containers/nerdctl/nerdctl_git.bb
@@ -239,8 +239,8 @@ do_compile() {
 }
 
 do_install() {
-        install -d "${D}${BIN_PREFIX}/bin"
-        install -m 755 "${S}/src/import/_output/nerdctl" "${D}${BIN_PREFIX}/bin"
+        install -d ${D}${bindir}
+        install -m 755 ${S}/src/import/_output/nerdctl ${D}${bindir}
 }
 
 INHIBIT_PACKAGE_STRIP = "1"
-- 
2.37.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [meta-virtualization][kirkstone][master][PATCH v2 2/3] kubernetes: install the binaries in OE standard places
  2022-08-09 16:31 [meta-virtualization][kirkstone][master][PATCH v2 1/3] nerdctl: install the binaries in OE standard places Jose Quaresma
@ 2022-08-09 16:31 ` Jose Quaresma
  2022-08-09 16:31 ` [meta-virtualization][kirkstone][master][PATCH v2 3/3] kubernetes: replace tabs with spaces for identation Jose Quaresma
  1 sibling, 0 replies; 3+ messages in thread
From: Jose Quaresma @ 2022-08-09 16:31 UTC (permalink / raw)
  To: meta-virtualization; +Cc: Jose Quaresma

- Install the binaries in the default "/usr/bin" and not in "/bin"

- The env BIN_PREFIX is there from the beginning but it is not defined,
also fix a fatal QA errors installed-vs-shipped (only enabled with usrmerge):

ERROR: kubernetes-1_v1.23.6+gitfbcfa33018159c033aee77b0d5456df6771aa9b5-r0 do_package: QA Issue: kubernetes: Files/directories were installed but not shipped in any package:
  /bin
  /bin/k8s-init
Please set FILES such that these items are packaged. Alternatively if they are unneeded, avoid installing them or delete them within do_install.
kubernetes: 2 installed and not shipped files. [installed-vs-shipped]
ERROR: kubernetes-1_v1.23.6+gitfbcfa33018159c033aee77b0d5456df6771aa9b5-r0 do_package: Fatal QA errors were found, failing task.

Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io>
---

 v2: update commit description, drop k3s patch

 recipes-containers/kubernetes/kubernetes_git.bb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/recipes-containers/kubernetes/kubernetes_git.bb b/recipes-containers/kubernetes/kubernetes_git.bb
index e9460d4..82b75b1 100644
--- a/recipes-containers/kubernetes/kubernetes_git.bb
+++ b/recipes-containers/kubernetes/kubernetes_git.bb
@@ -103,8 +103,8 @@ do_install() {
     install -m 0644 ${WORKDIR}/git/release/cmd/kubepkg/templates/latest/deb/kubeadm/10-kubeadm.conf  ${D}${systemd_unitdir}/system/kubelet.service.d/
 
     if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)}; then
-	install -d "${D}${BIN_PREFIX}/bin"
-	install -m 755 "${WORKDIR}/k8s-init" "${D}${BIN_PREFIX}/bin"
+	install -d ${D}${bindir}
+	install -m 755 ${WORKDIR}/k8s-init ${D}${bindir}
 
 	install -d ${D}${sysconfdir}/sysctl.d
 	install -m 0644 "${WORKDIR}/99-kubernetes.conf" "${D}${sysconfdir}/sysctl.d"
@@ -141,7 +141,7 @@ FILES:kube-proxy = "${bindir}/kube-proxy"
 FILES:${PN}-misc = "${bindir} ${sysconfdir}/sysctl.d"
 
 ALLOW_EMPTY:${PN}-host = "1"
-FILE:${PN}-host = "${BIN_PREFIX}/bin/k8s-init"
+FILE:${PN}-host = "${bindir}/k8s-init"
 RDEPENDS:${PN}-host = "${PN}"
 
 RRECOMMENDS:${PN} = "\
-- 
2.37.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [meta-virtualization][kirkstone][master][PATCH v2 3/3] kubernetes: replace tabs with spaces for identation
  2022-08-09 16:31 [meta-virtualization][kirkstone][master][PATCH v2 1/3] nerdctl: install the binaries in OE standard places Jose Quaresma
  2022-08-09 16:31 ` [meta-virtualization][kirkstone][master][PATCH v2 2/3] kubernetes: " Jose Quaresma
@ 2022-08-09 16:31 ` Jose Quaresma
  1 sibling, 0 replies; 3+ messages in thread
From: Jose Quaresma @ 2022-08-09 16:31 UTC (permalink / raw)
  To: meta-virtualization; +Cc: Jose Quaresma

Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io>
---
 recipes-containers/kubernetes/kubernetes_git.bb | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/recipes-containers/kubernetes/kubernetes_git.bb b/recipes-containers/kubernetes/kubernetes_git.bb
index 82b75b1..4d65b27 100644
--- a/recipes-containers/kubernetes/kubernetes_git.bb
+++ b/recipes-containers/kubernetes/kubernetes_git.bb
@@ -103,11 +103,11 @@ do_install() {
     install -m 0644 ${WORKDIR}/git/release/cmd/kubepkg/templates/latest/deb/kubeadm/10-kubeadm.conf  ${D}${systemd_unitdir}/system/kubelet.service.d/
 
     if ${@bb.utils.contains('DISTRO_FEATURES','systemd','true','false',d)}; then
-	install -d ${D}${bindir}
-	install -m 755 ${WORKDIR}/k8s-init ${D}${bindir}
+        install -d ${D}${bindir}
+        install -m 755 ${WORKDIR}/k8s-init ${D}${bindir}
 
-	install -d ${D}${sysconfdir}/sysctl.d
-	install -m 0644 "${WORKDIR}/99-kubernetes.conf" "${D}${sysconfdir}/sysctl.d"
+        install -d ${D}${sysconfdir}/sysctl.d
+        install -m 0644 "${WORKDIR}/99-kubernetes.conf" "${D}${sysconfdir}/sysctl.d"
     fi
 }
 
-- 
2.37.1



^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-08-09 16:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-09 16:31 [meta-virtualization][kirkstone][master][PATCH v2 1/3] nerdctl: install the binaries in OE standard places Jose Quaresma
2022-08-09 16:31 ` [meta-virtualization][kirkstone][master][PATCH v2 2/3] kubernetes: " Jose Quaresma
2022-08-09 16:31 ` [meta-virtualization][kirkstone][master][PATCH v2 3/3] kubernetes: replace tabs with spaces for identation Jose Quaresma

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 NNTP newsgroup(s).