patchAVR Downloader/UploaDEr - Patches: patch #6543, serial-over-tcpip support for...

 
 

You are not allowed to post comments on this tracker with your current authentification level.

patch #6543: serial-over-tcpip support for cygwin

Submitted by:  Janos Sallai <sallai>
Submitted on:  Thu 19 Jun 2008 09:25:02 PM UTC  
 
Category: NonePriority: 5 - Normal
Status: PostponedPrivacy: Public
Assigned to: Joerg Wunsch <joerg_wunsch>Open/Closed: Open

Tue 17 Feb 2009 01:15:25 PM UTC, comment #1:

Besides mixing some unrelated (and meanwhile obsolete) patches
into the actual patch, there is one fundamental issue with
this patch: it removes the -mno-cygwin option from CFLAGS
for Cygwin builds. However, -mno-cygwin has been explicitly
added to allow Win32-API-only builds (i.e. MinGW32 builds)
even when building under Cygwin, so no dependency to
cygwin1.dll is required. As far as I know, this feature is
crucial to the way the WinAVR build environment works.

For that reason, adding a native Cygwin build should be
implemented as a configure option (like --enable-native-cygwin),
defaulting to the current way (i.e. --disable-native-cygwin).
This is an acceptance requirement.

Also, as Cygwin basically offers a Posix-compliant API, I
think it would be better to massage the build system so
native Cygwin builds just go ahead, and use ser_posix.c
directly, rather than duplicating much of its functionality
in ser_win32.c. This is not a requirement, but a wish that
allows for easier maintenance.

I'd kindly ask to submit a modified patch.

Joerg Wunsch <joerg_wunsch>
Project AdministratorIn charge of this item.
Thu 19 Jun 2008 09:25:02 PM UTC, original submission:

Attached is a patch against the current CVS head (20080619) that adds serial-over-tcpip support if avrdude is built for cygwin.

This patch is required by TinyOS users who work under cygwin and want to use the MIB600 ethernet programming board from Crossbow.

Janos Sallai <sallai>

 

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by joerg_wunsch (Posted a comment)
  • -unavailable- added by sallai (Submitted the item)
  • -unavailable- added by sallai
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 17 Feb 2009 01:15:25 PM UTCjoerg_wunschStatusNone=>Postponed
      Assigned toNone=>joerg_wunsch
    Thu 19 Jun 2008 09:25:02 PM UTCsallaiAttached File-=>Added avrdude-5.6cvs20080619-cygwin-net-serial.patch, #15878
      Carbon-Copy-=>Added sallai

    Back to the top


    Powered by Savane 3.1-cleanup