Kinsey Moore started a new discussion on rtemslwip/greth/include/lwipbspopts.h: 
https://gitlab.rtems.org/rtems/pkg/rtems-lwip/-/merge_requests/40#note_149323

 >  #if GRETH_DYN_ARP
 > -    #define GRETH_STATIC_ARP 0    /**< Disable static ARP */
 >      #define ARP_TMR_INTERVAL 1000 /**< ARP timer interval in ms */
 >      #define ARP_TABLE_SIZE   10   /**< Number of dynamic ARP table entries 
 > */
 > -#endif
 > -
 > -#if GRETH_STATIC_ARP
 > -    #define GRETH_DYN_ARP 0 /**< Disable dynamic ARP */
 > +#else
 >      #define ETHARP_SUPPORT_STATIC_ENTRIES \
 >    1 /**< Enable LwIP support 
 >                                                  for static ARP entries */
 >  #endif
 > +
 > +#undef LWIP_IPV6
 > +#define LWIP_IPV4 1

Instead of blindly removing or defining these, thrown a #error on `#if 
LWIP_IPV6` to warn the user that things they've configured won't work and if 
it's not set then set it to 0.

LWIP_IPV4 is already set by default in rtemslwip/include/lwipopts.h and does 
not need additional handling here, though you could detect whether the user has 
set it to 0 and throw an error.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/pkg/rtems-lwip/-/merge_requests/40#note_149323
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to