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

Fix project profile parsing #3330

Merged
merged 1 commit into from
Dec 2, 2016

Conversation

theotherjimmy
Copy link
Contributor

Description

Copied from #3313

Description

  • Type: Bug
  • Priority: Minor

Bug

Target
Any

Toolchain:
GCC_ARM

Toolchain version:
Any

mbed-cli version:
0.9.10

meed-os sha:
a1c0840 Merge pull request #3227 from ARMmbed/release-candidate

Expected behavior

mbed export --profile ./mbed-os/tools/profiles/debug.json -i make_gcc_arm 

should use the debug profile when generating the Makefile.

Actual behavior

Traceback (most recent call last):
 File "/dev/mbed-os/tools/project.py", line 247, in <module>
   main()
 File "/dev/mbed-os/tools/project.py", line 239, in main
   profile = extract_profile(parser, options, toolchain_name)
 File "/dev/mbed-os/tools/options.py", line 107, in extract_profile
   contents = load(open(filename))
IOError: [Errno 2] No such file or directory: 'm'
[mbed] ERROR: "python" returned error code 1.
[mbed] ERROR: Command "python -u /dev/mbed-os/tools/project.py -i make_gcc_arm -m >MAX32550 --source . --profile mbed-os/tools/profiles/debug.json" in "/dev"

Status

READY

Todos

  • Tests

thanks @sdive!

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 29, 2016

@theotherjimmy What tests are planned ?

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph export-build

Output

mbed Build Number: 36

All exports and builds passed!

Copy link
Contributor

@bridadan bridadan left a comment

Choose a reason for hiding this comment

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

LGTM

@0xc0170 0xc0170 merged commit b08e1ec into ARMmbed:master Dec 2, 2016
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.

4 participants