Changes between Initial Version and Version 1 of Ticket #7972, comment 10


Ignore:
Timestamp:
Mar 24, 2016, 4:29:18 PM (8 years ago)
Author:
ryandesign (Ryan Carsten Schmidt)
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #7972, comment 10

    initial v1  
    55The suggested patch doesn't work because the case statement doesn't have the details needed to match.  The "configure.status" file pointed me at why:
    66
     7{{{
    78s,@target@,i386-apple-darwin8.7.1,;t t
    89s,@target_cpu@,i386,;t t
    910s,@target_vendor@,apple,;t t
    1011s,@target_os@,darwin8.7.1,;t t
     12}}}
    1113
    1214The case is comparing "target_os" which doesn't have CPU type as part of it.
    … …  
    1416So building off of the "solaris" case later in the original configure.in file from libnet, I added an additional case within the darwin section to check "target".  I guess it is debatable if the case check should just check against 'target_cpu'...
    1517
    16 --------------------------
    17 
     18{{{
    1819*darwin*)
    1920    AC_DEFINE(HAVE_SOCKADDR_SA_LEN)
    … …  
    3637
    3738    ;;
    38 
    39 -----------------------------------
     39}}}
    4040
    4141Not being really up on the use of case, I wasn't sure I needed the match all at the end but put it in anyways to be safe.