[PATCH] D138632: [CMake]Allow user specified CPack Options
Hans Wennborg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 24 01:20:20 PST 2022
hans added a comment.
Allowing developers to override the variables sgtm.
I'm not sure about changing the default dir. Is it a common use case to have multiple versions installed? Users who want to can already select a different dir when installing. Also I think the installer will bring up a "Do you want to uninstall the old version before installing the new one", even if the install dir is changed. Having a default install dir without the version also makes it a little easier for simple scripts to find it.
But overall I don't feel strongly about this. Changing the dir probably shouldn't break anything, but I also don't know if there are real benefits?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138632/new/
https://reviews.llvm.org/D138632
More information about the llvm-commits
mailing list