-
Notifications
You must be signed in to change notification settings - Fork 3k
Correct single file excludes for some exporters #6145
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
Correct single file excludes for some exporters #6145
Conversation
/morph build |
941b0a7
to
166a463
Compare
Build : SUCCESSBuild number : 1190 Triggering tests/morph test |
@theotherjimmy as you're working on the exporters, I'm checking if this is reported already: the gnuarmeclipse creates unnecessary include file entries in the .cproject for a lot of Feature/... directories. For the target -m BLUEPILL_F103C8 for example for lorawan, mbedtls, nanostack and more. Are these features default? In the target.json they are not defined for this target, how is it determined which features to add? |
Yes.
They need to be behind |
Test : FAILUREBuild number : 990 |
/morph test |
Exporter Build : SUCCESSBuild number : 864 |
Test : FAILUREBuild number : 993 |
Build was tagged with SHA 941b0a70d971d9dd22fb3f7b012ab3b7a2397ade , however tests are looking for 166a463 @cmonr any idea ? |
@studavekar Seems similar to #6078 (comment) ? Where can I obtain sha as you did ? |
Seems like first build was before the last commit, rerunning. @studavekar Shall this be fixed (kill any running job for the PR if we run again) ? /morph build |
Build : SUCCESSBuild number : 1202 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 874 |
Test : FAILUREBuild number : 1004 |
/morph test |
Test : SUCCESSBuild number : 1009 |
Resolves #6060