Merge branch 'mikebrady:development' into development

Update RELEASENOTES-DEVELOPMENT.md

Adjust the time needed to allow the clock to settle and the lead time.

Look for 0.15 seconds of a lead time...

Flush if the first packet is less than 100 ms in the future.

Wait for 300 secodns before complaining about the age of a sync.

Send an empty timing list when a player is acquired.

Automated docker tagging & s6-overlay implementation.

Set the minimum bus mastership time to two seconds. Turn on a debug message.

Merge branch 'development' of github.com:mikebrady/shairport-sync into development

Documentation update.

Update sample dbus commands

slightly simplify function generate_preliminary_string.

Update RELEASENOTES-DEVELOPMENT.md

Update RELEASENOTES-DEVELOPMENT.md

Begin to try to figure out how to respond to remote control connections.

Update RELEASENOTES-DEVELOPMENT.md

Make debug messages easier to see on macOS Terminal.

Update RELEASENOTES-DEVELOPMENT.md

Update MOREINFO.md

Update MOREINFO.md

Update MOREINFO.md

Update MOREINFO.md

Update RELEASENOTES-DEVELOPMENT.md

Only include code to modify the RTSP buffer length if metadata support is included in the configuration.

Update BUILDFORAP2.md

Update RELEASENOTES-DEVELOPMENT.md

Merge branch 'mptre-master' into development

Thanks to Anton Lindqvist (https://github.com/mptre) for this PR.

Merge "openbsd corrections #1357" after resolving one conflict.

Update AIRPLAY2.md

Merge pull request #1356 from tecdude/patch-1

Update AIRPLAY2.md

Update RELEASENOTES-DEVELOPMENT.md

Update RELEASENOTES-DEVELOPMENT.md

Merge branch 'development' of git.aillwee.com:base/shairport-sync into development

Add update to ejurgensen's pair_ap library. Thanks as always.

Merge branch 'development' of git.aillwee.com:base/shairport-sync into development
Copy in new developments.

Merge pull request #1367 from Lorilandly/development

fixed dependency on the `xxd` tool.

Thanks. I don't know how I missed that!

fixed dependency

Merge branch 'development' of git.aillwee.com:base/shairport-sync into development

Memory leak fixes, move play lock to the first setup, hold the timing peer list until a full teardown.

Add client and server ip metadata for AP2. Remove timing peer list and DACP code only for a full AP2 teardown.

Fix another memory leak.

Add a bogus exit for AP1 to make it easier to evaluate valgrind results.

Another memory leak.

Fix a few more memory leaks

Possibly fix a few memory leaks. Add a bogus exit to allow SPS to quit at the end of a play session, to make it easier to use valgrind.

Merge branch 'development' of git.aillwee.com:base/shairport-sync into development

A few small changes.

Merge branch 'development' of https://github.com/mikebrady/shairport-sync into development

Merge.

Stop checking that the timing peers are within the same subnet.

Quieten a few debug messasges.

stop removing the play lock at the end of a play session. That's the job of the rtsp thread.

quieten a debug message

Look for at least 0.1 seconds of leadtime, ensure a master clock is at least 0.275 seconds old before use, quieten a few debug messages.

Merge branch 'development' of git.aillwee.com:base/shairport-sync into development

Just checking.

Clang Format

Add txtAirPlay data to info response. Don't pause long RTSP message reception.

Don't ask for cover art if not asking for metadata too. Duh.

Skip some invalid AAC packets, don't decode outdated frames, wait for timing information to be valid.

Wait for that mutex for longer. (Don't know why yet...)

Remember to add some exta fields later...

Stop clearing the timing peer list and stop removing the anchor clock info.

Merge pull request #1362 from mptre/mutex

Avoid a recursive mutex acquisition in the `sndio` backend.

avoid recursive mutex acquisition in sndio backend

Ending a RTSP session while running on OpenBSD using sndio backend causes the
following crash:

#0  thrkill ()
#1  0x000005208224403e in _libc_abort
#2  0x00000520821b77be in _rthread_mutex_trylock
#3  _rthread_mutex_timedlock
#4  0x0000051e0d54e2c0 in stop ()
#5  0x0000051e0d544e85 in player_thread_cleanup_handler
#6  0x0000052082243126 in _libc_pthread_exit
#7  0x000005209a158700 in sigthr_handler
#8  <signal handler called>
#9  _thread_sys_poll ()
#10 0x000005208223533e in _libc_poll_cancel
#11 0x00000520df54c9a0 in sio_psleep
#12 0x00000520df54cc1f in sio_write
#13 0x0000051e0d54e27a in play
#14 0x0000051e0d547fc0 in player_thread_func
#15 0x000005209a158cc1 in _rthread_start
#16 0x000005208223565a in __tfork_thread

The player thread is blocking inside sio_write() -> poll(2) while the thread is
being terminated. The stop routine tied to the same backend is invoked through
player_thread_cleanup_handler() which tries to acquire the mutex which it
already acquired before invoking sio_write(). Avoiding blocking writes would
require switching to async I/O which is quite an undertaking.

The fact that there's only one `struct sio_hdl *' instance in the compilation
unit sort of implies there can only be one player thread at a time. Therefore
fix the crash by only trying to acquire the mutex and continue as usual if it's
already acquired.

add missing includes on openbsd

do not pass null to printf

OpenBSD issues a warning using syslog whenever NULL is passed as a variadic
argument to the printf-like family of functions.

configure: openbsd is supported

configure: fix handling of non linux operating systems

The OS whitelist conditional is malformed due to missing test(1) commands
causing the following error to be emitted during configure while using something
other than Linux:

../configure[3340]: xopenbsd: not found

Instead, favor a switch statement which makes the line a bit shorter as well.

Also, the AC_MSG_ERROR argument must be wrapped in brackets due to presence of a
comma. Otherwise, the tail will be interpreted as an additional argument.

clang format only

Set the timing peer list as soon as a SETPEERS message is received -- don't wait until play is about to begin. Not sure if it works universally yet.

Tidy up the statistics printout table and stop outputting (non existent) nominal fps for AP2 Realtime Streams.

Use PKG_CHECK_MODULES to find libavcodec, to make it work with Fedora 35 (?)

Update AIRPLAY2.md

Change behaviour after an unrecoverable error -- if not handled, exit Shairport Sync.

log statsistics headers whenever verbosity goes on or off or whenever statistics are requested

Merge branch 'development' of git.aillwee.com:base/shairport-sync into development

Use CLOCK_MONOLITHIC_RAW to avoid NTP effects, report raw and corrected timing from audio_alsa. Clean up statistics. Stop watchdog timing out AP2 sessions. CLean up some debug messages.

Clang format

Quieten some debug messages.

Merge branch 'development' of github.com:mikebrady/shairport-sync into development
Merge a pull request and documentation update.

Merge branch 'development' of https://github.com/mikebrady/shairport-sync into development

Updated documentation.

Update RELEASENOTES-DEVELOPMENT.md

Merge branch 'development' of https://github.com/mikebrady/shairport-sync into development

update configure.ac

Merge pull request #1346 from fwcd/libplist-configure-fix

Try finding libplist as libplist-2.0 on Linux too.

Add some debug messages to help check the output rate calculations.

change stats to report non-zero if output has just opened or if there has been any discontinuity in play from previous time.

Remove the commas in the statistics logs -- easier to get into a spreadsheet.

Don't overwrite service files if they already exist.

Try finding libplist as libplist-2.0 on Linux too

Add code to check divisions and mods for a potrntial divide by zero problem. MAy cause SPS to terminate but leave a log message.

Don't display output FPS, as it's not clear how accurate it is...

Check for the stats function before allowing expanded stats info.

Add a 'stats' function to the audio back ends to replace the 'rate_info' function. Zero all except the alsa back end.

attempt to increase the precision of the FPS timing

Working to check for divide / mod by 0's

Merge branch 'development' of git.aillwee.com:base/shairport-sync into development

Merge independent changes.

Update RELEASENOTES-DEVELOPMENT.md

Update to the latest version of ejurgensen's pair_ap library. With thanks.

Update RELEASENOTES-DEVELOPMENT.md

Fix a bug by ensuring certain ap2 variables are initialised by the player thread.

Try using the monotonic clock for checking output rate...

Ensure the frame rate calculator allows for silent frames added to disable standby. Clean up outmoded and unused stuff in audio_alsa.c. More to do.

Fix a bug that was causing a compiler warniong about a potentially uninitialised variable.