This resolves Bugzilla bug 15493.
Looks good. Although I could accept this, excuse me to sit here to listen in more opinions.
I wish this logic into llvm_add_library(). Versioning may be general among shared libs. I will do in future.
FYI, autoconf's --enable-shared emits the library like libLLVM-3.6.0svn.so.
LIBRARYNAME = LLVM-$(LLVM_VERSION_MAJOR).$(LLVM_VERSION_MINOR)$(LLVM_VERSION_SUFFIX) LIBRARYALIASNAME = LLVM-$(LLVMVersion)
LLVMVersion := 3.6.0svn LLVM_VERSION_MAJOR := 3 LLVM_VERSION_MINOR := 6 LLVM_VERSION_PATCH := 0 LLVM_VERSION_SUFFIX := svn
To which way shall we follow?
I think you are right, and that is a better way to handle this problem. I'll work on updated patches. I'll also add the LLVM_VERSION_SUFFIX to match the autotools implementation so that we have parity.