There was some debate about this last time. It was decided to use the .c file. The tarball ships with both. The .y file only used if its timestamp is newer, which is never the case.
The .y file should never have been imported in the first place. That was my fault. ~cy Sent from my BlackBerry 10 smartphone on the Bell network. Original Message From: Garrett Cooper Sent: Monday, May 4, 2015 10:49 To: Cy Schubert Cc: Craig Rodrigues; jenkins-ad...@freebsd.org; freebsd-current Current Subject: Re: Build failed in Jenkins: FreeBSD_HEAD #2725 On May 4, 2015, at 10:46, Garrett Cooper <yaneurab...@gmail.com> wrote: > > On May 4, 2015, at 10:38, Cy Schubert <cy.schub...@komquats.com> wrote: > >> I think jenkins copy of head still contains ntp_parser.y. I see no issue >> here. >> >> Cy Schubert >> <cy.schub...@cschubert.com> >> >> You need to do a better job at figuring out why you broke things. >> This is not the first time you've broken things with an ntp import. > > I don’t see that file either… > > $ ls contrib/ntp/ntpd/ntp_parser.y > ls: contrib/ntp/ntpd/ntp_parser.y: No such file or directory > $ svnversion > 282423M > $ svn status | grep -v \? > M Makefile > > Thanks, > -NGie That being said, I’m not sure if checking in bison-processed source was the wisest thing in the world (contrib/ntp/ntpd/ntp_parser.c). Thanks! _______________________________________________ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"
