[PATCH] D94941: Add minor version to libclang.so and libclang-cpp.so SONAME

[Dimitry Andric via Phabricator via cfe-commits]

dim added a comment.

@sylvestre.ledru removed the minor version from the binary (on purpose, I think?) in rGa8b717fda42294d1c8e1f05d71280503e5839f14 <https://reviews.llvm.org/rGa8b717fda42294d1c8e1f05d71280503e5839f14>:

  commit a8b717fda42294d1c8e1f05d71280503e5839f14
  Author: Sylvestre Ledru <sylvestre at debian.org>
  Date:   Thu Mar 29 10:05:46 2018 +0000
  
      Rename clang link from clang-X.Y to clang-X
  
      Summary:
      As we are only doing X.0.Z releases (not using the minor version), there is no need to keep -X.Y in the version.
      So, instead, I propose the following:
      Instead of having clang-7.0 in bin/, we will have clang-7
  
      Since also matches was gcc is doing.
  
      Reviewers: tstellar, dlj, dim, hans
  
      Reviewed By: dim, hans
  
      Subscribers: dim, mgorny, cfe-commits
  
      Differential Revision: https://reviews.llvm.org/D41808
  
      llvm-svn: 328769

But this seems to have been mainly about the clang executable that gets installed into `$prefix/bin`, i.e. `clang-11` instead of `clang-11.1`.

I think this doesn't matter for the clang executable here, since nobody uses its ABI. But indeed for the shared library it looks like a good solution, if you would want to install both versions side-by-side.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94941/new/

https://reviews.llvm.org/D94941