xiaoxiang781216 commented on pull request #415: URL: https://github.com/apache/incubator-nuttx-apps/pull/415#issuecomment-702752053
> What binary compatibility? In Binary mode, it supports IPv4 now on one interface, nothing more. I mean that the new code still can read the old config after we add IPv6 support. > > As Greg said: > > > But it is not clear how to integrate IPv6 support into the binary interface without *********useless****** ********explosive****** growth since the binary form contains all possible settings. > > We can version the structure if you think it is necessary. Or the user can use the Non binary version until you make it perfect. > Yes, the version is the mininal requirement. Once we announce this feature, the user may select to use the binary format. > Now we can not change the setting at run time and that is a BIG problem. It is very easy to cherry-pick the patch once this PR get merged. I don't think it's so critical to rush it into release 10.0. Especially, this PR contain the code to save/load the user data. It's always the highest issue to ensure the use never lose their data in any situation. ---------------------------------------------------------------- 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: us...@infra.apache.org
