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

Modernize and Refurbish #3

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

Conversation

pajacobson
Copy link

  • Excuse the newb Ruby code in sim_test_fixture.rb
  • Updated to ceedling 0.31.1
  • Updated to use MDB

@mvandervoord
Copy link
Member

Thanks for taking a look at this. I often forget about these little peripheral projects.

To simplify support for this one, we could remove all of Ceedling, etc from the repo itself and update this project.yml file to use the gem version of ceedling.

@pajacobson
Copy link
Author

Thanks for taking a look at this. I often forget about these little peripheral projects.

To simplify support for this one, we could remove all of Ceedling, etc from the repo itself and update this project.yml file to use the gem version of ceedling.

I removed the vendor folder and ceedling.
project.yml is new, and I've tried to make minimal changes to the default.

I finally figured out how to make the throwtheswitch.org/build/cross code work, so the sim_test_fixture has been updated to incorporate that. For some reason the PIC24 isn't triggering the breakpoint on the UnityHelperDeadLoop.

I've changed the "loop" to while(0){}; because the XC16 code hangs up otherwise.
PIC32/XC32 triggers the breakpoint OK with this arrangement so there appears to be correct behaviour as far as I've tested.

sim_test_fixture intercepts STDOUT and STDERR.
MDB displays errors relating to Java logr and NetBeans preferences every time it's launched.
That is super annoying and pointless so the code is filtering those from STDERR output.

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.

2 participants