* [Buildroot] [PATCH] host-genimage: add host-mtools to dependency list @ 2019-02-01 9:13 Alexandru Ardelean 2019-02-01 11:59 ` Peter Korsgaard 0 siblings, 1 reply; 4+ messages in thread From: Alexandru Ardelean @ 2019-02-01 9:13 UTC (permalink / raw) To: buildroot From: Alexandru Ardelean <ardeleanalex@gmail.com> The mcopy command is needed to generate vfat images. If the host system has mtools installed, all is fine, since `genimage` would use it. But since there is already a `host-mtools` build in the buildroot packages, then this can be used instead. Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com> --- package/genimage/genimage.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package/genimage/genimage.mk b/package/genimage/genimage.mk index 393c3e9b94..18eea58af2 100644 --- a/package/genimage/genimage.mk +++ b/package/genimage/genimage.mk @@ -7,7 +7,7 @@ GENIMAGE_VERSION = 10 GENIMAGE_SOURCE = genimage-$(GENIMAGE_VERSION).tar.xz GENIMAGE_SITE = https://github.com/pengutronix/genimage/releases/download/v$(GENIMAGE_VERSION) -HOST_GENIMAGE_DEPENDENCIES = host-pkgconf host-libconfuse +HOST_GENIMAGE_DEPENDENCIES = host-pkgconf host-libconfuse host-mtools GENIMAGE_LICENSE = GPL-2.0 GENIMAGE_LICENSE_FILES = COPYING -- 2.17.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] host-genimage: add host-mtools to dependency list 2019-02-01 9:13 [Buildroot] [PATCH] host-genimage: add host-mtools to dependency list Alexandru Ardelean @ 2019-02-01 11:59 ` Peter Korsgaard 2019-02-01 13:07 ` Alexandru Ardelean 0 siblings, 1 reply; 4+ messages in thread From: Peter Korsgaard @ 2019-02-01 11:59 UTC (permalink / raw) To: buildroot >>>>> "Alexandru" == Alexandru Ardelean <ardeleanalex@gmail.com> writes: > From: Alexandru Ardelean <ardeleanalex@gmail.com> > The mcopy command is needed to generate vfat images. > If the host system has mtools installed, all is fine, since `genimage` > would use it. > But since there is already a `host-mtools` build in the buildroot > packages, then this can be used instead. > Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com> There are use cases for genimage that does not require mcopy, so this isn't really nice. But it is indeed true that a lot of genimage use cases require host-dosfstools and host-mtools: E.G. from raspberrypi_defconfig: # Required tools to create the SD image BR2_PACKAGE_HOST_DOSFSTOOLS=y BR2_PACKAGE_HOST_GENIMAGE=y BR2_PACKAGE_HOST_MTOOLS=y > --- > package/genimage/genimage.mk | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > diff --git a/package/genimage/genimage.mk b/package/genimage/genimage.mk > index 393c3e9b94..18eea58af2 100644 > --- a/package/genimage/genimage.mk > +++ b/package/genimage/genimage.mk > @@ -7,7 +7,7 @@ > GENIMAGE_VERSION = 10 > GENIMAGE_SOURCE = genimage-$(GENIMAGE_VERSION).tar.xz > GENIMAGE_SITE = https://github.com/pengutronix/genimage/releases/download/v$(GENIMAGE_VERSION) > -HOST_GENIMAGE_DEPENDENCIES = host-pkgconf host-libconfuse > +HOST_GENIMAGE_DEPENDENCIES = host-pkgconf host-libconfuse host-mtools > GENIMAGE_LICENSE = GPL-2.0 > GENIMAGE_LICENSE_FILES = COPYING > -- > 2.17.1 > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] host-genimage: add host-mtools to dependency list 2019-02-01 11:59 ` Peter Korsgaard @ 2019-02-01 13:07 ` Alexandru Ardelean 2019-02-04 10:10 ` Yann E. MORIN 0 siblings, 1 reply; 4+ messages in thread From: Alexandru Ardelean @ 2019-02-01 13:07 UTC (permalink / raw) To: buildroot On Fri, Feb 1, 2019 at 1:59 PM Peter Korsgaard <peter@korsgaard.com> wrote: > > >>>>> "Alexandru" == Alexandru Ardelean <ardeleanalex@gmail.com> writes: > > > From: Alexandru Ardelean <ardeleanalex@gmail.com> > > The mcopy command is needed to generate vfat images. > > If the host system has mtools installed, all is fine, since `genimage` > > would use it. > > > But since there is already a `host-mtools` build in the buildroot > > packages, then this can be used instead. > > > Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com> > > There are use cases for genimage that does not require mcopy, so this > isn't really nice. > > But it is indeed true that a lot of genimage use cases require > host-dosfstools and host-mtools: > Another idea I was thinking of [in case this one wasn't very popular], is to add in Config.in.host a symbol that adds/selects host-mtools if enabled. Personally, I don't mind if we leave this as-is and manually select host-mtools for genimage. For a use-case I encountered, adding host-mtools like this change seemed useful. Thanks Alex > E.G. from raspberrypi_defconfig: > > # Required tools to create the SD image > BR2_PACKAGE_HOST_DOSFSTOOLS=y > BR2_PACKAGE_HOST_GENIMAGE=y > BR2_PACKAGE_HOST_MTOOLS=y > > > > --- > > package/genimage/genimage.mk | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > diff --git a/package/genimage/genimage.mk b/package/genimage/genimage.mk > > index 393c3e9b94..18eea58af2 100644 > > --- a/package/genimage/genimage.mk > > +++ b/package/genimage/genimage.mk > > @@ -7,7 +7,7 @@ > > GENIMAGE_VERSION = 10 > > GENIMAGE_SOURCE = genimage-$(GENIMAGE_VERSION).tar.xz > > GENIMAGE_SITE = https://github.com/pengutronix/genimage/releases/download/v$(GENIMAGE_VERSION) > > -HOST_GENIMAGE_DEPENDENCIES = host-pkgconf host-libconfuse > > +HOST_GENIMAGE_DEPENDENCIES = host-pkgconf host-libconfuse host-mtools > > GENIMAGE_LICENSE = GPL-2.0 > > GENIMAGE_LICENSE_FILES = COPYING > > > -- > > 2.17.1 > > > _______________________________________________ > > buildroot mailing list > > buildroot at busybox.net > > http://lists.busybox.net/mailman/listinfo/buildroot > > -- > Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] host-genimage: add host-mtools to dependency list 2019-02-01 13:07 ` Alexandru Ardelean @ 2019-02-04 10:10 ` Yann E. MORIN 0 siblings, 0 replies; 4+ messages in thread From: Yann E. MORIN @ 2019-02-04 10:10 UTC (permalink / raw) To: buildroot Alexandru, All, On 2019-02-01 15:07 +0200, Alexandru Ardelean spake thusly: > On Fri, Feb 1, 2019 at 1:59 PM Peter Korsgaard <peter@korsgaard.com> wrote: > > > > >>>>> "Alexandru" == Alexandru Ardelean <ardeleanalex@gmail.com> writes: > > > > > From: Alexandru Ardelean <ardeleanalex@gmail.com> > > > The mcopy command is needed to generate vfat images. > > > If the host system has mtools installed, all is fine, since `genimage` > > > would use it. > > > > > But since there is already a `host-mtools` build in the buildroot > > > packages, then this can be used instead. > > > > > Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com> > > > > There are use cases for genimage that does not require mcopy, so this > > isn't really nice. > > > > But it is indeed true that a lot of genimage use cases require > > host-dosfstools and host-mtools: > > > > Another idea I was thinking of [in case this one wasn't very popular], > is to add in Config.in.host a symbol that adds/selects host-mtools if > enabled. That would give the same outcome: host-mtools would be buiklt, even in situations they are nto needed. > Personally, I don't mind if we leave this as-is and manually select > host-mtools for genimage. > > For a use-case I encountered, adding host-mtools like this change seemed useful. A few other tools could be usefull as wel, like host-e2fsprogs or such. But we do not want to build all of them in all cases, so I side with Peter here: we have to rely on the user selecting what they need. So, I've marked this as rejected in patchwork. Thanks! Regards, Yann E. MORIN. > Thanks > Alex > > > E.G. from raspberrypi_defconfig: > > > > # Required tools to create the SD image > > BR2_PACKAGE_HOST_DOSFSTOOLS=y > > BR2_PACKAGE_HOST_GENIMAGE=y > > BR2_PACKAGE_HOST_MTOOLS=y > > > > > > > --- > > > package/genimage/genimage.mk | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > diff --git a/package/genimage/genimage.mk b/package/genimage/genimage.mk > > > index 393c3e9b94..18eea58af2 100644 > > > --- a/package/genimage/genimage.mk > > > +++ b/package/genimage/genimage.mk > > > @@ -7,7 +7,7 @@ > > > GENIMAGE_VERSION = 10 > > > GENIMAGE_SOURCE = genimage-$(GENIMAGE_VERSION).tar.xz > > > GENIMAGE_SITE = https://github.com/pengutronix/genimage/releases/download/v$(GENIMAGE_VERSION) > > > -HOST_GENIMAGE_DEPENDENCIES = host-pkgconf host-libconfuse > > > +HOST_GENIMAGE_DEPENDENCIES = host-pkgconf host-libconfuse host-mtools > > > GENIMAGE_LICENSE = GPL-2.0 > > > GENIMAGE_LICENSE_FILES = COPYING > > > > > -- > > > 2.17.1 > > > > > _______________________________________________ > > > buildroot mailing list > > > buildroot at busybox.net > > > http://lists.busybox.net/mailman/listinfo/buildroot > > > > -- > > Bye, Peter Korsgaard > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-02-04 10:10 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-02-01 9:13 [Buildroot] [PATCH] host-genimage: add host-mtools to dependency list Alexandru Ardelean 2019-02-01 11:59 ` Peter Korsgaard 2019-02-01 13:07 ` Alexandru Ardelean 2019-02-04 10:10 ` Yann E. MORIN
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox