Re: openssh-2.9.9p2 versus Solaris 2.6, Forte C version 6upd2 The compiler treated the call to do_pre_login in session.c line 581 as a prototype, warned of inconsistency with the function definition starting line 628: "session.c", line 628: identifier redeclared: do_pre_login current : static function(pointer to struct Session {[struct definition suppressed]}) returning void previous: function() returning int : "session.c", line 581 and failed with no further hint: cc: acomp failed for session.c make: *** [session.o] Error 2 To work around this I had to add a true prototype: *** session.c.ORIG Sun Sep 16 18:17:15 2001 --- session.c Thu Nov 15 14:45:40 2001 *************** *** 132,137 **** --- 132,140 ---- void do_child(Session *, const char *); void do_motd(void); int check_quietlogin(Session *, const char *); + #ifdef LOGIN_NEEDS_UTMPX + static void do_pre_login(Session *); + #endif static void do_authenticated1(Authctxt *); static void do_authenticated2(Authctxt *); FWIW, -- Rick Troxel rick at helix.nih.gov 301/435-2983 ///////////////////////////////////////////////////////////////// All effort and exertion put forth by man from the fullness of his heart is worship, if it is prompted by the highest motives and the will to do service to humanity. --Abdu'l-Baha
mouring at etoh.eviladmin.org
2001-Nov-15 21:18 UTC
Solaris 2.6: acomp failed for session.c
This has been fixed in the 3.0 and the latest 3.0.1 releases. - Ben On Thu, 15 Nov 2001, Rick Troxel wrote:> Re: openssh-2.9.9p2 versus Solaris 2.6, Forte C version 6upd2 > > The compiler treated the call to do_pre_login in session.c line 581 as a > prototype, warned of inconsistency with the function definition starting > line 628: > > "session.c", line 628: identifier redeclared: do_pre_login > current : static function(pointer to struct Session {[struct > definition suppressed]}) returning void > previous: function() returning int : "session.c", line 581 > > and failed with no further hint: > > cc: acomp failed for session.c > make: *** [session.o] Error 2 > > To work around this I had to add a true prototype: > > *** session.c.ORIG Sun Sep 16 18:17:15 2001 > --- session.c Thu Nov 15 14:45:40 2001 > *************** > *** 132,137 **** > --- 132,140 ---- > void do_child(Session *, const char *); > void do_motd(void); > int check_quietlogin(Session *, const char *); > + #ifdef LOGIN_NEEDS_UTMPX > + static void do_pre_login(Session *); > + #endif > > static void do_authenticated1(Authctxt *); > static void do_authenticated2(Authctxt *); > > FWIW, > -- > Rick Troxel rick at helix.nih.gov 301/435-2983 > ///////////////////////////////////////////////////////////////// > All effort and exertion put forth by man from the fullness of his > heart is worship, if it is prompted by the highest motives and > the will to do service to humanity. --Abdu'l-Baha > >