On Tue, Jul 17, 2018, 9:31 AM Mats Wichmann <m...@wichmann.us> wrote:
> On 07/16/2018 03:27 PM, Ignacio Ocampo wrote: > > Great, let me switch to the master branch and make the adjustments to fix > > the build for OSX. > > > > Regarding the network monitoring improvement suggested from Gregg and > other > > improvements, I don't understand, I thought it was enough compile, pass > the > > tests and build, isn't? > > There are dependencies on operating system services in various parts of > the code, some smaller, some larger. There are some efforts at these > for OSX and iOS "from before", but they may or may not be correct and/or > up to date, and, most particularly, there may or may not actually be > good tests which confirm the behavior; the days when the Mac was > considered a relatively supported platform are "before my time", and the > technologies covered by iotivity have evolved quite a bit over time. So, > at the very best, expect what we might call bit rot, at worst new stuff > that needs writing. Gregg seems to have a lot better handle on what > might be needed! > The network monitoring stuff is an implementation detail; it is not required by the spec, AFAIK. Consider the dynamics. Network connections come and go. You have an OCF client on your mobile and you move in and out of network range. That means you need to e.g. fix up your sockets. I believe that's what the network monitoring stuff is for but I was not involved in writing it so there may be more or less to it. In any case, you can run iotivity without network monitoring. As an example, I have three wifi networks, my usual one, a hub, and a raspberry pi configured as an access point. For testing I switch around. Without the network monitoring code I have to restart stuff. There are other platform-specific bits. Poke around in https://github.com/iotivity/iotivity/tree/master/resource/c_common. Hth > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#9781): https://lists.iotivity.org/g/iotivity-dev/message/9781 Mute This Topic: https://lists.iotivity.org/mt/23534908/21656 Group Owner: iotivity-dev+ow...@lists.iotivity.org Unsubscribe: https://lists.iotivity.org/g/iotivity-dev/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
