I studied the code, before suggesting that, and Shane confirmed it works ok.


The "dead" function, has been absorbed by an existing function.

The existing function is just following the "to be commented" lines, so the functionality os ok.

[But I think the "dead" routine could be mantained, inside vpopmail, for compatibility, giving time to people to update various patches.]

Ciao,

Tonino

At 04/09/03 04/09/03 -0400, Charles Sprickman wrote:
FWIW, I'm getting a similar error with 5.3.27:

./load qmail-smtpd rcpthosts.o commands.o timeoutread.o  timeoutwrite.o
ip.o ipme.o ipalloc.o control.o constmap.o  ssl_timeoutio.o ndelay.a
-L/usr/local/ssl/lib -lssl -lcrypto  -L/home/vpopmail/lib
-L/usr/local/lib/mysql  -lvpopmail -lmysqlclient  received.o date822fmt.o
now.o qmail.o cdb.a fd.a wait.a  datetime.a getln.a open.a sig.a case.a
env.a stralloc.a  alloc.a substdio.a error.a str.a fs.a auto_qmail.o
base64.o  `cat  socket.lib`
qmail-smtpd.o: In function `realrcpt_check':
qmail-smtpd.o(.text+0xc51): undefined reference to `vget_real_domain'
*** Error code 1

As mentioned below, if I comment out the section referencing
"vget_real_domain" in qmail-smtpd.c, it does compile, but I would think
that would affect functionality, no?

[EMAIL PROTECTED]/usr/local/src/toaster/vpopmail/vpopmail-5.3.27]# grep
vget_real_dom *

ChangeLog:         - removed the need for vget_real_domain() function
ChangeLog:           tracked it down to vget_real_domain calling
ChangeLog:           the vget_real_domain() function to change the domain
vpopmail.h:int vget_real_domain(char *domain, int len );

Looks like the function is gone...

Charles

On Sun, 31 Aug 2003, tonix (Antonio Nati) wrote:

> Shane,
>
> I did not try the last versions of vpopmail.
>
> It looks like function vget_real_domain does not exists anymore.
>
> Try commenting out these lines of code, and please let me know.
>
> /* Check if domain is a real domain */
> /*
>          if (!stralloc_0 (&domain)) die_nomem();
>          vget_real_domain(domain.s, domain.a);
>          domain.len = strlen (domain.s);
>          if (domain.len > (domain.a - 1)) die_nomem();
> */
> /* Let's get domain's real path */
>
> Tonino
>
> At 01/09/03 01/09/03 +0800, Shane Chrisp wrote:
> >  Im setting up a new system and I found that there is an error when
> >trying to use the chkusr patch with the latest version. Below is the
> >error from qmail. I tried compiling again with 5.3.24 successfully
> >so I then tried 5.3.25 and that failed also, with the same error.
> >
> >
> >./load qmail-smtpd qregex.o rcpthosts.o commands.o timeoutread.o \
> >timeoutwrite.o ip.o ipme.o ipalloc.o control.o constmap.o \
> >received.o date822fmt.o now.o qmail.o cdb.a fd.a wait.a \
> >datetime.a getln.a open.a sig.a case.a env.a stralloc.a \
> >alloc.a substdio.a error.a str.a fs.a auto_qmail.o base64.o  `cat \
> >socket.lib` -lssl -lcrypto dns.o `cat dns.lib` -lcrypt \
> >`head -1 conf-vpopmail`/lib/libvpopmail.a \
> >/usr/lib/mysql/libmysqlclient.a -lz
> >qmail-smtpd.o: In function `realrcpt_check':
> >qmail-smtpd.o(.text+0xf29): undefined reference to `vget_real_domain'
> >collect2: ld returned 1 exit status
> >make: *** [qmail-smtpd] Error 1
> >
> >
> >Regards
> >
> >Shane
> >
>
>
> ------------------------------------------------------------
>          [EMAIL PROTECTED]            Interazioni di Antonio Nati
>     http://www.interazioni.it      [EMAIL PROTECTED]
> ------------------------------------------------------------
>


------------------------------------------------------------
        [EMAIL PROTECTED]            Interazioni di Antonio Nati
   http://www.interazioni.it      [EMAIL PROTECTED]
------------------------------------------------------------




Reply via email to