Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add most recent Ubuntu LTS (20.04) to our automated testing #8

Open
daverodgman opened this issue Feb 15, 2021 · 2 comments · May be fixed by #32
Open

Add most recent Ubuntu LTS (20.04) to our automated testing #8

daverodgman opened this issue Feb 15, 2021 · 2 comments · May be fixed by #32
Labels
enhancement New feature or request needs: preceding PR Product Backlog size-m Estimated task size: medium (~1w)

Comments

@daverodgman
Copy link
Contributor

Add support for most recent Ubuntu LTS - currently this is 20.04 (and the next will be 22.04). We could consider adding e.g. 21.04 (or both) but probably should always aim to have the most recent LTS in our testing.

@gilles-peskine-arm
Copy link
Contributor

Clang's UBsan in Ubuntu 20.04 catches some mistakes that we tend to make, and that nothing on the current CI with older Clang catches. Specifically, more than once, I've seen that an Asan+UBsan build passed on the CI but fails on my Ubuntu 20.04 machine because some code does p + x with p==NULL and x==0, which is harmless in practice but technical undefined behavior.

@gilles-peskine-arm gilles-peskine-arm transferred this issue from another repository Sep 3, 2021
@daverodgman daverodgman transferred this issue from another repository Oct 8, 2021
@daverodgman daverodgman added the enhancement New feature or request label Oct 8, 2021
@gilles-peskine-arm gilles-peskine-arm added size-m Estimated task size: medium (~1w) Product Backlog labels Oct 8, 2021
@minosgalanakis minosgalanakis linked a pull request Jan 18, 2022 that will close this issue
4 tasks
@minosgalanakis
Copy link
Contributor

As part of the work needed for PR31 I have to enable testing for 20.04.

Unfortunately it did not pass all tests, and some dependencies have been identified. The linked PR is not to be merged until the tasks identified at is are addressed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs: preceding PR Product Backlog size-m Estimated task size: medium (~1w)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants