Skip to content

Valgrind option added for unittests #940

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jan 7, 2020
Merged

Valgrind option added for unittests #940

merged 1 commit into from
Jan 7, 2020

Conversation

AnttiKauppila
Copy link

Added option to select valgrind to be used with unittests

@AnttiKauppila
Copy link
Author

Mbed OS needs to be updated first

@AnttiKauppila
Copy link
Author

Python 2.7 error is "ERROR: Unknown Error: (128, 'Download failed!\nPlease try again later.')".
I cannot retrigger the build

@AnttiKauppila
Copy link
Author

@mark-edgeworth @madchutney Who is in charge of this repo? Mbed OS PR has been merged and this wouold need a test restart + review from stakeholder(s).

@mark-edgeworth
Copy link
Collaborator

The mbed build tools team manage this repo. I've triggered a rebuild as requested.

@madchutney
Copy link
Collaborator

This doesn't look like it is related to Python 2.7, this test case actually does a lot more than the Python 3 tests. It is probably this way as the extra tests also run against older Mbed versions that do not support Python 3.

I believe the failure is related to a recent problem with a redirect from mbed.org.

[mbed] Adding library "mbed" from "https://mbed.org/users/mbed_official/code/mbed/builds" at rev #3d0ef94e36ec
[mbed] Downloading library build "3d0ef94e36ec" (might take a while)
[mbed] ERROR: Unknown Error: (128, 'Download failed!\nPlease try again later.')

@AnttiKauppila
Copy link
Author

What should I do with this now?

@AnttiKauppila
Copy link
Author

It might work is mbed.org is replaced by os.mbed.org That is the place where it redirects

@madchutney madchutney merged commit db385eb into ARMmbed:master Jan 7, 2020
@AnttiKauppila AnttiKauppila deleted the valgrind_hint branch January 8, 2020 07:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants