clang-format-vsix: add a date stamp to the VSIX version number to ensure upgradability

[amaiorano]

Summary

Presently, the version number of the VSIX matches the LLVM version number. However, as this number doesn't change often, it means that as we release new versions of this VSIX, it will have the same version number, which means users must first uninstall the old version before installing the new one. With this change, we generate a 4th part to the version number that is a date stamp (year, month, day); for example: 4.0.0.161203.

[hans]

When building the snapshots and releases, I usually put the SVN revision number in there to avoid this same problem: http://llvm-cs.pcc.me.uk/utils/release/build_llvm_package.bat#25

Your patch seems like a good default though. Should we put the hour and minute in there as well, in case one builds multiple versions the same day?

[amaiorano]

In D27438#614219, @hans wrote:

When building the snapshots and releases, I usually put the SVN revision number in there to avoid this same problem: http://llvm-cs.pcc.me.uk/utils/release/build_llvm_package.bat#25

Ah, right, I was wondering how that worked. Well, if you don't think it's necessary, we don't have to do it. I didn't realize we had this script to build the official VSIX. Do you think you'd switch to using this method? Or would you prefer that the official releases be bound to the SVN rev number?

Your patch seems like a good default though. Should we put the hour and minute in there as well, in case one builds multiple versions the same day?

If you still want this change, I can definitely add the hour and minute. Let me know!

[hans]

In D27438#614223, @amaiorano wrote:

In D27438#614219, @hans wrote:

When building the snapshots and releases, I usually put the SVN revision number in there to avoid this same problem: http://llvm-cs.pcc.me.uk/utils/release/build_llvm_package.bat#25

Ah, right, I was wondering how that worked. Well, if you don't think it's necessary, we don't have to do it. I didn't realize we had this script to build the official VSIX. Do you think you'd switch to using this method? Or would you prefer that the official releases be bound to the SVN rev number?

I'd prefer to keep the official releases on the SVN rev.

Your patch seems like a good default though. Should we put the hour and minute in there as well, in case one builds multiple versions the same day?

If you still want this change, I can definitely add the hour and minute. Let me know!

Yes, I think this is still a worthwhile change. Let's add the hour and minute and get it landed.

[amaiorano]

Added hour and minute.

[amaiorano]

Yes, I think this is still a worthwhile change. Let's add the hour and minute and get it landed.

Alright, added hour and minute. How do we land this change? Is this something I can now do myself? Or would you need to commit this for me again?

I have a couple of more interesting changes coming. Perhaps it would make sense to grant me commit rights?

[hans]

In D27438#615396, @amaiorano wrote:

Yes, I think this is still a worthwhile change. Let's add the hour and minute and get it landed.

Alright, added hour and minute. How do we land this change? Is this something I can now do myself? Or would you need to commit this for me again?

I have a couple of more interesting changes coming. Perhaps it would make sense to grant me commit rights?

It's worth a shot :-) Follow the instructions at http://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access to ask Chris for commit access.

cfe/trunk/tools/clang-format-vs/CMakeLists.txt

 option(BUILD_CLANG_FORMAT_VS_PLUGIN "Build clang-format VS plugin" OFF)
 if (BUILD_CLANG_FORMAT_VS_PLUGIN)
   add_custom_target(clang_format_exe_for_vsix
       ${CMAKE_COMMAND} -E copy_if_different
       "${LLVM_TOOLS_BINARY_DIR}/clang-format.exe"
       "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat/clang-format.exe"
       DEPENDS clang-format)
 
   add_custom_target(clang_format_license
       ${CMAKE_COMMAND} -E copy_if_different
       "${CLANG_SOURCE_DIR}/LICENSE.TXT"
       "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat/license.txt")
 
+  # Build number added to Clang version to ensure that new VSIX can be upgraded
+  string(TIMESTAMP CLANG_FORMAT_VSIX_BUILD %y%m%d%H%M UTC)
+
   if (NOT CLANG_FORMAT_VS_VERSION)
-    set(CLANG_FORMAT_VS_VERSION "${LLVM_VERSION_MAJOR}.${LLVM_VERSION_MINOR}.${LLVM_VERSION_PATCH}")
+    set(CLANG_FORMAT_VS_VERSION "${LLVM_VERSION_MAJOR}.${LLVM_VERSION_MINOR}.${LLVM_VERSION_PATCH}.${CLANG_FORMAT_VSIX_BUILD}")
   endif()
 
   configure_file("source.extension.vsixmanifest.in"
       "${CMAKE_CURRENT_SOURCE_DIR}/ClangFormat/source.extension.vsixmanifest")
 
   find_program(NUGET_EXE nuget PATHS ${NUGET_EXE_DIR})
   if (NOT NUGET_EXE)
     message(FATAL_ERROR "Could not find nuget.exe. Download from https://www.nuget.org/nuget.exe"