[GitHub] [incubator-nuttx] anchao commented on pull request #2415: igmp: bypass MULTICAST_LOOP/TTL options

[GitBox <gi...@apache.org>]

anchao commented on pull request #2415:
URL: https://github.com/apache/incubator-nuttx/pull/2415#issuecomment-735241753


   > @xiaoxiang781216 so Xiaomi is using some ported lib that needs this, and the solution is to fake it? Why not do the right thing and implement it? Or at a minimum Kconfig it defaulted to error-ing out, with a Knob to fake the result? Things like this as landmines that can cause huge time syncs for the next poor person that steps on it.
   > 
   > @patacongo @antmerlino - any opinion on this?
   
   Hi @davids5 ,
   
   Could you please remove the nouns like xiaomi? It makes me feel uncomfortable, this is just a technical discussion, do not rise to benefit, back to the PR, if the implementation of these options completely block the results of application, I will definitely consider the priority implementation before raise this PR. But in fact, TTL and LOOP are not mandatory parts in use of IGMP multicast case. Of course, I totally agree with your insistence, I will consider implement this feature, please keep this PR, thanks.
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org