Skip to content

Conversation

@ryandesign
Copy link

These changes attempt to implement the feature I requested in #65.

I made these changes many months ago, and they seemed to work at the time, but I have not looked at it since then. There has been a new version of naught released since then, and I don't know how naught changed since I wrote this.

The server example script I just committed was also written back then. I felt that naught was trying to recommend a variety of best practices, but not really showing a complete example of how to do that, so I tried to create one. I'm not confident that it is correctly exemplifying everything naught recommends, so it should be examined and fixed as needed.

@andrewrk
Copy link
Owner

This looks like a cool feature. It's a little beyond what I am able to review and test at this time. Would you be willing to rebase it against current master and then wait for one or two other people to review and test it?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not 100% sure, but I suspect the test here could be wrong.

A non defined LISTEN_FD will result in a NaN value, so still truthy value.
Maybe this should be :

if (!isNaN(listenFd)) { ..

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NaN isn't truthy.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correct. my bad.
👍

@ryandesign
Copy link
Author

Would you be willing to rebase it against current master and then wait for one or two other people to review and test it?

I'm not currently working on node stuff, so I would welcome someone else updating this.

@andrewrk
Copy link
Owner

I'm not currently working on node stuff, so I would welcome someone else updating this.

Same here. I've been working on Genesis and libsoundio.

@thelinuxlich
Copy link

So it is impossible to use naught with systemd right now?

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.

4 participants