[PATCH Xenial/Yakkety v2] UBUNTU: [Debian] Install lsvmbus in cloud tools

Andy Whitcroft apw at canonical.com
Fri May 27 12:21:45 UTC 2016


On Wed, May 25, 2016 at 10:17:02AM -0600, Tim Gardner wrote:
> BugLink: http://bugs.launchpad.net/bugs/1585311
> 
> Signed-off-by: Tim Gardner <tim.gardner at canonical.com>
> ---
> 
> v2 - changed the commit subject a little
> 
>  debian/rules.d/2-binary-arch.mk  | 3 +++
>  debian/rules.d/3-binary-indep.mk | 1 +
>  2 files changed, 4 insertions(+)
> 
> diff --git a/debian/rules.d/2-binary-arch.mk b/debian/rules.d/2-binary-arch.mk
> index 0e7fedb..8ce1fa8 100644
> --- a/debian/rules.d/2-binary-arch.mk
> +++ b/debian/rules.d/2-binary-arch.mk
> @@ -391,6 +391,7 @@ ifeq ($(do_tools_hyperv),true)
>  	$(LN) ../../$(src_pkg_name)-tools-$(abi_release)/hv_kvp_daemon $(cloudpkgdir)/usr/lib/linux-tools/$(abi_release)-$*
>  	$(LN) ../../$(src_pkg_name)-tools-$(abi_release)/hv_vss_daemon $(cloudpkgdir)/usr/lib/linux-tools/$(abi_release)-$*
>  	$(LN) ../../$(src_pkg_name)-tools-$(abi_release)/hv_fcopy_daemon $(cloudpkgdir)/usr/lib/linux-tools/$(abi_release)-$*
> +	$(LN) ../../$(src_pkg_name)-tools-$(abi_release)/lsvmbus $(cloudpkgdir)/usr/lib/linux-tools/$(abi_release)-$*
>  endif
>  endif
>  
> @@ -669,6 +670,8 @@ ifeq ($(do_tools_hyperv),true)
>  		$(cloudpkgdir)/usr/lib/$(src_pkg_name)-tools-$(abi_release)
>  	install -m755 $(builddirpa)/tools/hv/hv_fcopy_daemon \
>  		$(cloudpkgdir)/usr/lib/$(src_pkg_name)-tools-$(abi_release)
> +	install -m755 $(builddirpa)/tools/hv/lsvmbus \
> +		$(cloudpkgdir)/usr/lib/$(src_pkg_name)-tools-$(abi_release)
>  endif
>  endif
>  
> diff --git a/debian/rules.d/3-binary-indep.mk b/debian/rules.d/3-binary-indep.mk
> index 29cc541..32bc90c 100644
> --- a/debian/rules.d/3-binary-indep.mk
> +++ b/debian/rules.d/3-binary-indep.mk
> @@ -127,6 +127,7 @@ ifeq ($(do_tools_common),true)
>  	install -m755 debian/tools/generic $(cloudsbin)/hv_kvp_daemon
>  	install -m755 debian/tools/generic $(cloudsbin)/hv_vss_daemon
>  	install -m755 debian/tools/generic $(cloudsbin)/hv_fcopy_daemon
> +	install -m755 debian/tools/generic $(cloudsbin)/lsvmbus
>  	install -m755 debian/cloud-tools/hv_get_dhcp_info $(cloudsbin)
>  	install -m755 debian/cloud-tools/hv_get_dns_info $(cloudsbin)
>  	install -m755 debian/cloud-tools/hv_set_ifconfig $(cloudsbin)
> -- 

There is nothing wrong with the patch per-see.  However, this is a
python2 thing and we should be seeing if it can be run with python3 at
least.  Also it should have a manual page if it is going to be
installed.

/me goes test whether it will work as python3.

-apw




More information about the kernel-team mailing list