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 a Unix socket connection #591

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

Add a Unix socket connection #591

wants to merge 17 commits into from

Conversation

smurfix
Copy link

@smurfix smurfix commented Dec 5, 2023

Unix-domain sockets have less overhead than TCP.
They also tend to work better on a system that's built without networking. ;-)

pemensik and others added 17 commits January 31, 2021 21:16
Fedora uses /usr/lib64 on 64-bit systems. Allow easy configuration in
cmake to install to correct directories.
This resolves joan2937#455 with the discussion with @guymcswain that
gpioCfgSetInternals() should return an error if gpioInitialise()
has been called previously to report that it's no longer possible,
for example, to disable signal handling. If successful it returns
0 as before. Tested it by calling gpioCfgSetInternals, gpioInitialise,
gpioCfgSetInternals and reports 0 and -32 which looks OK.
Fixed the documentation that the command set_PWM_range() actually
returns the actual range. This resolves joan2937#453.
gpioCfgSetInternals now returns PI_INITIALISED after gpioInitialise
Allow building on 64-bit system using lib64
Locking the socket interface prevents APIs running concurrently.
This is problematic when some API are long runnnig and therefore
effectively block other API threads from running.

This reverts commit 9dcea25.
Unix-domain sockets have less overhead than TCP.
@smurfix
Copy link
Author

smurfix commented Dec 5, 2023

Rebased onto develop branch.

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

6 participants