[Queue-developers] Bugfixes/updates in CVS repository.

[Werner G. K. <wer...@ya...>]

I've collected the various bugfixes and put them in the CVS repository.

You should be able to obtain these changes by doing the following:

- unpacking the queue distribution.
- running the command "cvs update" from inside the distribution
directory.

This assumes you have cvs installed. Many linux distributions install
this by default; you can get it from ftp.gnu.org . There is also a web
browser of the repository on Sourceforge, but this is much harder to use
if you just want to update your files.

Hopefully, I can encourage more developers to become familar with CVS so
that they can upload changes themselves, and these can then be
distributed much more quickly between developers and testers. With write
access, you make your changes and then run "cvs ci"; you're prompted for
your password and a ChangeLog entry. This change is permanently recorded
with your SourceForge ID and log entry, but can be easily undone by
another developer if it doesn't work out as expected. All other
developers can then see your change by running "cvs update", presumably
in response to a message on one of the lists.

The main fixes are elimination of the strange commentation in handle.c
pointed out by Eric Deal and elimination of the potential deadlock
condition pointed out by eh...@so...

Here is the complete ChangeLog:

*August 11, 2000      W. G. Krebs <wk...@gn...>

        In Makefile.am :

        "rd...@io..."<rd...@io...> pointed out that random must be
        installed by Makefile.am (rather than in install-local-stuff)
        because it needs to link gethostbyname under Solaris. So, it
        was registered with automake as a noinst program. See
Makefile.am


*August 11, 2000      W. G. Krebs <wk...@gn...>

        Eric Deal <eri...@co...> pointed out strange
commentation
        are setrlimit source in handle.c that affected the socket setup.

        This deed indeed look strange. In fact, it probably caused a new
bug
        in the handling of X-windows applications (which close stdin and

        stdout) that I know wasn't present in old versions of Queue.

        So, I've fixed handle.c to just comment out the setrlimit code.
This
        may also have been commented out because of Solaris. If so, it
        should be put back in with an "#ifndef solaris" instead of a
comment.

*August 11, 2000      W. G. Krebs <wk...@gn...>

        Two bugs:

        1.

        Bug spotted by eh...@so....
        QueueD could deadlock if both are trying to transmit at same
time to
        each other. So, always fork by setting TRANSMIT_DEBUG in
queued.c

        2.

        Eric Deal <eri...@co...> pointed out that 1260-1264 in

        queued.c causes a hang Solaris, so I added an "ifndef solaris"
to this
        section of the code to prevent it from being compiled into
Solaris.


*August 11, 2000      W. G. Krebs <wk...@gn...>

        Made some changes to pty.c for linux. #ifdef linux causes
        utmp to be processed slightly differently under linux. Also,
        added check to make sure /dev/tty doesn't get chmod'ded on
        all platforms.