From: wusel@hactar.hanse.de (Kai 'wusel' Siering)
Newsgroups: alt.sys.amiga.uucp
Subject: Re: uucico sd.1 Part 1/1 (repost)
Date: Tue, 29 Dec 1992 12:48:18 +0100
Reply-To: Kai 'wusel' Siering <wusel@hactar.hanse.de>
Organization: Villa Straylight, Hamburg, FRG
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
X-Newsreader: Arn V1.00

In article <9212281813.AA21761@enet-gw.pa.dec.com>, Steve Drew writes:

%
% This contains source and binary. See README in archive.
%
% begin 666 uucico_sd.lha
[...]

Thank you! (Wrong group, but at least it arrived =:->)

     But there is something really unwise (in my opinion) regarding the
notification stuff:

uucico.c
---------------------------------------8<---------------------------------------
[...]
mail_notify(from,real,subject)
char *from;
char *real;
char *subject;
{
    char mailcmd[NAMESIZE+100];
    char *to = "postmaster";
    /*
     * should really inquire on SENDMAIL variable in config file
     */
    char *sendmail = "sendmail";

    if (strlen(subject) > 200) subject[200] = '\0';  /* cli input limits */

    sprintf(mailcmd, "%s <nil: >nil: -f %s -R \"%s\" -t %s -s \"%s\"",
        sendmail,from,real,to,subject);

    Execute(mailcmd,NULL,NULL);
}
--------------------------------------->8---------------------------------------

     First, this doesn't work with my sendmail (it expects a mail
(including headers) on stdin -- adding `-raw' would help (just checked));
but even if it would, starting the mailer synchronously isn't a wise move:
just think of another mail arriving at the same time or, even worse, the
user started a mailreader (e. g.  AmiELM) to read postmaster's folder. The
result would be a uucico waiting for sendmail to return, which itself waits
for an UnLockFile("uumail:postmaster") to happen -- quite expensive in case
of long distance calls =:-<

Would you please be so kind and change this to

    sprintf(mailcmd, "run %s <nil: >nil: -f %s -R \"%s\" -t %s -s \"%s\" -raw",
        sendmail, from, real, to, subject);

I assume this should work with the AmigaUUCP sendmail, too (can someone
please check this out?). [I would do this myself but neither have the time
nor the compiler to do this -- and to cut down the number of different
uucicos floading around, I really would prefer Steve to release a new
version.]

Happy new year,
			kai

P.S.: Steve, don't get me wrong! I appreciate your improvements, but I'm
      in a terrible hurry (have to leave town in 5 hours), otherwise I
      would have tried to recompile it myself (which means I will have
      to unpack the 1.16 sources and start running into problems because
      of using SAS instead of DICE. Well, I intend to take this uucico
      and add the `uuspool:system/' scheme by February '93 unless some-
      one els is willing to do this =:->).

--  _
 _ //   Kai 'wusel' Siering, Villa Straylight, Hamburg, Germany. Go for MIME!
 \X/: <wusel@hactar.hanse.de>  Exile: <wusel@wolfhh.hanse.de>  iso-8859-1: 

              /away 18.12.1992 - 04.01.1993 (Phone: +49-4755-648)

"Ich    weiss,    ich   bin   ein   archaischer   Verfechter   von   heidnischen
Stammtischbraeuchen  in Zeiten weltweiter Vernetzung, aber ich lamentiere lieber
bei  einer guten Pizza und einem Donnergurgler (fuer Puristen Gargleblaster - Hi
mw :-) /Kaffee/etc. als elektronisch."
                      -- Bjoern Kriews in <1992Jun17.231852.4074@drdhh.hanse.de>
Newsgroups: alt.sys.amiga.uucp,comp.sys.amiga.misc,comp.sys.amiga.datacomm