[Buildroot] [PATCH] package/mstpd: fix install path

[Buildroot] [PATCH] package/mstpd: fix install path

[Francois Perrad]

note: the current configuration works only if BR2_ROOTFS_MERGED_USR is enabled
Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
 package/mstpd/mstpd.mk | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/package/mstpd/mstpd.mk b/package/mstpd/mstpd.mk
index 907d09424..22c91a639 100644
--- a/package/mstpd/mstpd.mk
+++ b/package/mstpd/mstpd.mk
@@ -10,4 +10,7 @@ MSTPD_AUTORECONF = YES
 MSTPD_LICENSE = GPL-2.0, RSA Data Security (md5)
 MSTPD_LICENSE_FILES = LICENSE hmac_md5.c
 
+# mstpd requires that it is installed into /sbin, not /usr/sbin
+MSTPD_CONF_OPTS = --sbindir=/sbin
+
 $(eval $(autotools-package))
-- 
2.34.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/mstpd: fix install path

[Colin Foster]

On Sat, Jul 09, 2022 at 05:55:57PM +0200, Francois Perrad wrote:
> note: the current configuration works only if BR2_ROOTFS_MERGED_USR is enabled
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>

If you use these:

Tested-by: Colin Foster <colin.foster@in-advantage.com>

I'll fix up MSTPD as well, since it is currently incompatible with
busybox `pidof` and we'll be off to the races!

> ---
>  package/mstpd/mstpd.mk | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/package/mstpd/mstpd.mk b/package/mstpd/mstpd.mk
> index 907d09424..22c91a639 100644
> --- a/package/mstpd/mstpd.mk
> +++ b/package/mstpd/mstpd.mk
> @@ -10,4 +10,7 @@ MSTPD_AUTORECONF = YES
>  MSTPD_LICENSE = GPL-2.0, RSA Data Security (md5)
>  MSTPD_LICENSE_FILES = LICENSE hmac_md5.c
>  
> +# mstpd requires that it is installed into /sbin, not /usr/sbin
> +MSTPD_CONF_OPTS = --sbindir=/sbin
> +
>  $(eval $(autotools-package))
> -- 
> 2.34.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/mstpd: fix install path

[Thomas Petazzoni via buildroot]

On Sat,  9 Jul 2022 17:55:57 +0200
Francois Perrad <fperrad@gmail.com> wrote:

> note: the current configuration works only if BR2_ROOTFS_MERGED_USR is enabled
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
>  package/mstpd/mstpd.mk | 3 +++
>  1 file changed, 3 insertions(+)

Applied to master with an improved commit message and comment in the
.mk file.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] package/mstpd: fix install path

[Peter Korsgaard]

>>>>> "Francois" == Francois Perrad <fperrad@gmail.com> writes:

 > note: the current configuration works only if BR2_ROOTFS_MERGED_USR is enabled
 > Signed-off-by: Francois Perrad <francois.perrad@gadz.org>

Committed to 2022.05.x and 2022.02.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot