Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Joan, with your review and approval, V74 is ready to merge to the master branch. I plan to do this next week (Feb).
This release contains the following changes:
Mea culpa, this change initializes the parameter for setsockopt() to enable address reuse. Issues Delay on reboot/shutdown #274 and restarting pigpiod result in "bind to port 8888 failed (Address already in use)" #298 are the targets of this change. I did not hear back from the authors of those issues but my own testing shows this to be an intermittent problem that is fixed.
New hardware revision decode scheme addresses issues patch for pigpiod to work with 64bit gentoo #266 and Could not detect Raspberry Pi 3B+ on linux 4.19 64-bit #316. This change was tested by the authors of those issues and reported to work on 64-bit OS. My own testing found an issue with a legacy revision code and this was subsequently fixed. My testing covered all generations of raspberry pi except Pi4 (I don't own this hw, yet). Mostly for this reason, I'd like to keep this PR open until someone can verify on a Pi4. User level debug messages were added that will enable a quick verification of the failure signature should a user encounter a problem.
A 'cherry-picked' commit from PR #223: Fix waveform length when merging pulses #225 that fixes issue Incorrect waveform length when using multiple invocations of "pigs wvag ..." #223. All tests pass.