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