[ovs-dev] [PATCH] Add include/odp-netlink.h to build instructions in INSTALL.Windows

Gurucharan Shetty shettyg at nicira.com
Tue Aug 5 14:46:47 UTC 2014


On Tue, Aug 5, 2014 at 12:50 AM, Nithin Raju <nithin at vmware.com> wrote:
> Since commit 837eefc7 (Do not seemingly #include Linux-specific
> headers on other platforms), openvswitch.h is being auto-generated
> as include/odp-netlink.h.
>
> On Windows, the auto-generated files are explicitly built.  In this
> change, we update the list of auto-generated files to also refer
> to include/odp-netlink.h.

Looks like, when BUILD.Windows was changed to INSTALL.Windows, it lost
some information.

You should now be doing just a 'make' (and not pick and choose
executables and files).

Would you add back the couple of lines lost from BUILD.Windows?
You will also need a Signed-off-by.
(If you prefer that I do it, let me know).

Thanks,
Guru


> ---
>  INSTALL.Windows | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/INSTALL.Windows b/INSTALL.Windows
> index b048134..e0543e7 100644
> --- a/INSTALL.Windows
> +++ b/INSTALL.Windows
> @@ -65,7 +65,8 @@ or from a distribution tar ball.
>
>  * Run make for the ported executables in the top source directory, e.g.:
>
> -  % make lib/vswitch-idl.h lib/vtep-idl.h ofproto/ipfix-entities.def
> +  % make lib/vswitch-idl.h lib/vtep-idl.h ofproto/ipfix-entities.def \
> +         include/odp-netlink.h
>    % make ovsdb/ovsdb-server.exe ovsdb/ovsdb-tool.exe ovsdb/ovsdb-client.exe \
>           utilities/ovs-vsctl.exe utilities/ovs-ofctl.exe \
>           utilities/ovs-dpctl.exe vswitchd/ovs-vswitchd.exe \
> --
> 1.9.1
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev



More information about the dev mailing list