Skip to content

Conversation

orenc17
Copy link
Contributor

@orenc17 orenc17 commented Dec 4, 2018

Description

Don't generate files if a test does not contain additional partitions

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the reason for removing this file as part of this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when the tool scans the psa test folder it should not generate a psa_setup.c file for tests that do not declare new partition

so this file is not needed anymore

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 6, 2018

This needs a rebase now.

@alzix Rereview?

@cmonr
Copy link
Contributor

cmonr commented Dec 6, 2018

Also, is this targeting RC3?

* Don't generate files if a test does not contain additional partitions
@orenc17
Copy link
Contributor Author

orenc17 commented Dec 6, 2018

rebased, it could be in RC3
its not that important

@cmonr
Copy link
Contributor

cmonr commented Dec 6, 2018

CI started

@mbed-ci
Copy link

mbed-ci commented Dec 7, 2018

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

@cmonr cmonr merged commit 035e8e0 into ARMmbed:master Dec 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants