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 missing format string argument #1693

Merged
merged 1 commit into from
Jun 20, 2023

Conversation

michieldwitte
Copy link

Adds a missing format string argument.

@mikebrady mikebrady merged commit 2f52e69 into mikebrady:development Jun 20, 2023
9 checks passed
@mikebrady
Copy link
Owner

Many thanks!

mtremer pushed a commit to ipfire/ipfire-2.x that referenced this pull request Sep 3, 2024
- Update from version 4.3.2 to 4.3.4
- Update of rootfile
- Changelog is only defined for 4.3, 4.2 etc so the below changelog is for all of 4.3
   Cannot determine which things were alreday fixed in 4.3.2 and earlier and which are
   from 4.3.3 onwards.
    4.3
**Security Updates**
	* A crashing bug in NQPTP has been fixed.
	* The communications protocol used between NQPTP and Shairport Sync has been
	  revised and made more resilient to attempted misuse.
	* In Linux systems, NQPTP no longer runs as `root` -- instead it runs as the
	  restriced user `nqptp`, with access to ports 319 and 320 set by the installer
	  via the `setcap` utility.
**Enhancements**
	* A new volume control profile called `dasl-tapered` has been added in which
	  halving the volume control setting halves the output level.
	  For example, moving the volume slider from full to half reduces the output
	  level by 10dB, which roughly corresponds with a perceived halving of the audio
	  volume level.
	  Moving the volume slider from half to a quarter reduces the output level by a
	  a further 10dB.
	  The tapering rate is slightly modified at the lower end of the range if the
	  device's attenuation range is restricted (less than about 55dB).
	  To activate the `dasl-tapered` profile, set the `volume_control_profile` to
	  `"dasl_tapered"` in the configuration file and restart Shairport Sync.
	  Many thanks to David Leibovic, aka [dasl-](https://github.com/dasl-), for this.
	* On graceful shutdown, an `active_end` signal should now be generated if the
	  system was in the active state. Addresses issue
	  [#1647](mikebrady/shairport-sync#1647). Thanks to
	  [Tucker Kern](https://github.com/mill1000) for raising the issue.
**Bug Fixes**
	* Fixed a bug that causes the Docker image to crash occasionally when OwnTone
	  interrupted an existing iOS session. Thanks to
	  [aaronk6](https://github.com/aaronk6) for the report.
	* Fixed a cross-compliation error caused by not looking for the correct version
	  of the `ar` tool. The fix was to substitute the correct version during the
	  `autoreconf` phase. Thanks to
	  [sternenseemann](https://github.com/sternenseemann) for raising the
	  [issue](mikebrady/shairport-sync#1705) and the
	  [PR](mikebrady/shairport-sync#1706) containing the fix.
	* Updated the mDNS strings for the Classic AirPlay feature of AP2, so that it
	  does not appear to provide MFi authentication. Addresses
	  [this discussion](mikebrady/shairport-sync#1691).
	* Always uses a revision number of 1 when looking for status updates on the DACP
	  remote control port. This follows a suggestion in
	  [Issue #1658](mikebrady/shairport-sync#1658). Thanks
	  to [ejurgensen](https://github.com/ejurgensen), as ever, for the report and
	  the suggested fix.
	* Fixed a `statistics` bug (the minimum buffer size was incorrectly logged) and
	  also tidy up the statistics logging interval logic for resetting min and max
	  counters.
	* Added an important missing format string argument to a call in the Jack Audio
	  backend. Many thanks to [michieldwitte] for their
	  [PR](mikebrady/shairport-sync#1693).
**Maintenance**
	* Stopped using a deprecated FFmpeg data structure reference.
	* Stopped using deprecated OpenSSL calls. Thanks to [yubiuser] for their
	  [PR](mikebrady/shairport-sync#1684) -- which did some
	  of the updating -- and for their guidance.
	* Run workflow-based tests on PRs automatically. Thanks to [yubiuser]
	  for their [PR](mikebrady/shairport-sync#1687).

Signed-off-by: Adolf Belka <adolf.belka@ipfire.org>
Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
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.

2 participants