Update chipKIT support to compile with 2014-08-x MPIDE and build tools from Digilent #256

Closed
wants to merge 7 commits into
from

Projects

None yet

3 participants

@peplin
Contributor
peplin commented Sep 10, 2014

Don't merge this until #266 is resolved, please and thanks!

@sudar sudar added the chipKIT label Sep 10, 2014
@sej7278
Collaborator
sej7278 commented Sep 10, 2014

can we create a new branch to eventually merge this into as it seems pretty massive changes to just merge into master. would make it easier to test.

we'd have to change quite a lot of the packaging too.

oh forget that, looks like a lot of this has been merged already

@sudar
Owner
sudar commented Sep 10, 2014

oh forget that, looks like a lot of this has been merged already

@sej7278 Only changes that @peplin did for getting the travis CI working as part of #254 got merged. The commits in this pull request are not merged yet unless both the branches which @peplin used shared the same commits.

@sej7278
Collaborator
sej7278 commented Sep 10, 2014

@sudar i meant the new directory structure is already in place for the travis-ci stuff, and this PR relies on that

@peplin
Contributor
peplin commented Sep 10, 2014

This branch was based off my branch with the automated test runner, I can re-send when this is wrapped up with a different base if you'd like. It's not quite ready to merge yet.

@sudar
Owner
sudar commented Sep 10, 2014

This branch was based off my branch with the automated test runner, I can re-send when this is wrapped up with a different base if you'd like. It's not quite ready to merge yet.

I think it is fine for now. You can rebase it when you are ready to send the pull request.

@peplin
Contributor
peplin commented Sep 19, 2014

I had a chance to test this on a chipKIT Max32 tonight, and I can say all is well. I rebased this branch against master and merged in #255 as that is required to work with this version of MPIDE. I'm not in any rush as I'll just use my fork in OpenXC for now, but in case there are other chipKIT users out there - this branch works with the 2014-08-21 release of MPIDE.

@peplin peplin referenced this pull request in peplin/Arduino-Makefile Oct 28, 2014
@peplin peplin Update test suite to latest MPIDE.
Almost fixed sudar#253
but for sudar#255
43a4009
@sej7278 sej7278 added a commit to sej7278/Arduino-Makefile that referenced this pull request Jul 9, 2015
@sej7278 sej7278 Took another couple of changes from @peplin's PR #256 for chipKIT 28210c8
@sudar sudar closed this in 3dd06a6 Jul 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment