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

Some fixes for build scripts #151

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

hirooih
Copy link
Contributor

@hirooih hirooih commented Oct 19, 2021

These are last fixes I found during my porting work (#148).

  • run_*.py: check the return value from verify_benchmark()
    • decode_results() gets the return value, but it has never been used.
  • build_all.py: catch a general Exception on config file
  • build_all.py: fix log-output format
  • .gitignore: add some common entries

decode_results() gets the return value, but it has never been used.

- run_gdbserver_sim.py
- run_mac.py
- run_native.py
- run_stm32f4-discovery.py
- During loading a config file, various exceptions may occur.
  "PermissionError" is replaced by "Exception".
- add missing formatting specifier 'f' on the error message.
- suppress the output of unnecessary blank lines
- output a blank line after each group compiled or linked
.venv: a commion directory for Python virtual environment
.vscode: configuration file directory for VS Code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant