[ci] Set r=ON where possible
#20305
[ci] Set r=ON where possible
#20305
Conversation
Fix the following warning:
```txt
math/rtools/src/RMinimizer.cxx:28:19: error: variable length arrays in C++ are a Clang extension [-Werror,-Wvla-cxx-extension]
28 | double z[size];
| ^~~~
math/rtools/src/RMinimizer.cxx:28:19: note: read of non-const variable 'size' is not allowed in a constant expression
math/rtools/src/RMinimizer.cxx:26:23: note: declared here
26 | unsigned int size = y.GetNoElements();
| ^
```
Test Results 22 files 22 suites 3d 22h 43m 26s ⏱️ Results for commit 28a5dfa. ♻️ This comment has been updated with latest results. |
guitargeek
changed the title
r=ON globally to see where we stand with Rr=ON where possible
|
Side note from:
|
|
That's a good point. Indeed I argued against continuing to develop these R bindings, but as they are already here we should at least test what we have I think. It doesn't cost much |
So far, the R bindings were only built on Ubuntu 22.04, which is becoming a bit dated already. Since the R bindings usually don't cause any trouble, it would be good to build them on all platforms where we can easily do that.
2 participants
So far, the R bindings were only built on Ubuntu 22.04, which is becoming a bit dated already.
Since the R bindings usually don't cause any trouble, it would be good to build them on all platforms where we can easily do that.