Review Request 121400: Add a new macro to create VERSIONINFO resources for Windows

[Alex Merry]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121400/#review71745
-----------------------------------------------------------


What happens if you miss out an argument (ie: you put an empty string in the file)? Is that the same as not including the corresponding VALUE line in the file at all?

Also, it would be nice to be able to optionally pass the version as an argument, in case either PROJECT_VERSION isn't set for some reason or the user wants a different version for some reason. The default should still be PROJECT_VERSION, of course (but your function should check it is set and cause a fatal error otherwise).

- Alex Merry


On Dec. 9, 2014, 5:01 a.m., Nicolás Alvarez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121400/
> -----------------------------------------------------------
> 
> (Updated Dec. 9, 2014, 5:01 a.m.)
> 
> 
> Review request for Build System and KDE Frameworks.
> 
> 
> Repository: extra-cmake-modules
> 
> 
> Description
> -------
> 
> Windows has a way for executables and shared libraries (DLLs) to contain version information in a machine-readable format. This information can be seen in Explorer in the file properties (http://goo.gl/UbJGte) and also in the tooltip when hovering the file. It's also used by Windows Installer to avoid overwriting an existing DLL with an older version.
> 
> This change adds a CMake macro to generate a Windows resource script with version information passed to the macro.
> 
> The macro documentation is a single line, obviously needs work.
> 
> 
> Diffs
> -----
> 
>   modules/ECMCreateVersionResource.cmake PRE-CREATION 
>   modules/ECMVersionResource.rc.in PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/121400/diff/
> 
> 
> Testing
> -------
> 
> Tested with CMake 3.0.2 in kcoreaddons (review /r/121401), MSVC2013, Windows 7. Explorer tooltip shows description and version. File properties shows all the new information.
> 
> 
> Thanks,
> 
> Nicolás Alvarez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20141210/829c79d5/attachment.html>