Return a nonzero value on error

Merge OpenBSD audio driver

Make a few fixes debian init script.

Add avahi as a start dependency, fix the provide line.
Also add a note about updating the logrotate file if non-standard
log and error files are used.

fix shell script

typo

add support for openbsd sndio sound daemon

README: add debian package suggestions

rtsp: send responses as single packets

Under some circumstances, RTSP responses would be split into multiple
packets. This causes Airfoil to barf, as it interprets the end of the
packet as the end of the response. Since TCP_CORK is not portable, this
fix buffers the packet on the stack before using a single write.
This fixes issue #213.

rtsp: treat pthread_t as properly opaque

We use the mutex's state to determine whether the playing_thread is
valid or not. Also now use pthread_equal, as we should. Should address
issue #236.

rtsp: don't try and take our own player lock

This can happen during a shutdown triggered by the playing thread. Fixes
issue #233 reported by therealmuffin.

Merge pull request #237 from jclehner/1.0-dev

Allow empty SET_PARAMETER request

Allow empty SET_PARAMETER request

Certain senders (AirAudio for Android for example) periodically
send SET_PARAMETER requests with Content-Length: 0, to which
shairport responds with an error, making those apps misbehave.

Merge pull request #232 from plietar/fix-rtsp-close

Reset playing_thread on connection close. Fixes #217

Reset playing_thread on connection close. Fixes abrasive/shairport#217

pthread seems to reuse thread identifiers (at least on arm).
Reset it to zero after closing a playing connection, or else future
thread might believe they are playing.

daemon, shairport: POSIX compatibility fix

Do not print daemon's pid after forking if a pidfile is used.

Add a logrotate script.

Add and init script for debian.

Only ignore files in top directory.

For some reason, git ignored all subdirectories. This fixes it.

In daemon mode, send error messages to parent to write them on stderr.

This allows to get a better error message than "Spawning the daemon failed."
when initialization fails.

The error message is sent through the pipe. When init is OK, just send
a null byte.

configure: clean up pkg-config code into a function

Makefile: force rebuild after reconfigure

audio_ao, audio_pulse: make usage indent consistent

configure: fix message

TODO updates (nearly there!)

README.md: updates

Alsa output driver

Based on a bit of code of @Skaman and bit of mine, I've added support
for mixer and audio Alsa support.

check supplied AP name length to comply with mDNS spec.

shairport: do not try and deinit output if it hasn't been set yet

Tiny cleanup for merge

Use line buffering on log files.
This makes sure the contents are written on the disk, so the user can
read the log.

Support log redirection to a file.

Remove pid file when exiting.

Move daemon related code to a separate file.

Add support to write pid to pidfile when running in daemon mode.

Revert "Fix pulse audio output driver in daemon mode."

This reverts commit 8045e00d4f3d694cff99c30e3a5cd63eca3b4e5c.
Now, pulseaudio can be initialised at startup, thus detecting connection
errors early.

Conflicts:
audio_pulse.c

Merge pull request #223 from plietar/pulseaudio

Add a PulseAudio output driver, courtesy of Paul Lietar

daemonising: fork before doing any audio init

audio: update usage messages

add long options, and update usage message

getopt_long: neater errors

add getopt_long implementation and detection

pulseaudio: Make the application name configurable

If no name is specified, use the AirPlay access point name.

pulseaudio: fix option parsing

ao, shairport: fix option parsing

- don't push unrecognised args to the audio driver
- get ao to parse from first argument
- ao checks for unrecognised options

Merge pull request #224 from plietar/fix-libao-driver-check

Fix libao output driver.

Fix pulse audio output driver in daemon mode.
libpulse was initialized before the fork. This caused calls to libpulse
made after the fork fail.
Now initialize lib pulse when we need it, after the fork.

Add a PulseAudio output driver.
It uses libpulse's synchronous API, to output to PulseAudio
without using libao.

Fix libao output driver.

The driver id returned by libao wasn't checked correctly.
When the driver did not exist, libao returns a negative number. The
libao output driver checked for a non zero id.
This caused shairport to refuse to use the first driver, and not fail
when using a non-existant driver.

rtsp: track and join all RTSP threads

rtsp: do not race accept() and select()

rtp/rtsp: more debugging in verbose mode

configure: do not default openssl libs, cleanup messages

add native avahi-client support

requires extra packages on some systems, eg. libavahi-client-dev on
Debian/Ubuntu.

TODO updated

die/warn functions now have implicit newline at the end

add a pipe/FIFO audio backend

LICENSES updated w/ alac.c pointer

provide -B/-E for running shell commands on begin/end of playing

Implements functionality described by MaZderMind <github@mazdermind.de>
in issue #201.

mdns: die iff the MDNS advertising process dies

and not if any other child does!

change default port to 5002

As in the previous version. Fixes #214.

whitespace fixes

rtsp: cleaner bind handling

- nicer debug messages
- arbitrary number of addresses, for multihomed hosts
- get rid of AI_ADDRCONFIG

PKGBUILD: architecture armhf -> armv6h

TODO updates

restore '-d' (daemonise) flag

player: more robust seq_t handling

player: remove redundant code path for fetch/play thread

conditional wait code was redundant since a contributed change that
plays silence if nothing is in the buffer. that also broke the buffer
fill estimator reset pathway.

rtsp: cleanup debug message

player: cleanups

should get rid of some sign issues in debug messages on ARM

player: fix missing frame at stream startup

scripts: add PKGBUILD and systemd service

Makefile: create target directory if needs be

Makefile: add install target

cleanups

Merge pull request #200 from mrpippy/1.0-dev

Compile fixes for OpenBSD, and call freeaddrinfo() in rtp.c

Call freeaddrinfo() in rtp.c

Plug a (small) memory leak from getaddrinfo()

Merge remote-tracking branch 'abrasive/1.0-dev' into 1.0-dev

rtsp: safer buffer handling

rtsp: handle volume messages

player: talk about established buffer fill preference

debug: multilevel debug functions

rtp: don't resync on mystery resend-related packet

rtsp: fix endian bug in challenge computation

on MIPS platform the IP address was reversed...

rtsp: add a Session header to keep iTunes happy

With thanks to Quentin Smart (sm3rt) for the fix.

readme: ditch simultaneous streams

Merge pull request #194 from MaZderMind/ReadmeUpdate

readme: remove perl-script, mention build process and iPhones

Merge pull request #192 from MaZderMind/MakefileChanges

Makefile cleanups

Merge pull request #193 from MaZderMind/getoptint

change getopt return storage from char to int

remove perl-script from readme, mention build process and iPhones

change getopt return from char to int

don't fail clean target, when binary does not exist

add default target w/ name 'all'

Define AI_ADDRCONFIG (needed for OpenBSD and NetBSD)

AI_ADDRCONFIG is not supported by some BSDs (OpenBSD and NetBSD).
It's not a critical flag, so define it to 0 just to fix the compile
error.

Add #includes for OpenBSD

Add includes for <sys/types.h> and <netinet/in.h> to rtp.c and rtsp.c, needed
for the declarations of IPPROTO_TCP, IPV6_V6ONLY, and others.

rtsp: set playing_thread safely

fix compile warning

fix audio buffer allocation

...and make the implicit data dependency on frame_bytes more obvious

use AI_ADDRCONFIG with getaddrinfo()

Fixes IPv4 systems with IPv6 present but disabled.
Fix courtesy of Quentin Smart (sm3rt)

fix IPv6 support