search

maximkulkin / esp-homekit

Apple HomeKit accessory server library for ESP-OPEN-RTOS
MIT License
1.11k stars 170 forks source link

posix implementation #22

Closed aep closed 5 years ago

aep commented 6 years ago

obviously that can't be merged. It'll break the esp build. let's discuss if you want to merge this at all. We'll clean it up then.

The worst parts we need to talk about:

  1. mdnsresponder is a half rewrite because low level network works different on posix
  2. queue.h: i hacked together an API compatible queue thing. This works fine because..
  3. i made it all single threaded. Which is fine i guess.
  4. storage is a nasty hack
  5. i dont understand how wolfssl works. i had to include options.h in user-settings.h, but you didn't

it's not actually that bad if we move the reimplementation of lwip api into separate files i guess. your call.

closes #20

maximkulkin commented 6 years ago

It will take me some time to review this and articulate how I see a best solution for this. Writing this just to let you know that I'm not ignoring it.