avrdude-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[avrdude-dev] [patch #9315] Add PRUSS programmer type for xmega pdi


From: Joerg Wunsch
Subject: [avrdude-dev] [patch #9315] Add PRUSS programmer type for xmega pdi
Date: Tue, 6 Jun 2017 08:11:39 -0400 (EDT)
User-agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:49.0) Gecko/20100101 Firefox/49.0

Follow-up Comment #1, patch #9315 (project avrdude):

Basically, a good idea.

Documentation patches are missing (either to the .texi or the 
man page file, preferrably for both).

I see a couple of problems with the firmware files though:

First, we are guests on the FSF's servers, and they only accept 
opensource projects.  Thus, we are not allowed to host a 
closed-source firmware for the programmer.

Second, hardcoding a path name in the application is completely 
a "don't ever do this".  This should preferrably be handled 
using autoconf parameters, depending from --prefix.  However, 
on Windows, this is usually completely different.  See 
winconf.c for how this is implemented wrt. avrdude.conf.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?9315>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]