Hi, trying "current CVS" on AIX 4.3.3, yields: gcc -O2 -Wall -I/usr/local/include -I/gnulocal/include -I/gnu/include -I. -I./openbsd-compat -I. -DETCDIR=\"/etc\" -D_PATH_SSH_PROGRAM=\"/gnu/bin/ssh\" -D_PATH_SSH_ASKPASS_DEFAULT=\"/gnu/libexec/ssh-askpass\" -D_PATH_SFTP_SERVER=\"/gnu/libexec/sftp-server\" -DHAVE_CONFIG_H -c auth.c auth.c: In function `allowed_user': auth.c:145: warning: implicit declaration of function `loginrestrictions' auth.c:145: `S_RLOGIN' undeclared (first use in this function) auth.c:145: (Each undeclared identifier is reported only once auth.c:145: for each function it appears in.) gmake: *** [auth.o] Error 1 S_RLOGIN is declared in /usr/include/login.c. config.h has "#define HAVE_LOGIN_H 1", but the #include <login.h> statement at top of auth.c includes "./login.c" due to the -I. line. Why do we need -I. ? Shouldn't those things be included with #include "something.h", as opposed to #include <something.h>? Should we rename ./login.h to ssh-login.h? gert -- USENET is *not* the non-clickable part of WWW! //www.muc.de/~gert/ Gert Doering - Munich, Germany gert at greenie.muc.de fax: +49-89-35655025 gert.doering at physik.tu-muenchen.de
Hi, On Fri, Feb 16, 2001 at 03:15:05PM +0100, Gert Doering wrote:> config.h has "#define HAVE_LOGIN_H 1", but the #include <login.h> > statement at top of auth.c includes "./login.c" due to the -I. line. > > Why do we need -I. ? Shouldn't those things be included with > #include "something.h", as opposed to #include <something.h>? > > Should we rename ./login.h to ssh-login.h?login.h is only included from session.c - renaming it to ssh-login.h and changing session.c makes it compile... gert -- USENET is *not* the non-clickable part of WWW! //www.muc.de/~gert/ Gert Doering - Munich, Germany gert at greenie.muc.de fax: +49-89-35655025 gert.doering at physik.tu-muenchen.de