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 stale.yaml

Set the limit to 45 days

Update RELEASENOTES.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.

Fix build with automake 1.16.5. Thanks for Fabrice Fontaine for this.

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.

Merge pull request #1326 from a-x-/patch-2

typo(MQTT): Please not -> Please note

typo(MQTT): Please not -> Please note

Update RELEASENOTES-DEVELOPMENT.md

Merge pull request #1314 from ffontaine/development

configure.ac: fix build with automake 1.16.5

Many thanks!

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

Forgot some online documentation updates.

slight tidying up

Update BUILDFORAP2.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update RELEASENOTES-DEVELOPMENT.md

Return 501 "Not Implemented" for unexpected RTSP commands -- it seems to work better. The old code, 400, didn't seem to get the required response.

configure.ac: fix build with automake 1.16.5

Fix the following build failure with convolution raised since automake
1.16.5 and
https://git.savannah.gnu.org/cgit/automake.git/commit/?id=f4a3a70f69e1dbccb6578f39ef47835098a04624:

configure.ac:305: error: AM_INIT_AUTOMAKE expanded multiple times
/home/giuliobenetti/autobuild/run/instance-1/output-1/host/share/aclocal-1.16/init.m4:29: AM_INIT_AUTOMAKE is expanded from...
configure.ac:6: the top level
/home/giuliobenetti/autobuild/run/instance-1/output-1/host/share/aclocal-1.16/init.m4:29: AM_INIT_AUTOMAKE is expanded from...
configure.ac:305: the top level

Fixes:
 - http://autobuild.buildroot.org/results/464148bdccb705d8992dc860262bfdeb01b7e2a1

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Update BUILDFORAP1.md

Update README.md

Update README.md