[PATCH] D27438: clang-format-vsix: add a date stamp to the VSIX version number to ensure upgradability
Antonio Maiorano via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 5 20:01:59 PST 2016
amaiorano added a comment.
In https://reviews.llvm.org/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!
https://reviews.llvm.org/D27438
More information about the cfe-commits
mailing list