Re: pkg/50279 (emulators/tme build failure on NetBSD/amd64 7.0)

[Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>]

The following reply was made to PR pkg/50279; it has been noted by GNATS.

From: Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>
To: joerg%britannica.bec.de@localhost
Cc: joerg%NetBSD.org@localhost, gnats-bugs%NetBSD.org@localhost, pkg-manager%netbsd.org@localhost,
        gnats-admin%netbsd.org@localhost, bsiegert%NetBSD.org@localhost, tsutsui%ceres.dti.ne.jp@localhost
Subject: Re: pkg/50279 (emulators/tme build failure on NetBSD/amd64 7.0)
Date: Thu, 15 Oct 2015 01:37:31 +0900

 joerg@ wrote:
 
 > > > I am doing my testing on NetBSD current with clang. I don't know right
 > > > now what is pulling in stdint.h. I am not saying it is enough, just that
 > > > I can't reproduce the problem in my test environment.
 > > 
 > > Why didn't you write it in your commit log?
 > > What you wrote in the log ("explicitly") was very confusing.
 > >  >> Explicitly pull in limits.h, it sometimes hasn't been included yet.
 > > 
 > > Anyway, please don't commit random changes you can't test.
 > 
 > Fine, I'll just leave things broken the next time.
 
 If you mean the original "(((type) 0) - ((type) 1))" was broken,
 it's fine to leave it.
 
 Making build broken even on the default NetBSD environment is
 much worse than that, i.e you should revert your untested change.
 
 ---
 Izumi Tsutsui