Skip to content

Tools: Add Uvision6 exporter #7569

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

Merged
merged 11 commits into from
Aug 27, 2018
Merged

Conversation

theotherjimmy
Copy link
Contributor

Description

This PR adds a new exporter: uvision6. This new exporter shares an
IDE and template with the uvision5 exporter, changing the compiler
used by the exported project from Arm compiler 5 to Arm compiler 6.

Pull request type

[ ] Fix
[ ] Refactor
[ ] New target
[x] Feature
[ ] Breaking change

@deepikabhavnani
Copy link

deepikabhavnani commented Jul 20, 2018

*** Using Compiler 'V6.9', folder: 'C:\Keil_v5\ARM\ARMCLANG\Bin'
Build target '7559_export_test'
<built-in>(327): warning: In file included from...
<command line>(5): warning: ISO C99 requires whitespace after the macro name [-Wc99-extensions]
#define data-sections 1
            ^
<command line>(6): warning: ISO C99 requires whitespace after the macro name [-Wc99-extensions]
#define no-armcc-pragma-push-pop 1
          ^
<command line>(8): warning: ISO C99 requires whitespace after the macro name [-Wc99-extensions]
#define no-armcc-pragma-anon-unions 1
          ^
<command line>(8): warning: 'no' macro redefined [-Wmacro-redefined]
#define no-armcc-pragma-anon-unions 1
        ^
<command line>(6): note: previous definition is here
#define no-armcc-pragma-push-pop 1

@theotherjimmy
Copy link
Contributor Author

@deepikabhavnani Those are all technically warnings. Did the build error?

@deepikabhavnani
Copy link

Sorry missed the error in copy paste

*** Using Compiler 'V6.9', folder: 'C:\Keil_v5\ARM\ARMCLANG\Bin'
Build target '7559_export_test'
<built-in>(327): warning: In file included from...
<command line>(5): warning: ISO C99 requires whitespace after the macro name [-Wc99-extensions]
#define data-sections 1
            ^
<command line>(6): warning: ISO C99 requires whitespace after the macro name [-Wc99-extensions]
#define no-armcc-pragma-push-pop 1
          ^
<command line>(8): **error:** macro name must be an identifier
#define 0 1
        ^
<command line>(10): warning: ISO C99 requires whitespace after the macro name [-Wc99-extensions]
#define no-exceptions 1
          ^
<command line>(10): warning: 'no' macro redefined [-Wmacro-redefined]
#define no-exceptions 1
        ^
<command line>(6): note: previous definition is here
#define no-armcc-pragma-push-pop 1
        ^
<command line>(31): **error**: macro name must be an identifier
#define ... 1
        ^
<built-in>(1): **error: '**.mbed_config.h' file not found
#include ".mbed_config.h"
         ^~~~~~~~~~~~~~~~
4 warnings and 3 errors generated.

Copy link

@deepikabhavnani deepikabhavnani left a comment

Choose a reason for hiding this comment

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

Verified with M2351 👍

@cmonr
Copy link
Contributor

cmonr commented Aug 14, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Aug 14, 2018

Build : SUCCESS

Build number : 2791
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7569/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Aug 14, 2018

@adbridge
Copy link
Contributor

@theotherjimmy Looks like this has broken uvision5 :( Please investigate

@mbed-ci
Copy link

mbed-ci commented Aug 14, 2018

@cmonr
Copy link
Contributor

cmonr commented Aug 14, 2018

Test results appear to be due to NRF51 devices in a bad state. Power cycled, but not restarted since this PR needs work.

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 22, 2018

Is this intended for 5.10 (as labelled) ? in case yes, this needs update soon otherwise we shall remove the label

@cmonr cmonr added the risk: G label Aug 23, 2018
@cmonr
Copy link
Contributor

cmonr commented Aug 23, 2018

@0xc0170 It is. The internal PREQ is IOTMORF-1258

@cmonr cmonr added risk: A and removed risk: G labels Aug 24, 2018
@theotherjimmy
Copy link
Contributor Author

@adbridge Fixed. Quotes and such.

@cmonr
Copy link
Contributor

cmonr commented Aug 25, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Aug 25, 2018

Build : SUCCESS

Build number : 2904
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7569/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Aug 25, 2018

@mbed-ci
Copy link

mbed-ci commented Aug 25, 2018

@cmonr cmonr merged commit 543d6c2 into ARMmbed:master Aug 27, 2018
@cmonr cmonr removed the risk: G label Sep 2, 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.

7 participants