Introduce option to use clang++, and option to skip apt packages#971
Introduce option to use clang++, and option to skip apt packages#971
Conversation
uweseimet
left a comment
There was a problem hiding this comment.
I did not find apt-install for clang++ in the changes. I don't think it is installed per default, but I'm not sure.
Good catch, I forgot about that. Added now. |
uweseimet
left a comment
There was a problem hiding this comment.
Are you sure that clang also installs clang++ and that the package name is actually correct? On some platforms the package name referts to llvm, which clang is part of.
|
@uweseimet I believe |
|
I don't think I'm on board with having easyinstall install clang. This is going to take extra space and extra time to install it on everyone's raspberry pi, when very very few people are using it. @uweseimet - is your intention to switch over completely to clang++? If so, then I'm OK with leaving it in. But then I would expect a PR coming to switch the Makefile over. |
|
@akuker As I already mentioned in my ticket, I'm totally fine with rejecting it. This is just a low prioriity suggestion, not more than that. |
|
As long as we provide this as "experimental" feature I agree it would make sense to install clang only when explicitly chosen. Save the majority of users the overhead and bloat of having those packags. |
|
I leave the decision to you. I don't have a strong opinion on this. I mainly created this ticket because @akuker mentioned that some users (still) have issues with compiling due to low memory, and I verified that there are no such issues with clang++. |
|
Let's do it! Let's switch to clang :-) |
|
@uweseimet - would you like to update the Makefile? Or should I? |
|
@akuker I suggest we hold off on this PR until the Makefile changes have been made. Do we want to offer gcc as a command line option for easyinstall, e.g. |
|
So since it seems we will need g++ in the Makefile for CI/CD for the time being (more research and testing required) I propose what we do in this PR, is to pivot to clang-11 as the default options in easyinstall and keep gcc in the Makefile. This gives us the benefit of the performance boost for users upgrading their RaSCSI installations, without breaking UTs and static analysis in the short term. Thoughts? |
|
Kudos, SonarCloud Quality Gate passed! |
|
Sounds good to me! |








Uh oh!
There was an error while loading. Please reload this page.