[ovs-dev] [PATCH v3] datapath-windows: Set Version correctly for OVSExt
Guru Shetty
guru at ovn.org
Wed May 17 16:50:40 UTC 2017
On 17 May 2017 at 09:30, Shashank Ram <rams at vmware.com> wrote:
> - Previously, the 'Version' property passed to MSBuild
> was not being passed to the RcComplile section. To
> use the value of 'Version' property in the rc file,
> it needs to be passed.
>
> - Adds a macro to convert the Version to a string literal.
> Previously, the Version was simply being converted
> to a literal text 'Version' instead of the the version
> number passed using the 'Version' property to MSBuild.
>
> Signed-off-by: Shashank Ram <rams at vmware.com>
> Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
>
Applied to master, thank you!
> ---
> datapath-windows/ovsext/ovsext.rc | 5 +++--
> datapath-windows/ovsext/ovsext.vcxproj | 12 ++++++------
> 2 files changed, 9 insertions(+), 8 deletions(-)
>
> diff --git a/datapath-windows/ovsext/ovsext.rc b/datapath-windows/ovsext/
> ovsext.rc
> index a7cf0a7..e6334c2 100644
> --- a/datapath-windows/ovsext/ovsext.rc
> +++ b/datapath-windows/ovsext/ovsext.rc
> @@ -9,6 +9,7 @@ LANGUAGE LANG_ENGLISH, SUBLANG_ENGLISH_US
> #pragma code_page(1252)
>
> #define STR(x) #x
> +#define VER_TO_STR(v) STR(v)
> ////////////////////////////////////////////////////////////
> /////////////////
> //
> // Version
> @@ -33,12 +34,12 @@ BEGIN
> BEGIN
> VALUE "CompanyName", "The Linux Foundation (R)"
> VALUE "FileDescription", "Open vSwitch Extension"
> - VALUE "FileVersion", STR(Version)
> + VALUE "FileVersion", VER_TO_STR(VersionWithDots)
> VALUE "InternalName", "OVSExt.SYS"
> VALUE "LegalCopyright", "Licensed under the Apache License,
> Version 2.0 (the ""License"")"
> VALUE "OriginalFilename", "OVSExt.SYS"
> VALUE "ProductName", "Open vSwitch"
> - VALUE "ProductVersion", STR(Version)
> + VALUE "ProductVersion", VER_TO_STR(VersionWithDots)
> END
> END
> BLOCK "VarFileInfo"
> diff --git a/datapath-windows/ovsext/ovsext.vcxproj
> b/datapath-windows/ovsext/ovsext.vcxproj
> index ecfc0b8..cccf066 100644
> --- a/datapath-windows/ovsext/ovsext.vcxproj
> +++ b/datapath-windows/ovsext/ovsext.vcxproj
> @@ -295,12 +295,12 @@
> <ClCompile Include="Vport.c" />
> <ClCompile Include="Vxlan.c" />
> <ResourceCompile Include="ovsext.rc">
> - <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win8
> Release|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1;
> VersionWithCommas=$(Version.Replace('.',','))</PreprocessorDefinitions>
> - <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win8
> Debug|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1;
> VersionWithCommas=$(Version.Replace('.',','))</PreprocessorDefinitions>
> - <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win8.1
> Release|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1;
> VersionWithCommas=$(Version.Replace('.',','))</PreprocessorDefinitions>
> - <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win8.1
> Debug|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1;
> VersionWithCommas=$(Version.Replace('.',','))</PreprocessorDefinitions>
> - <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win10
> Release|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1;
> VersionWithCommas=$(Version.Replace('.',','))</PreprocessorDefinitions>
> - <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win10
> Debug|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1;
> VersionWithCommas=$(Version.Replace('.',','))</PreprocessorDefinitions>
> + <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win8
> Release|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1;
> VersionWithDots=$(Version);VersionWithCommas=$(Version.Replace('.',','))</
> PreprocessorDefinitions>
> + <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win8
> Debug|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1;
> VersionWithDots=$(Version);VersionWithCommas=$(Version.Replace('.',','))</
> PreprocessorDefinitions>
> + <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win8.1
> Release|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1;
> VersionWithDots=$(Version);VersionWithCommas=$(Version.Replace('.',','))</
> PreprocessorDefinitions>
> + <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win8.1
> Debug|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1;
> VersionWithDots=$(Version);VersionWithCommas=$(Version.Replace('.',','))</
> PreprocessorDefinitions>
> + <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win10
> Release|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1;
> VersionWithDots=$(Version);VersionWithCommas=$(Version.Replace('.',','))</
> PreprocessorDefinitions>
> + <PreprocessorDefinitions Condition="'$(Configuration)|$(Platform)'=='Win10
> Debug|x64'">%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1;
> VersionWithDots=$(Version);VersionWithCommas=$(Version.Replace('.',','))</
> PreprocessorDefinitions>
> </ResourceCompile>
> </ItemGroup>
> <ItemGroup>
> --
> 2.9.3.windows.2
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
More information about the dev
mailing list