Merge commit 'remotes/fml/master'

afs.h: Doxify comment for struct audio_file_data.

afs.h: Kill unused function declaration close_audio_file().

Add new "screenshots".

Add new status items num_chunks and chunk_time.

Merge commit 'remotes/fml/master'

alsa_write.c: Don't abort on EAGAIN.

Just retry a bit later.

command_util.sh: Add missing newline.

Fix some gcc warnings on Solaris:

warning: int format, pid_t arg
warning: int format, long int arg
int format, uid_t arg
int format, gid_t arg

CREDITS update.

net.c: Make host_and_port() static.

It's only used in net.c. Some more trivial cleanups also.

http_send.c: Fix another doxygen warning.

Found \endcond command without matching \cond

net.c: Fix a doxygen warning and trivial coding style cleanups.

No functional changes. Fixes the following doxygen warning:

Warning: documented function `getsockname for local or fn getpeername' was not declared or defined.

Cosmetic cleanups include:

- Reduce line width to 80 chars in most cases.
- Replace @param by \param.
- Replace tabs by spaces.

Trivial whitespace cleanups.

Merge commit 'remotes/fml/master'

Fix gcc warning on FreeBSD.

daemon.c:184: warning: long int format, int arg (arg 2)

Mention IPv6 support in NEWS and FEATURES.

Fix compilation on NetBSD.

At least on NetBSD-3.1, AI_V4MAPPED, AI_ALL, and AI_ADDRCONFIG
are all undefined. The fix is to #define these to zero if they
are not defined in netdb.h. This means that no hints will be
given in this case which is the best we can do.

alsa_write: Print more detailed error message.

06_Unix-Domain-Sockets.diff

[LOCAL SOCKETS]: Tidy up and share common code

This patch just suggested itself - after all the other changes have been made,
the only remaining customers of PARA_CONNECT were the Unix domain sockets.

The patch generates a counterpart to create_local_socket(), for the peer talking
over a Unix domain socket.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

05_DCCP-Remove-Unused.diff

Since DCCP now also uses the makesock function from net.c, it does no longer
need the functions from dccp.c -- the file is thus removed by this patch.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

04_DCCP-socket-functions.diff

This converts the DCCP streaming unit to use the makesock interface.

As much as possible, the format of the system calls was modelled after /
adapted to that of the HTTP sender/receiver.

This includes the use of a `name' field for the remote socket name of
the connecting client.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

03_TCP-socket-functions.diff

This converts the control connection and the HTTP streaming module to use the makesock() interface.

The specific changes are:
 * replace tcp_connect() directly with makesock();
 * replace tcp_listen() with para_listen() (in turn a wrapper around makesock());
 * init_sockaddr() becomes obsolete;
 * replaced use of sockaddr_in in para_accept() with NULL sockaddr argument
  -- the `remote' name of the socket is always available via getpeername(2),
-- this is exploited by replacing the reverse-name lookup with remote_name();
 * the sockaddr_in field `addr' of `struct http_client' has been replaced with `name' string
  -- name is filled in as in para_accept() above, using remote_name(),
-- unlike above, the remote_fd() call can later not be done, due to close_listed_fds().

The one thing I am really not sure about is host_in_access_perm_list():
 * It needed to be converted to allow address-independent HTTP streaming.
 * The conversion represents the obvious choice for doing this.
 * However, I have not (yet) tested it.
 * A problem that may arise is that when listening using AF_UNSPEC then _all_ addresses appear
   as IPv6 addresses (i.e. the test "family == AF_INET" would never hold true). In this case,
   a possible solution is to switch between AF_UNSPEC and AF_INET dependint on whether the
   configurable access-list is empty or non-empty, respectively.
 * A possible place for this chance would be http_send_init() or any other function calling
   open_tcp_port().

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

02_Reverse-lookup-support.diff

In some parts of the code a reverse-lookup (e.g. "connection from xxx") is used.
An address-family independent reverse-lookup function is provided by this patch,
exploiting that a socket is described by {src addr, src port, dst addr, dst port}:
 * host_and_port(): prints host and port number of given socket,
 * local_name():    pretty-prints src-addr/src-port,
 * remote_name():   pretty-prints dst-addr/dst-port;

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

01_IPv6-Basic-Support.diff

This patch provides an algorithm which, given a hostname and a numeric port identifier,
will look up all matching IPv4/IPv6 addresses and either bind it (for passive sockets)
or connect to it (for active sockets).

Certain socket options need to be set before a connection is established. Since in
paraslash the demand for such options is not very big at this time, the only present
case (setting SO_REUSEADDR on passive sockets) has been integrated into the main loop.

A more sophisticated variant for setting pre-connection socket options exists and is
available as part of the DCCP library. The current solution is extensible, i.e. if one
wants to later add more options, the alternative (more sophisticated approach) can be
used. But for now, that would just have meant code bloat.

The subsequent patches in this set will step by step replace the old infrastructure
with the use of this algorithm.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

INSTALL: Use $LOGNAME instead of hardcoding the user name.

Fix link to gitweb on the main page.

replace PUBLIC_KEY.

The new key adds my email address of tuebingen, and deletes
the tud and the www.paraslash.org email addresses.

Remove key.anonymous.

It was only needed for the live demo.

com_addatt: Minor improvements.

- Use get_attribute_bitnum_by_name() instead of open-coding.
- Use new error code E_ATT_TABLE_FULL.
- Move definition of objs[ATTCOL_NAME] down to where it is needed.
- Kill FIXME comment: We can not use find_greatest_att_bitnum() here.

string.h: for_each_line_ro() takes a _pointer_ to the line_handler.

afs.h: Add more documentation.

afs.h: Fix typo in comment.

afh_common.c: Add documentation of afh_init().

error.h: Cosmetics.

The setatt command needs at least two arguments.

Fix a couple of typos in error messages.

com_setatt(): Generate an afs event if attributes were modified.

giweb.css: Make backround color for commit messages a bit darker.

net.c: Typo.

afs.h: Make doxygen happy again.

It interprets <foo> as html.

README.afs: Minor improvements.

Rename init_tcp_socket() to tcp_listen() and further net.c cleanups.

Cleanups consist in the following:

- get rid of setserversockopts(). It had only one caller
  within net.c, so incorporate it into tcp_listen().

- Kill E_BIND, E_LISTEN, E_SETSOCKOPT and E_SOCKET. Return
  converted errno instead.

Introduce tcp_connect().

This replaces get_host_info(). The two callers of the latter,
client_connect() and http_recv_open(), did exactly the same, namely
initializing a struct sockaddr_in which was then used for PARA_CONNECT().

So move this common code to tcp_connect(). Additional benefit of this
is that the only remaining callers of init_sockaddr() are in net.c, so
make this function static.

net.c: Cosmetics.

Kill E_ACCEPT.

Get rid of E_SEND.

This patch also simplifies sendall() a bit and fixes two bugs in
add_one_audio_file():

- If the check for existence of a hash sister failed, we missed
  to unmap the audio file.
- If an error occurred we send out a message to the client. If
  this message can not be sent (because the client terminated
  the connection) we returned success anyway and happily
  tried to add the next audio file.

Some documentation improvements as well.

Get rid of E_RECV.

recv_bin_buffer() returns the converted errno on errors.
So there's no point in having a paraslash error number for recv errors.

Minor doxygen improvements.

ogg_afh.c: Increase eof timeval.

gui: Reorder display of status items.

gui.c: Handle empty status items properly.

gui.c: Cosmetics.

gui.c: Increase numbers of color pairs not related to themes.

So that they do not overlap with status items color pairs.

Rename mood_method "name_like" to "path_matches".

Fix rename detection.

If a file is renamed and re-added the old code would segfault because of
a NULL pointer dereference in save_chunk_table(). Fix is trivial.

com_lsblob(): Fix sorting order.

Merge commit 'remotes/fml/master'

Define XOPEN_SOURCE and __EXTENSIONS__ on Solaris.

These are needed for passing fd through a socket.

Don't garbage-collect sections.

This does not work on Solaris, NetBSD and MacOS. Leave it in as
a comment though.

Rename mark_fd_nonblock() to mark_fd_nonblocking().

Just to be consistent with mark_fd_blocking().

Rename sched() to schedule().

We already have struct sched..

REQUIREMENTS: Add pointers to software packages.

net.c: Return proper error value in recv_bin_buffer().

In handle_connect() we really need a blocking fd for the client.
At least on NetBSD the fd returned by accept() might be in non-blocking
mode. So introduce mark_fd_blocking() and use it in handle_connect().

Replace shebang in bash scripts.

NetBSD does not have /bin/bash.

afs.c: Simplify activate_mood_or_playlist().

This also gets rid of

warning: `mode' might be used uninitialized in this function

on gcc-3.3.3.

Add wrapper for isspace().

This avoids

warning: subscript has type `char'

on netbsd.

gitweb.css: Improve readability.

Add some gitweb-releated stuff.

overview.sk: Minor improvements.

Remove another link to the live-demo.

Kill live-demo web page.

Add final 0.2.17 stuff.

the paraslash-0.2.17 release tarball

Fix html header.

The location of the cgi-bin dir on systemlinux is different from
paraslash.org. This broke the gitweb link in the left menu.

download.in.html: Replace paraslash.org in clone command.

Merge commit 'remotes/meins/v0.3' into v0.3

Announce NetBSD Port.

aft.c: Simplify print_list_item().

aft.c: Do not print directory if LS_FLAG_FULL_PATH was not given.

aft.c ls -lm: Fix unmapping of lyrics file.

Fix handling of invalid entries in the score table.

It may happen that the score table contains references to files that
no longer exist in the file system. The current code fails badly in
this case.

In open_next_audio_file(), if the file corresponding to the next
entry in the score table can not be opened, delete this entry from
the score table and try the next entry.

If the score table becomes empty, send NO_ADMISSIBLE_FILES back to para_server
and make recv_afs_result() switch to stop mode in this case.

Add additional sanity checks to osl_get_rank().

Check whether the riven rank is greater than zero and not greater than
the number of rows in the table.

Fix osl_del_row().

Only free volatile columns of the deleted row if OSL_DONT_FREE
is not set.

open_and_update_audio_file(): Interchange two arguments.

And add documentation of the "score" parameter.

REQUIREMENTS: Kill an incomplete sentence, add git.

Introduce vss_stopped().

Fix com_pause().

If the vss status is stopped, we must not switch to pause status.

fade.c: Fix client_cmd()

We have to wait for children to terminate because otherwise the
client commands may be executed in random order.

Since no caller checked the return value of client_cmd(), make it
abort on errors and change the return value to void.

Also, each caller only passed one string, so make it take only a
char * pointer.

stat.c: Cosmetics.

make_empty_status_items(): Add spaces after status item identifiers.

Identifiers without spaces confuse para_gui.

Fix tv_divide().

The old code contained an integer overflow which caused the mp3 audio
format handler to compute the chunk of large mp3 files incorrectly.

Moreover, the code was more complicated than necessary. So replace it
by a simple calculation which just transforms the given struct timeval
into its number of microseconds, does the division and transforms
the quotient back to a struct timeval.

Thanks to Gerrit Renker for pointing out the problem.

Fix com_nomore.

Avoid some gcc warnings.

Avoid empty lines in the status output.

Introduce make_empty_status_items().

At the end of the stream we have to reset the status items
because audiod would start a new receiver otherwise.

Add more status items.

Use FOR_EACH_STATUS_ITEM everywhere.

handle status items via autoconf.