[Buildroot] [PATCH] package/nfs-utils: nfsiostat used python3, not python2

Petr Vorel petr.vorel at gmail.com
Sun Feb 6 14:34:46 UTC 2022


Hi Peter,

> >>>>> "Peter" == Peter Korsgaard <peter at korsgaard.com> writes:

> s/used/uses/ in the subject naturally.

+1

>  > nfsiostat is a python3 script, so keep it if python3 is enabled and not
>  > python2:

Good catch, thanks!
Reviewed-by: Petr Vorel <petr.vorel at gmail.com>

Kind regards,
Petr

>  > head -n 1 target/usr/sbin/nfsiostat
>  >  #!/usr/bin/python3

>  > Signed-off-by: Peter Korsgaard <peter at korsgaard.com>
>  > ---
>  >  package/nfs-utils/nfs-utils.mk | 2 +-
>  >  1 file changed, 1 insertion(+), 1 deletion(-)

>  > diff --git a/package/nfs-utils/nfs-utils.mk b/package/nfs-utils/nfs-utils.mk
>  > index 6a3a801359..44653e2707 100644
>  > --- a/package/nfs-utils/nfs-utils.mk
>  > +++ b/package/nfs-utils/nfs-utils.mk
>  > @@ -114,7 +114,7 @@ define NFS_UTILS_REMOVE_NFSIOSTAT
>  >  endef

>  >  # nfsiostat is interpreted python, so remove it unless it's in the target
>  > -NFS_UTILS_POST_INSTALL_TARGET_HOOKS += $(if $(BR2_PACKAGE_PYTHON),,NFS_UTILS_REMOVE_NFSIOSTAT)
>  > +NFS_UTILS_POST_INSTALL_TARGET_HOOKS += $(if $(BR2_PACKAGE_PYTHON3),,NFS_UTILS_REMOVE_NFSIOSTAT)

>  >  define HOST_NFS_UTILS_BUILD_CMDS
>  >  	$(MAKE) -C $(@D)/tools/rpcgen
>  > -- 
>  > 2.20.1



More information about the buildroot mailing list