You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
to run sanitized builds for gcc/clang under linux/macOS in CI
make these sanitizer options global (not just for libmamba, but for the whole project) - there is a C++ code in micromamba and libmambapy as well, so we might have some bugs there as well
Just having these options unfortunately doesn't prevent us from bugs and they are much more difficult to fix over time.
The text was updated successfully, but these errors were encountered:
@jjerphan I actually think this change is super beneficial in the long run.
And it will probably require deep level of understanding the project, because bugs might be everywhere in the repo.
So, if someone wants to work on this, it would be great.
Note that this was already the intent for a while but the blocking issue is that sanitizers will either not work or report false-positives because of dependencies not being built with sanitizers or with debug info. Our dependencies come from conda packages which are exclusively "release" kind of builds so that doesnt work. What would work is to have a separate set of builds not using conda packages, with dependencies built from sources with sanitizers enabled, therefore probably vcpkg-based. That can also lead to an invisible divergence between how conda dependencies are built and vcpkgs. So the other solution is to have sanitizer builds of conda packages (or maybe just debug would work? not sure on windows). Both solutions are not really trivial.
We have options to enable sanitizers in
libmamba
.It would be nice:
libmamba
, but for the whole project) - there is a C++ code inmicromamba
andlibmambapy
as well, so we might have some bugs there as wellJust having these options unfortunately doesn't prevent us from bugs and they are much more difficult to fix over time.
The text was updated successfully, but these errors were encountered: