Fixed ChipKIT paths for avrdude

[ColinWaddell]

Hey - I was having some problems with the ChipKIT Makefile and had to update some paths in the template file. I'm using Mpide 0023-macosx-20120803. Not sure if these changes were needed because I'm using a different version of Mpdie than yourself. Thought I'd send you over my modifications none-the-less.

Hope this finds you well,
Colin

[sudar]

I don't use ChipKIT myself, so not sure if these changes are needed or not.

@peplin, Do you think these changes are needed?

[peplin]

I think they are the paths in an older version. I haven't tested the build with the latest (mpide-0023-linux32-20130817-test) but I just took a look and the Makefile has the correct path to avrdude.

[ColinWaddell]

I'll double check tomorrow evening, this may be a Mac issue. I'm running
the most recent mpide version offered on the official site, but this may
not be the most recent build available.

Cheers guys.
On Dec 2, 2013 9:21 PM, "Christopher Peplin" notifications@github.com
wrote:

I think they are the paths in an older version. I haven't tested the build
with the latest (mpide-0023-linux32-20130817-test) but I just took a look
and the Makefile has the correct path to avrdude.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/133#issuecomment-29658707
.

[peplin]

They recently moved the downloads, check out http://chipkit.net/started/ (although the version here is untested with the Makefile).

[ColinWaddell]

No worries. I'll test this against the makefile and report back. You can
ditch this pull request for now if you want.
On Dec 2, 2013 10:51 PM, "Christopher Peplin" notifications@github.com
wrote:

They recently moved the downloads, check out http://chipkit.net/started/(although the version here is untested with the Makefile).

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/133#issuecomment-29666601
.

[sudar]

No worries. I'll test this against the makefile and report back. You can ditch this pull request for now if you want.

Sure. I will wait for your feedback before closing this pull request. Report back once you had a chance to check it out.

[peplin]

I just patched up the Makefile for the latest MPIDE release in #136 - no changes required to the avrdude paths, but a few other things.

[ColinWaddell]

Just done an upstream pull of master and downloaded the latest version of mpide mpide-0023-macosx-20130715.dmg. I'm now running up against a heap of errors on make. I'm running make on the BlinkChipKIT example provided in the repo.

Here are the errors when running make for Mega architecture.

Here are the errors when running make for Uno architecture.

[sudar]

@ColinWaddell I just merged #136 to master. Will it fix the issue that you were trying to fix?

[ColinWaddell]

I'll test it when I get home this evening and report back.
On Dec 5, 2013 6:24 AM, "Sudar" notifications@github.com wrote:

@ColinWaddell https://github.com/ColinWaddell I just merged #136https://github.com/sudar/Arduino-Makefile/pull/136to master. Will it fix the issue that you were trying to fix?

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/133#issuecomment-29874433
.

[ColinWaddell]

Ok. I've been through mpide version 20131118, 20130817, 20130715 and 20120903 for the Mac and they all have a file structure which doesn't seem to agree with $AVRDUDE

The directory $(ARDUINO_DIR)/hardware/tools looks like this on the Mac

hardware
 \pic32
 \arduino
 \tools
    \avr
      \avr
      \avr-3
      \avr-4
      \bin
        \avrdude
      \doc
      \etc
        \avrdude.conf
      \include
      \info
      \lib
      \libexec
      \man
      \manual
      \revision.txt
      \share
      \uninstall
      \versions.txt    

A simple fix would be to allow AVRDUDE_DIR, AVRDUDE and AVRDUDE_CONF to be user configurable the same way the likes of MONITOR_PORT is.

Would you like me to add this into my fork/branch for you to see?

[sudar]

Would you like me to add this into my fork/branch for you to see?

Kindly do that and if @peplin agrees, then I will merge them.

[peplin]

It looks like the Linux version is the outlier, the same as with Arduino. I just submitted another pull request with a conditional check. There could be a way to share the logic between Arduino/chipKIT, but it may not be worth it.

[sudar]

Closing this, since @peplin implemented these changes in #137