cvs commit: src/lib/libc/rpc auth_des.c auth_none.c auth_unix.c clnt_dg.c clnt_raw.c clnt_simple.c clnt_vc.c getpublickey.c key_call.c mt_misc.c mt_misc.h rpc_com.h rpc_generic.c rpc_soc.c rpcb_cl

[Daniel Eischen]

On Tue, 28 Feb 2006, Xin LI wrote:
>
> On Mon, Feb 27, 2006 at 10:10:59PM +0000, Daniel Eischen wrote:
> > deischen    2006-02-27 22:10:59 UTC
> >
> >   FreeBSD src repository
> >
> >   Modified files:
> >     lib/libc/rpc         auth_des.c auth_none.c auth_unix.c
> >                          clnt_dg.c clnt_raw.c clnt_simple.c
> >                          clnt_vc.c getpublickey.c key_call.c
> >                          mt_misc.c rpc_com.h rpc_generic.c
> >                          rpc_soc.c rpcb_clnt.c svc.c svc_auth.c
> >                          svc_dg.c svc_generic.c svc_raw.c
> >                          svc_run.c svc_simple.c svc_vc.c
> >   Added files:
> >     lib/libc/rpc         mt_misc.h
>
> I think we used to have a compiler bug which has hidden the
> API inconsistency where yp related functions having weak
> constant constraints, while the other part in libc do.
>
> Seems that getpublickey.c change has triggered the compiler
> to (correctly) generate warnings about the const/variable
> inconsistency, and thus broken the world...  Perhaps we
> should add const constraint for certain yp related stuff?

I built world with these changes, so I'm not sure where
there is a problem.  If you can reproduce a build failure
and know what the fix is, please commit it.

-- 
DE