All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.