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

[dashing backport] enable address sanitizers only on 64bit machines (#149) #162

Merged
merged 1 commit into from
Sep 20, 2019

Conversation

Karsten1987
Copy link
Collaborator

@Karsten1987 Karsten1987 commented Sep 19, 2019

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

armhf32:
Build Status

* enable address sanitizers only on 64bit machines

Signed-off-by: Karsten Knese <karsten@openrobotics.org>

* remove quotes to compare integers

Signed-off-by: Karsten Knese <karsten@openrobotics.org>
@tfoote tfoote changed the title [backport] enable address sanitizers only on 64bit machines (#149) [backport dashing] enable address sanitizers only on 64bit machines (#149) Sep 19, 2019
@tfoote tfoote changed the title [backport dashing] enable address sanitizers only on 64bit machines (#149) [dashing backport] enable address sanitizers only on 64bit machines (#149) Sep 19, 2019
@tfoote
Copy link
Contributor

tfoote commented Sep 19, 2019

@Karsten1987 can you make a release after landing this?

@Karsten1987 Karsten1987 merged commit 3fbb93b into dashing Sep 20, 2019
@delete-merged-branch delete-merged-branch bot deleted the backport_142 branch September 20, 2019 16:07
@Karsten1987
Copy link
Collaborator Author

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants