All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-oe][PATCH] nvme-cli: Use install-spec target
@ 2019-10-09 15:19 Khem Raj
  2019-10-12 12:44 ` [meta-oe][PATCH] nvme-cli: defer host ID generation to post installation Stefan Wiehler
  0 siblings, 1 reply; 5+ messages in thread
From: Khem Raj @ 2019-10-09 15:19 UTC (permalink / raw)
  To: openembedded-devel

install target build install-hostparams as well, which tries to run nvme
tool during build, we defer that to runtime post_install anyway so we
can ignore building it during install task

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb b/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
index 256d83f039..fa47c9e7bc 100644
--- a/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
+++ b/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
@@ -15,7 +15,7 @@ S = "${WORKDIR}/git"
 inherit bash-completion systemd
 
 do_install() {
-    oe_runmake install DESTDIR=${D} PREFIX=${prefix} \
+    oe_runmake install-spec DESTDIR=${D} PREFIX=${prefix} \
         UDEVDIR=${nonarch_base_libdir}/udev SYSTEMDDIR=${systemd_unitdir}
 }
 
-- 
2.23.0



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

* [meta-oe][PATCH] nvme-cli: defer host ID generation to post installation
  2019-10-09 15:19 [meta-oe][PATCH] nvme-cli: Use install-spec target Khem Raj
@ 2019-10-12 12:44 ` Stefan Wiehler
  2019-10-12 12:44   ` Stefan Wiehler
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Wiehler @ 2019-10-12 12:44 UTC (permalink / raw)
  To: openembedded-devel


Right now, we only generate the hostnqn post-installation, but not the
hostid; I agree however that it makes more sense to perform both tasks
on the target.



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

* [meta-oe][PATCH] nvme-cli: defer host ID generation to post installation
  2019-10-12 12:44 ` [meta-oe][PATCH] nvme-cli: defer host ID generation to post installation Stefan Wiehler
@ 2019-10-12 12:44   ` Stefan Wiehler
  2019-10-12 15:37     ` Khem Raj
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Wiehler @ 2019-10-12 12:44 UTC (permalink / raw)
  To: openembedded-devel

Signed-off-by: Stefan Wiehler <stefan.wiehler@missinglinkelectronics.com>
---
 meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb b/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
index 256d83f03..061c7a0fe 100644
--- a/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
+++ b/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
@@ -4,7 +4,7 @@ HOMEPAGE = "https://github.com/linux-nvme/nvme-cli"
 SECTION = "console/utils"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=8264535c0c4e9c6c335635c4026a8022"
-DEPENDS = "util-linux util-linux-native"
+DEPENDS = "util-linux"
 PV .= "+git${SRCPV}"
 
 SRC_URI = "git://github.com/linux-nvme/nvme-cli.git"
@@ -15,15 +15,18 @@ S = "${WORKDIR}/git"
 inherit bash-completion systemd
 
 do_install() {
-    oe_runmake install DESTDIR=${D} PREFIX=${prefix} \
+    oe_runmake install-spec DESTDIR=${D} PREFIX=${prefix} \
         UDEVDIR=${nonarch_base_libdir}/udev SYSTEMDDIR=${systemd_unitdir}
 }
 
 pkg_postinst_ontarget_${PN}() {
     ${sbindir}/nvme gen-hostnqn > ${sysconfdir}/nvme/hostnqn
+    ${bindir}/uuidgen > ${sysconfdir}/nvme/hostid
 }
 
 PACKAGES =+ "${PN}-dracut ${PN}-zsh-completion"
 
 FILES_${PN}-dracut = "${libdir}/dracut/dracut.conf.d"
 FILES_${PN}-zsh-completion = "${datadir}/zsh/site-functions"
+
+RDEPENDS_${PN} = "util-linux-uuidgen"
-- 
2.19.2



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

* Re: [meta-oe][PATCH] nvme-cli: defer host ID generation to post installation
  2019-10-12 12:44   ` Stefan Wiehler
@ 2019-10-12 15:37     ` Khem Raj
  2019-10-15  6:56       ` [meta-oe][PATCH v2] " Stefan Wiehler
  0 siblings, 1 reply; 5+ messages in thread
From: Khem Raj @ 2019-10-12 15:37 UTC (permalink / raw)
  To: Stefan Wiehler, openembedded-devel

Stefan

Please rebase it on top of latest master and resend please. Since half
of this patch is already merged.

On Sat, 2019-10-12 at 14:44 +0200, Stefan Wiehler wrote:
> Signed-off-by: Stefan Wiehler <
> stefan.wiehler@missinglinkelectronics.com>
> ---
>  meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb b/meta-
> oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
> index 256d83f03..061c7a0fe 100644
> --- a/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
> +++ b/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
> @@ -4,7 +4,7 @@ HOMEPAGE = "https://github.com/linux-nvme/nvme-cli"
>  SECTION = "console/utils"
>  LICENSE = "GPLv2"
>  LIC_FILES_CHKSUM =
> "file://LICENSE;md5=8264535c0c4e9c6c335635c4026a8022"
> -DEPENDS = "util-linux util-linux-native"
> +DEPENDS = "util-linux"
>  PV .= "+git${SRCPV}"
>  
>  SRC_URI = "git://github.com/linux-nvme/nvme-cli.git"
> @@ -15,15 +15,18 @@ S = "${WORKDIR}/git"
>  inherit bash-completion systemd
>  
>  do_install() {
> -    oe_runmake install DESTDIR=${D} PREFIX=${prefix} \
> +    oe_runmake install-spec DESTDIR=${D} PREFIX=${prefix} \
>          UDEVDIR=${nonarch_base_libdir}/udev
> SYSTEMDDIR=${systemd_unitdir}
>  }
>  
>  pkg_postinst_ontarget_${PN}() {
>      ${sbindir}/nvme gen-hostnqn > ${sysconfdir}/nvme/hostnqn
> +    ${bindir}/uuidgen > ${sysconfdir}/nvme/hostid
>  }
>  
>  PACKAGES =+ "${PN}-dracut ${PN}-zsh-completion"
>  
>  FILES_${PN}-dracut = "${libdir}/dracut/dracut.conf.d"
>  FILES_${PN}-zsh-completion = "${datadir}/zsh/site-functions"
> +
> +RDEPENDS_${PN} = "util-linux-uuidgen"
> -- 
> 2.19.2
> 



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

* [meta-oe][PATCH v2] nvme-cli: defer host ID generation to post installation
  2019-10-12 15:37     ` Khem Raj
@ 2019-10-15  6:56       ` Stefan Wiehler
  0 siblings, 0 replies; 5+ messages in thread
From: Stefan Wiehler @ 2019-10-15  6:56 UTC (permalink / raw)
  To: openembedded-devel

Signed-off-by: Stefan Wiehler <stefan.wiehler@missinglinkelectronics.com>
---
 meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb b/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
index 6133b3a27..92c902b75 100644
--- a/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
+++ b/meta-oe/recipes-bsp/nvme-cli/nvme-cli_1.9.bb
@@ -4,7 +4,7 @@ HOMEPAGE = "https://github.com/linux-nvme/nvme-cli"
 SECTION = "console/utils"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=8264535c0c4e9c6c335635c4026a8022"
-DEPENDS = "util-linux util-linux-native"
+DEPENDS = "util-linux"
 PV .= "+git${SRCPV}"
 
 SRC_URI = "git://github.com/linux-nvme/nvme-cli.git"
@@ -21,6 +21,7 @@ do_install() {
 
 pkg_postinst_ontarget_${PN}() {
     ${sbindir}/nvme gen-hostnqn > ${sysconfdir}/nvme/hostnqn
+    ${bindir}/uuidgen > ${sysconfdir}/nvme/hostid
 }
 
 PACKAGES =+ "${PN}-dracut ${PN}-zsh-completion"
@@ -28,3 +29,5 @@ PACKAGES =+ "${PN}-dracut ${PN}-zsh-completion"
 FILES_${PN} += "${systemd_system_unitdir}"
 FILES_${PN}-dracut = "${libdir}/dracut/dracut.conf.d"
 FILES_${PN}-zsh-completion = "${datadir}/zsh/site-functions"
+
+RDEPENDS_${PN} = "util-linux-uuidgen"
-- 
2.19.2



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

end of thread, other threads:[~2019-10-15  6:56 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-09 15:19 [meta-oe][PATCH] nvme-cli: Use install-spec target Khem Raj
2019-10-12 12:44 ` [meta-oe][PATCH] nvme-cli: defer host ID generation to post installation Stefan Wiehler
2019-10-12 12:44   ` Stefan Wiehler
2019-10-12 15:37     ` Khem Raj
2019-10-15  6:56       ` [meta-oe][PATCH v2] " Stefan Wiehler

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.