bug-inetutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: talk/talkd on solaris 2.8


From: Alain Magloire
Subject: Re: talk/talkd on solaris 2.8
Date: Tue, 4 Sep 2001 22:24:57 -0400 (EDT)

> 
> On Wed, 18 Jul 2001, Alain Magloire wrote:
> 
> > > > The current cvs tree:
> > > >         http://savannah.gnu.org/projects/inetutils/
> > > Today I got the current cvs tree and saw that Sergey already did some of
> > > the changes I needed but in a slightly different way. Since my patch now
> > > doesn't apply anymore but my local copy works alright, I'll just leave
> > > it as it is for now. On the next update I'll see if Sergey's changes
> > Yes, except that your patches corrected some stuff, that was not part of
> > Sergey's work.
> I now rechecked against current cvs and it seems that Sergey's changes
> corrected almost anything my patch did, too. He just does it diffent.
> 
> For example: I searched the tty's in utmp(x) and especially remembered
> "console" as a last resort to fall back to if no other tty is found for
> that user since on Solaris outputs to /dev/console don't appear anywhere
> unless you have an xconsole open or shut down X completely and work on
> the text console emulation (which certainly almost noone does). Sergey
> therefore just chose to search the 'newest' tty, which means the one with
> the lowest idle time which by the way does what I need for Solaris and is
> a much better solution anyway.
> 
> Therefore I just needed to adjust my utmpx-patch and it worked OOB. I just
> have to recheck tomorrow if it works okay under X since I compiled it
> remotely from my home Linux box and just checked it from there on text
> console. I can give you ultimate feedback on that then, but I'm very
> confident that it does work fine like it is now.

Patch apply.
You forgot the entries for talkd/ChangeLog, add one for you.
Thanks for looking at this.

> > > already do the job or something more is still needed and then I'll do it
> > > right from the beginning and send you a patch against current cvs and a
> > > changelog entry.
> > That would be great, thanks for taking the time.
> So here it is, the patch against latest cvs for using utmpx if available
> which makes talkd work on my Solaris 2.8 box.
> 
> Perhaps one should add some check if utmpx is really used on that system
> and the user wants to use it too since I know that at least IRIX 6.5 has
> utmpx-support but also keeps a utmp in sync for compatibility with older
> applications.

The direction is to move away from utmp, completely.  And if we need
support for it, we could provide stub functions in libinetutils.

Something like.
void setutxent ()
{
        setutent ();
}

This will reduce the number of #ifdef and make the code readable.

But it will probably not happen for this release, I'm too busy
with other projects.

> Apropos IRIX: Is there some need for experiences with IRIX 6.5? I have
> such a box and the system's talkd is a bit wacky but not so much that I'd
> replace it just for fun. But if you'd like some report on compiling and
> running inetutils on such boxes I'd volunteer...


IRIX, is not on my list of target for the release, but if you have time
to poke around, I will not refuse clean patches.

--
alain




reply via email to

[Prev in Thread] Current Thread [Next in Thread]