Re: Build failed in Jenkins: FreeBSD_HEAD #2725

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!