tests/dtest.py: Maintain a separate test root for each test.

Now they can run in parallel without breaking each other, which is nice.
It's nice for two reasons: firstly, Automake changed behaviour and runs
tests in parallel by default nowadays; and secondly, the tests are
rather slow because they involve a bunch of waiting around for things,
and waiting for stuff in parallel is easy.

The downside is that we end up gobbling a lot of disk space with all of
the logfiles.  I think this is a good trade.

server/Makefile.am: Link `disorder_gstdecode' with `-lpcre'.

Linkers which don't transitively follow shared-library dependencies
complain.  This is fair enough, because there's a real direct dependency
here.

disobedience/misc.c: Muffle warning about `gdk_pixbuf_new_from_inline'.

Maybe we should use GResource at some point, but (a) it involves some
awful XML disaster, and (b) it's looking like way more effort than I
want to spend right now.

lib/configuration.c: Remove `type_stringlist', which nothing was using.

configure.ac: Link `-lgc' when searching for `GC_mumble' functions.

Otherwise we can't find them and fall back to using deprecated things.
This is a shame.

README, debian/control: Deprecate `sox' support.

sox introduced yet more command-line options in 14.2, whinges about the
old ones in 14.4, and threatens to remove them in 14.5 (because
obviously compatibility between minor versions is an old-fashioned idea
whose day is long past).

Rather than chase this obnoxiously moving target,

  (a) report that sox support is deprecated and will be removed in some
      future major version of DisOrder; and

  (b) have the Debian package Build-Depend on `libsamplerate0-dev' so
      that at least Debian users don't have to put up with sox's moaning
      in the logs.

Reported-by: Ross Younger

README: Mention optional support for GStreamer.

debian/etc.disorder.config: Remove bogus `trust root' line.

The `trust' option was removed in 657fdb79cbec..., but the default
Debian configuration wasn't changed to match.  Fix this.

Reported-by: Ross Younger

Email address

Update .gitignore

Replace code.google references with github references

Fix `distcheck' complaints.

  * Have `disobedience/Makefile.am' remove the images and desktop file
    it installs.

  * Remove `lib/versionstring.c' as well as its header file during
    clean.

  * Move source files (but not headers) from `lib/' to new `common/'
    directory if they're needed by other directories, and refer to them
    via the `VPATH' feature rather than using explicit pathnames.  (It
    doesn't work to have the whole of `lib/' on the VPATH because there
    are files in some of the other directories with the same names,
    which breaks in-tree builds.)

disobedience/disobedience.c: Show track title (or excuse) in title.

This doesn't do a great deal of good when the window is mapped.  But, in
some window managers at least, the title can be seen in a tooltip or
similar when the window is iconified or otherwise not visible.

Merge branch 'mdw/gstdecode'

* mdw/gstdecode:
  Add documentation and administrivia about `disorder-gstdecode'.
  server/gstdecode.c: Produce the configured sample format directly.
  server/gstdecode.c: Allow user to tweak the various parameters.
  server/gstdecode.c: New program, like `disorder-decode'.

Conflicts:
.gitignore
CHANGES.html
README
debian/control

CHANGES.html, README: Update the administrivia.

debian/control: Add Build-Depends for PulseAudio.

It really won't build without; and it probably shouldn't.

Add documentation and administrivia about `disorder-gstdecode'.

lib/cgi.c (cgi__input): Write terminating null within allocated buffer.

This one's been there forever.  It sometimes results in `malloc'
reporting a (rather impressive) assertion failure

disorder: malloc.c:3096: sYSMALLOc: Assertion `(old_top ==
(((mbinptr) (((char *) &((av)->bins[((1) - 1) * 2])) -
__builtin_offsetof (struct malloc_chunk, fd)))) &&
old_size == 0) || ((unsigned long) (old_size) >=
(unsigned long)((((__builtin_offsetof (struct malloc_chunk,
fd_nextsize))+((2 * (sizeof(size_t))) - 1)) & ~((2 *
(sizeof(size_t))) - 1))) && ((old_top)->size & 0x1) &&
((unsigned long)old_end & pagemask) == 0)' failed.

often after producing the beginning of the output page, so the end user
gets a blank window.  Not really what we want.

This fix closes issue #57.

disobedience/choose-search.c: Fix segfault when search terms change.

The observed problem:  Sometimes, when typing search terms into the box,
DisObedience crashes.  This seems to happen more often when the server
is slow to respond.

Here's what's going on.  Suppose that a search finds some results.  Then
`choose_make_visible' tries to expand the tree to display them.  If some
of the results aren't populated in the tree yet, then
`choose_make_one_visible' arranges to fetch the necessary information
from the server and informs its caller, `choose_make_visible', that
it'll have to try again later.  When this happens, `choose_make_visible'
hooks itself onto the `choose-more-tracks' event and waits for the
information to arrive.

Suppose that, while this is happening, the user edits the search terms;
a new search starts, but this one returns no results.  Then
`choose_search_completed' zaps the `choose_search_results' vector and
returns.

Now the metadata from the old search arrives from the server;
`choose_make_visible' is called back.  There are now no search results,
so it thinks its work is done and tries to scroll the list widget so
that the first result is visible -- but the result vector has been
clobbered.  Result: misery.

To fix this, cancel the `choose-more-tracks' event handler when a new
search finishes.  Now we can't get into this situation any more.

cgi/actions.c, lib/client*.[ch]: Don't use priv connection to check passwd.

If the CGI runs as the main `jukebox' user, then it can connect using
the special `.../private/socket', and the `find_server' function
arranges to do this if it can and no network address has been assigned
in the configuration.

The server doesn't bother to check passwords from clients on privileged
connections.  The result is that if the CGI program runs as the
`jukebox' user, its attempt to check the end-user's password through the
usual login machinery is stymied, since the library automatically sets
up a privileged connection and then the server ignores the password
entirely.  The end result is that, if you set things up in this way,
anyone can log into the CGI program with any known user and any password
at all, at which point they're given a cookie which can be used in
direct communication with the server.

Fix this as follows.

  * Introduce a new version (in the correct namespace for a change) of
    `find_server' which accepts some flags to guide the choice of server
    addresses.

  * Add a flag for the new `disorder_find_server' function to prevent
    trying the privileged socket.

  * Add a function to the client interface (and a bit of state to the
    client structure) to instruct the connection functions not to use a
    privileged connection even if they can.

  * Get the CGI program to use this new function when logging in with a
    password, so that it will always be checked.

Cookies are checked properly, even from privileged clients, so there's
nothing to fix there.

scripts/sedfiles.make, doc/disorder.cgi.8.in: Say where the file is.

The manpage used to say that the Makefile doesn't install the CGI
program anywhere, which hasn't been true for a while.  Remove the old
lies and just say where it was put.

configure.ac, lib/mem.c: Compatibility with older versions of libgc.

The `GC_get_all_interior_pointers()' function and its friends were
introduced after the libgc in Debian wheezy (which is quite impressive
seeing as they were added in 2009).  Make sure it's there; use the old
now-deprecated variable directly if it isn't.

Respond to libgc deprecation warning

Add nonnull attribute to a few function args.

Tinker with loop bounding to placate GCC 5.2.1

gzip -n in aid of reproducible builds

Warning tidy-up for FreeBSD.

- if dlfunc is available use that. Allows -Werror to be re-enabled.
- use modern struct member initializers
- cope with FD_SETSIZE being explicitly unsigned
- remove stray brackets
- remove unused objects
- use %ju to print sig_atomic_t, it might be bigger than int

Assign default http/cgi directories

FreeBSD doesn't have IFF_DYNAMIC

FreeBSD fixes

Update list of libdb versions.
Bodge in -liconv.

Placate gcc -flto

I don't think either of these are real bugs, but it keeps the compiler
happy.

init: source lsb init functions

Placates lintian a bit.

Suppress bogus doc-base registration warnings from lintian.

Remove redundant entry from .desktop file

Enable hardening flags harder.

Now with a workaround for broken GCC diagnostic.

debian: use debian compiler flags

debian: don't depend on essential packages

disorder.h: more consistent approach to function attributes

gitignore

tests: python unbuffered revisited

Set AUTOMAKE_OPTIONS=subdir-objects where necessary.

Windows support for command line client

Non-ASCII characters not properly supported.

Provide for a network initialization step

table search: cope without typeof

Remove some redundant includes.

Add uint16_t if missing.

Move up file inclusion.

UTF-16 support

Synchronous client now uses socketio sources and sinks instead of stdio

inputline variant which uses a source

Sources, by analogy with sinks

socketio sink

More general error handling for sinks

Buffered socket IO wrappers

Includes a nod to win32 API.

More error string formatting.

More conditional header inclusion.

Typos.

Commit version string files.

Allows for platforms with a rubbish build system.

Use private *printf functions a bit more widely.

C89 friendly declarations

Again, not a complete sweep.

log: more general error message formatting

Cope with various header files being missing.

Not a complete sweep - only things that will work on Windows have been
updated.

doxygen: remove obsolete config file entries

CHANGES

uaudio: newer compiler spotted a short memcpy

Bah

playrtp: support multiple unicast mode

server: implement multiple-unicast RTP

Updates the protocol definition and implementation and
exposes the uaudio-rtp rtp_mode variable in the config.

speaker: new comments to add/remove RTP recipients

client.c: new disorder_client_af reveals connectivity type

uaudio: RTP multiple unicast mode

speaker: protocol structure now has a union for different arg types

gitignore

CHANGES

uaudio: pulseaudio support

Playback only, no volume setting.

disobedience: menu item to select network playback API

disobedience: more sensible naming in menu setup

playrtp: --api option instead of a new option for every audio API

uaudio: more sophisticated choice of default playback API

autogen.sh: use /bin/sh

doc: correct & move description of raw format players

Remove now-redundant libao dependency.

gitignore

disobedience needs -lm

Explicit requirement only emerges in recent platform though.

Include version string in background program logs

no drag + drop

DisOrder 5.1.1

dist: wheezy

debian: always run configure

Prevents trouble if configure has been run but with non-Debiany
arguments.

autogen: use autoreconf -si

Merge tag 'branchpoint-5.1' into release

Branch point for 5.1 release.

Conflicts:
configure.ac
debian/changelog

CHANGES: reflect 'recent' bug fixes

server/gstdecode.c: Produce the configured sample format directly.

Introduce a GStreamer resampler, which is benign (and fast) if it has
nothing to do, and set the sink caps to match the configured audio
format.  Hopefully, the normalizer is now trivial.

This means that we must fetch the configuration file, which involves
adding gcrypt to the link.

server/gstdecode.c: Allow user to tweak the various parameters.

In particular, fiddling with the fallback gain level, and tuning the
audio conversion quality.

server/gstdecode.c: New program, like `disorder-decode'.

This uses GStreamer for the decoding, so it should be able to decode
more or less anything you can throw at it, assuming you have the
necessary plugins lying about.

More importantly (from my point of view) it does ReplayGain (optionally,
but by default).  This uses the per-album setting by default, but this
can be overridden.

debian/control: db4.8 is obsolete in wheezy.

The current version is 5.1, and that seems to work OK.

server/speaker.c: Only copy whole frames into collection buffers.

At least the ALSA playback function gets into a real mess if a buffer
contains a partial frame: it leaves the partial frame behind, but is
called again until the buffer is empty, resulting in an infinite loop.

Thanks for Joe Birr-Pixton for helping diagnose this bug and coming up
with the right fix.

lib/resample.c: Quieten some warnings better.

Newer GCC isn't fooled by `mumble = 0;' any more.  Apparently `if
(mumble) {}' works.

server/rescan.c: Remove tracks which no longer have a player.

If you ever add a player with an overly broad pattern, and it matches a
file which isn't actually an audio file but is legitimately part of the
directory tree, then there's no way of getting it out of DisOrder's
database short of hacking the files directly.

lib/configuration.c: Allow resetting `player' and `tracklength' lists.

Saith the manual

| If player is used without arguments, the list of players is cleared.

but the code disallows such requests.  So it's impossible to override
the player lists.

disobedience: tinker with About... box