gdal icon indicating copy to clipboard operation
gdal copied to clipboard
verified publisher icon georust

ASan

Open frewsxcv opened this issue 4 years ago • 2 comments

  • [ ] I agree to follow the project's code of conduct.
  • [ ] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

frewsxcv avatar Feb 23 '21 13:02 frewsxcv

Why did we lose the clippy check?

i brought back the clippy check in the latest commit, if you wanna see the errors that get returned. maybe there's new rules in nightly rust? not sure

Ah, so we can't run ASan without nightly?

michaelkirk avatar Feb 23 '21 16:02 michaelkirk

Why did we lose the clippy check?

i brought back the clippy check in the latest commit, if you wanna see the errors that get returned. maybe there's new rules in nightly rust? not sure

Ah, so we can't run ASan without nightly?

yep, that's the biggest downside. all -Z compiler options require nightly. the effects of that could be minimized by locking to a specific nightly, but still not ideal

frewsxcv avatar Feb 23 '21 16:02 frewsxcv

Reenvisioned and deployed here: https://github.com/georust/gdal/pull/398

metasim avatar May 18 '23 20:05 metasim