Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add freebsd rc script #15

Open
wants to merge 182 commits into
base: master
Choose a base branch
from
Open

Add freebsd rc script #15

wants to merge 182 commits into from

Conversation

kraduk
Copy link

@kraduk kraduk commented May 9, 2016

Note: i am not using the daemon flag as this doesnt appear to work, and the process just exits without forking to the background. So for the time being im running it in the foreground and utilizing the base system daemon command.

And re-organize the sources accordingly.

There is no reason to make this LIFX specific.

--HG--
rename : core/lifxd.c => core/lightsd.c
rename : core/lifxd.h => core/lightsd.h
rename : core/broadcast.c => lifx/broadcast.c
rename : core/broadcast.h => lifx/broadcast.h
rename : core/bulb.c => lifx/bulb.c
rename : core/bulb.h => lifx/bulb.h
rename : core/gateway.c => lifx/gateway.c
rename : core/gateway.h => lifx/gateway.h
rename : core/timer.c => lifx/timer.c
rename : core/timer.h => lifx/timer.h
rename : core/wire_proto.c => lifx/wire_proto.c
rename : core/wire_proto.h => lifx/wire_proto.h
This avoid stupid infinite loops on ENETUNREACH for example.
Use the j modifier when printing 64bits numbers.
Just call the callback for the discovery timeout event right away, it
starts the discovery and setup everything already...
A quick look around showed me that this project can perfectly run on iOS
and Android and it changed my perspectives.
…eway

lgtd_gateway_handle_light_status is called per bulb and not per gateway
and we trigger the refresh of all the bulbs on the gateway from this
function.

This means that we could potentially enqueue multiple GET_LIGHT_STATE
packets for all bulbs on the same gateway within a single libevent
iteration.

With this changeset each gateway now keep track of when the last
GET_LIGHT_STATE for all bulbs was sent. If it's more recent than when
the last packet was sent then we know that one has already been
enqueued.
Huuu we do need this file on linux, and I do need to step continuous
testing.
Fix build warnings on apple/clang.
Readings of lifx-gem & LIFXKit revealed the actual purpose of this
field.
We actually need that to support tagging.
We'll need this for debugging soon enough and to return real responses
as well.
By using intmax_t and %jd in lgtd_isotime_now.
This is will be needed to support tagging.
lopter and others added 26 commits November 7, 2015 23:26
This should significantly improve both latency and throughput.
This finishes what started the previous changeset: mock the wire_proto
module and properly do unit testing.
- properly set res required;
- properly listen on each gateway's socket;
- improved traffic logging.
Thanks a lot to /u/nullcompany for his feedback about the protocol
documentation:

https://www.reddit.com/r/lifx/comments/3s82yy/restrict_from_the_cloud/cxfnxlf
It now handles arbitrarily large and partial responses properly.
After all people who don't use apt-get will figure it out...
@lopter
Copy link
Owner

lopter commented May 10, 2016

Thank you @kraduk, I'll definitely add that in the coming releases, I also opened GH-16 about the broken -d option, I'll look into a fix for that. I have a bunch of flights coming up in June that are gonna be good for that.

I'm also gonna try to setup an automatic build on FreeBSD.

@kraduk
Copy link
Author

kraduk commented May 10, 2016

like I say its not a major problem, and as it stays in the forground you
get the added bonus of automated restarts via the the daemon command. You
wouldn't get this if you forked into the background.

On 10 May 2016 at 07:35, Louis Opter notifications@github.com wrote:

Thank you @kraduk https://github.com/kraduk, I'll definitely add that
in the coming releases, I also opened GH-16
#16 about the broken -d option,
I'll look into a fix for that. I have a bunch of flights coming up in June
that are gonna be good for that.

I'm also gonna try to setup an automatic build on FreeBSD.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#15 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants