svn commit: r235771 - head/bin/expr

Kevin Lo kevlo at FreeBSD.org
Tue May 22 06:06:18 UTC 2012


Baptiste Daroussin wrote:
> On Mon, May 21, 2012 at 09:20:59PM -0700, Garrett Cooper wrote:
> > On Mon, May 21, 2012 at 8:01 PM, Kevin Lo <kevlo at freebsd.org> wrote:
> > > Author: kevlo
> > > Date: Tue May 22 03:01:54 2012
> > > New Revision: 235771
> > > URL: http://svn.freebsd.org/changeset/base/235771
> > >
> > > Log:
> > >  Remove redundant declaration of yyparse
> > >
> > >  Reported by:  tinderbox
> > >
> > > Modified:
> > >  head/bin/expr/expr.y
> > >
> > > Modified: head/bin/expr/expr.y
> > > ==============================================================================
> > > --- head/bin/expr/expr.y        Tue May 22 02:37:04 2012        (r235770)
> > > +++ head/bin/expr/expr.y        Tue May 22 03:01:54 2012        (r235771)
> > > @@ -74,7 +74,6 @@ int           to_integer(struct val *);
> > >  void           to_string(struct val *);
> > >  int            yyerror(const char *);
> > >  int            yylex(void);
> > > -int            yyparse(void);
> > >
> > >  %}
> > 
> >     There are a lot more than just this unfortunately (I'm working on
> > a complete patchset).
> >     This breakage really makes me think that yacc isn't being built at
> > the right time or pathed properly in the build system.
> > Thanks,
> > -Garrett
> > _______________________________________________
> > svn-src-all at freebsd.org mailing list
> > http://lists.freebsd.org/mailman/listinfo/svn-src-all
> > To unsubscribe, send any mail to "svn-src-all-unsubscribe at freebsd.org"
> 
> The complete patch set is there:
> http://people.freebsd.org/~bapt/fix-after-yacc.diff
> (clang seems to not detect those redudant warning, while gcc complains)
> 
> I'm waiting for review before committing)
> 
> Thanks for this fix I'll remove it from my patchset

Looks good to me. Thanks for cooking up the patchset.

> regards,
> Bapt

	Kevin



More information about the svn-src-all mailing list