Build failed in Jenkins: FreeBSD_HEAD #2725

[cschuber at gmail.com]

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-admin at freebsd.org; freebsd-current Current
Subject: Re: Build failed in Jenkins: FreeBSD_HEAD #2725

On May 4, 2015, at 10:46, Garrett Cooper <yaneurabeya at gmail.com> wrote:

> 
> On May 4, 2015, at 10:38, Cy Schubert <Cy.Schubert at komquats.com> wrote:
> 
>> I think jenkins copy of head still contains ntp_parser.y. I see no issue
>> here.
>> 
>> Cy Schubert
>> <Cy.Schubert at 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!